commit aacd1d7eda186c0b32d2736c0f38d9749b6006b8
author jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Mon Jul 23 17:35:09 2012
committer jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Mon Jul 23 17:35:09 2012
tree 0ab3883448a1d082f5a78cad4a08577530364ddf
parent 87d5c05993b8a457cbf0cce00ec97fc3fcc1d270

Fix regression of bug 205 where a plugin in a window shown with window.open may not have the correct parent window. PluginTest.OpenPopupWindowWithPlugin covers this, but this bug is a race condition which is why the test works most of the time (although it shows up on the flakiness dashboard). When I tried to move this test to content_browsertests, the timing was different so it showed up most of the time.

BUG=205
Review URL: https://chromiumcodereview.appspot.com/10809051

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147874 0039d316-1c4b-4281-b951-d872f2087c98

content/browser/web_contents/web_contents_impl.cc

diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 785a1742..a6c6dca 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -1146,6 +1146,7 @@
   if (delegate_ && !delegate_->ShouldCreateWebContents(
           this, route_id, params.window_container_type, params.frame_name,
           params.target_url)) {
+    GetRenderViewHost()->GetProcess()->ResumeRequestsForView(route_id);
     return;
   }