Consider redirects to about:blank as canceling and don't flag conflicts

Consider redirects to about:blank as canceling and don't flag conflicts

If one extension redirects a webrequest to a different URL (e.g. from http://foo.com to https://foo.com) and another extension redirects to about:blank, the latter wants to express that it intends to cancel the request. With this CL we give precedence to cancelling requests by redirecting to about:blank and don't flag them redirect conflicts.


BUG=111700
TEST=no


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=129117

[battre, 2012-03-26 12:43:25]

Hi Matt,

please review this CL. I think this is the right thing to do even if we add
{cancelWithTransparentImage: true} and {cancelWithEmptyPage: true} as valid
return values for the onBeforeRequest handler.

Best regards,
Dominic

[Matt Perry, 2012-03-26 18:54:05]

lgtm

[commit-bot: I haz the power, 2012-03-26 19:47:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/battre@chromium.org/9857009/1003

[commit-bot: I haz the power, 2012-03-26 22:28:40]

Try job failure for 9857009-1003 (retry) on win_rel for step "browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2012-03-26 22:32:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/battre@chromium.org/9857009/1003

[battre, 2012-03-26 22:33:24]

On 2012/03/26 22:28:40, I haz the power (commit-bot) wrote:
> Try job failure for 9857009-1003 (retry) on win_rel for step "browser_tests".
> It's a second try, previously, step "browser_tests" failed.
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

Matt, could you please continue pressing the button? It looks like our CQ has a
bad day again. Thanks.

[Matt Perry, 2012-03-26 22:34:29]

On 2012/03/26 22:33:24, battre wrote:
> On 2012/03/26 22:28:40, I haz the power (commit-bot) wrote:
> > Try job failure for 9857009-1003 (retry) on win_rel for step
"browser_tests".
> > It's a second try, previously, step "browser_tests" failed.
> >
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...
> 
> Matt, could you please continue pressing the button? It looks like our CQ has
a
> bad day again. Thanks.

Will do.

[commit-bot: I haz the power, 2012-03-27 03:07:32]

Change committed as 129117