Merge "Fix HTTP redirect missing statuscode 303 and 307"

Jean-Baptiste Queru · Android Code Review · commit 6cbca108936d · 2010-12-01T08:25:12.000-08:00
diff --git a/media/libstagefright/HTTPDataSource.cpp b/media/libstagefright/HTTPDataSource.cpp
@@ -88,7 +88,7 @@ status_t HTTPDataSource::connectWithRedirectsAndRange(off_t rangeStart) {
             return OK;
         }
 
-        if (httpStatus != 301 && httpStatus != 302) {
+        if (httpStatus != 301 && httpStatus != 302 && httpStatus != 303 && httpStatus != 307) {
             LOGE("HTTP request failed w/ http status %d", httpStatus);
             return ERROR_IO;
         }

Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,7 @@ status_t HTTPDataSource::connectWithRedirectsAndRange(off_t rangeStart) {`
`88`	`88`	`return OK;`
`89`	`89`	`}`
`90`	`90`
`91`		`- if (httpStatus != 301 && httpStatus != 302) {`
	`91`	`+ if (httpStatus != 301 && httpStatus != 302 && httpStatus != 303 && httpStatus != 307) {`
`92`	`92`	`LOGE("HTTP request failed w/ http status %d", httpStatus);`
`93`	`93`	`return ERROR_IO;`
`94`	`94`	`}`