DescriptionUse content::NavigateIframeToURL in ProcessManagerBrowserTest
In 1413853005 I introduced NavigateIframeToURLAndWait in a test because
the current content::NavigateIframeToURL implementation does not account
for nested frames. https://codereview.chromium.org/1522443004/ fixes the
issue, so let's use it.
ChromeSitePerProcessTest.PluginWithRemoteTopFrame has been updated
because I noticed that the flash object that was supposed to be loaded
did not load because the file path was incorrect.
Committed: https://crrev.com/57afbcaf67478de645c7e27040795f209057c244
Cr-Commit-Position: refs/heads/master@{#366416}
Patch Set 1 #Patch Set 2 : Re-introduce wait-for-commit + fix PluginWithRemoteTopFrame test #Patch Set 3 : Stick to onload/onerror events #Patch Set 4 : Avoid 2 concurrent domAutomationController to pass ChromeSitePerProcessTest.PluginWithRemoteTopFrame #
Total comments: 2
Patch Set 5 : Rebase and use content::NavigateIframeToURL from https://codereview.chromium.org/1522443004/ #
Total comments: 5
Patch Set 6 : Remove paranoid onload check #
Messages
Total messages: 25 (11 generated)
|