Changeset 148054 in webkit

Timestamp:

Apr 9, 2013 2:35:39 PM (11 years ago)

Author:

andersca@apple.com

Message:

REGRESSION (r147454): Youtube annotation links to new window broken
​https://bugs.webkit.org/show_bug.cgi?id=114242
<rdar://problem/13609940>

Reviewed by Beth Dakin.

Set the current user gesture to "DefinitelyProcessingNewUserGesture" if popups are allowed.

• WebProcess/Plugins/PluginView.cpp:

(WebKit::PluginView::performFrameLoadURLRequest):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• WebProcess/Plugins/PluginView.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2013-04-09  Anders Carlsson  <andersca@apple.com>
+
+        REGRESSION (r147454): Youtube annotation links to new window broken
+        https://bugs.webkit.org/show_bug.cgi?id=114242
+        <rdar://problem/13609940>
+
+        Reviewed by Beth Dakin.
+
+        Set the current user gesture to "DefinitelyProcessingNewUserGesture" if popups are allowed.
+
+        * WebProcess/Plugins/PluginView.cpp:
+        (WebKit::PluginView::performFrameLoadURLRequest):
+
 2013-04-09  Anders Carlsson  <andersca@apple.com>
 

trunk/Source/WebKit2/WebProcess/Plugins/PluginView.cpp

@@ -1080 +1080 @@
     }
 
+    UserGestureIndicator gestureIndicator(request->allowPopups() ? DefinitelyProcessingNewUserGesture : PossiblyProcessingUserGesture);
+
     // First, try to find a target frame.
     Frame* targetFrame = frame->loader()->findFrameForNavigation(request->target());