Index: content/renderer/pepper/pepper_url_loader_host.cc |
diff --git a/content/renderer/pepper/pepper_url_loader_host.cc b/content/renderer/pepper/pepper_url_loader_host.cc |
index 94a03ae7683e1b1515eb98e29aa0c9b1255b84a0..fb602f4f407c55711781ac10bfd9ae957b4e9fb3 100644 |
--- a/content/renderer/pepper/pepper_url_loader_host.cc |
+++ b/content/renderer/pepper/pepper_url_loader_host.cc |
@@ -120,7 +120,7 @@ int32_t PepperURLLoaderHost::OnResourceMessageReceived( |
return PP_ERROR_FAILED; |
} |
-void PepperURLLoaderHost::willFollowRedirect( |
+bool PepperURLLoaderHost::willFollowRedirect( |
WebURLLoader* loader, |
WebURLRequest& new_request, |
const WebURLResponse& redirect_response, |
@@ -130,6 +130,7 @@ void PepperURLLoaderHost::willFollowRedirect( |
SaveResponse(redirect_response); |
SetDefersLoading(true); |
bbudge
2016/09/16 18:13:03
Should you return false here? Perhaps comment the
tyoshino (SeeGerritForStatus)
2016/09/26 14:48:41
This code originates from this patch you authored.
bbudge
2016/09/26 15:46:31
The linked patch is by brettw. But I agree with yo
tyoshino (SeeGerritForStatus)
2016/09/29 13:40:22
Oops!
|
} |
+ return true; |
} |
void PepperURLLoaderHost::didSendData( |