Fix not resuming a request after a plugin calls NPN_URLRedirectResponse.

Fix not resuming a request after a plugin calls NPN_URLRedirectResponse.

BUG=301510
TEST=http://www.dbtv.no/?vid=2750870727001 plays
R=scottmg@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=233516

[scottmg, 2013-11-07 04:48:55]

the test url plays ok for me in Canary, is that expected?

https://codereview.chromium.org/63643003/diff/1/content/child/npapi/plugin_ur...
File content/child/npapi/plugin_url_fetcher.cc (right):

https://codereview.chromium.org/63643003/diff/1/content/child/npapi/plugin_ur...
content/child/npapi/plugin_url_fetcher.cc:148: bridge_->SetDefersLoading(false);
this is the response to OnReceivedRedirect?

[scottmg, 2013-11-07 04:49:23]

(lgtm in any case, i'm assuming you had some non-working case)

[jam, 2013-11-07 04:58:18]

On 2013/11/07 04:48:55, scottmg wrote:
> the test url plays ok for me in Canary, is that expected?

This is only busted with NPAPI Flash, I assume you were using Pepper Flash?

https://codereview.chromium.org/63643003/diff/1/content/child/npapi/plugin_ur...
File content/child/npapi/plugin_url_fetcher.cc (right):

https://codereview.chromium.org/63643003/diff/1/content/child/npapi/plugin_ur...
content/child/npapi/plugin_url_fetcher.cc:148: bridge_->SetDefersLoading(false);
On 2013/11/07 04:48:55, scottmg wrote:
> this is the response to OnReceivedRedirect?

this is called by the plugin after we hit a redirect and had paused the request
while waiting for the plugin to call us back and let us know whether to follow
the redirect or not

[scottmg, 2013-11-07 05:01:27]

On 2013/11/07 04:58:18, jam wrote:
> On 2013/11/07 04:48:55, scottmg wrote:
> > the test url plays ok for me in Canary, is that expected?
> 
> This is only busted with NPAPI Flash, I assume you were using Pepper Flash?

ah, of course "NPN_". yes.

> 
>
https://codereview.chromium.org/63643003/diff/1/content/child/npapi/plugin_ur...
> File content/child/npapi/plugin_url_fetcher.cc (right):
> 
>
https://codereview.chromium.org/63643003/diff/1/content/child/npapi/plugin_ur...
> content/child/npapi/plugin_url_fetcher.cc:148:
bridge_->SetDefersLoading(false);
> On 2013/11/07 04:48:55, scottmg wrote:
> > this is the response to OnReceivedRedirect?
> 
> this is called by the plugin after we hit a redirect and had paused the
request
> while waiting for the plugin to call us back and let us know whether to follow
> the redirect or not

got it, lgtm.