| commit | b97abf827d0e9576951364fa86c58561e5530de9 | |
| author | alexmos <alexmos@chromium.org> | |
| Wed, 1 Oct 2014 01:08:12 +0000 (30 18:08 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Wed, 1 Oct 2014 01:09:32 +0000 (1 01:09 +0000) | ||
| tree | 064be5c59f912fc505b523da46203a2ca3711174 | treesnapshot (tar.gz zip) |
| parent | ffbb37ebec861db6e93471a5837727ea8d974f91 | commitdiff |
Let data URLs stay in same process in RenderFrameImpl::DecidePolicyForNavigation.
Currently, several tests in content_browsertests fail under --site-per-process when they use LoadHTML(), which times out. The reason is that LoadHTML navigates the top frame to a data URL, but RenderFrameImpl::DecidePolicyForNavigation decides that the URL should be rendered in a different process and passes back a WebNavigationPolicyIgnore. This CL tweaks the logic in DecidePolicyForNavigation to allow data URLs to stay in the same process.
BUG=417518
Review URL: https://codereview.chromium.org/619613004
Cr-Commit-Position: refs/heads/master@{#297568}
Currently, several tests in content_browsertests fail under --site-per-process when they use LoadHTML(), which times out. The reason is that LoadHTML navigates the top frame to a data URL, but RenderFrameImpl::DecidePolicyForNavigation decides that the URL should be rendered in a different process and passes back a WebNavigationPolicyIgnore. This CL tweaks the logic in DecidePolicyForNavigation to allow data URLs to stay in the same process.
BUG=417518
Review URL: https://codereview.chromium.org/619613004
Cr-Commit-Position: refs/heads/master@{#297568}
| content/renderer/render_frame_impl.cc | diffblobblamehistory |