| commit | 777bb78a111aee919e07f5206267915a87639f88 | |
| author | gsennton <gsennton@chromium.org> | |
| Thu, 16 Jul 2015 20:12:49 +0000 (16 13:12 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Thu, 16 Jul 2015 20:13:36 +0000 (16 20:13 +0000) | ||
| tree | 852c8f2cf62748fc0c28926fe6967e74f01b0cbc | treesnapshot (tar.gz zip) |
| parent | b839eab3001b70e7489661dfadffd3ecb3386ab5 | commitdiff |
Use resource throttle to implement shouldOverrideUrlLoading, core change
We have been using both a resource throttle and a sync IPC to implement
shouldOverrideUrlLoading, with this patch we use only a resource
throttle instead.
This patch depends on https://codereview.chromium.org/1194383003 which
adds a flag indicating that the current request was overridden. The flag
is passed down to AwWebContentsObserver.didFailLoad. That CL in turn
depends on https://codereview.chromium.org/1178273007/ which adds the
flag to blink errors.
BUG=325351
Review URL: https://codereview.chromium.org/1155713005
Cr-Commit-Position: refs/heads/master@{#339109}
We have been using both a resource throttle and a sync IPC to implement
shouldOverrideUrlLoading, with this patch we use only a resource
throttle instead.
This patch depends on https://codereview.chromium.org/1194383003 which
adds a flag indicating that the current request was overridden. The flag
is passed down to AwWebContentsObserver.didFailLoad. That CL in turn
depends on https://codereview.chromium.org/1178273007/ which adds the
flag to blink errors.
BUG=325351
Review URL: https://codereview.chromium.org/1155713005
Cr-Commit-Position: refs/heads/master@{#339109}
15 files changed: