Title: [142117] trunk/Source/WebKit/blackberry
Revision
142117
Author
[email protected]
Date
2013-02-07 07:08:22 -0800 (Thu, 07 Feb 2013)

Log Message

[BlackBerry] Reader Mode: Opening two links quickly from reader mode causes browser bad state
https://bugs.webkit.org/show_bug.cgi?id=109124

Patch by Sean Wang <[email protected]> on 2013-02-07
Reviewed by Yong Li.

RIM BUG 291246 Internally reviewed by YongLi.
Move the WebPageGroupLoadDeferrer object from ChromeClientBlackBerry::createWindow()
into WebPageClientImpl::createWindow() to make it more close to its protecting place.

* WebCoreSupport/ChromeClientBlackBerry.cpp:
(WebCore::ChromeClientBlackBerry::createWindow):

Modified Paths

Diff

Modified: trunk/Source/WebKit/blackberry/ChangeLog (142116 => 142117)


--- trunk/Source/WebKit/blackberry/ChangeLog	2013-02-07 15:07:03 UTC (rev 142116)
+++ trunk/Source/WebKit/blackberry/ChangeLog	2013-02-07 15:08:22 UTC (rev 142117)
@@ -1,3 +1,17 @@
+2013-02-07  Sean Wang  <[email protected]>
+
+        [BlackBerry] Reader Mode: Opening two links quickly from reader mode causes browser bad state
+        https://bugs.webkit.org/show_bug.cgi?id=109124
+
+        Reviewed by Yong Li.
+
+        RIM BUG 291246 Internally reviewed by YongLi.
+        Move the WebPageGroupLoadDeferrer object from ChromeClientBlackBerry::createWindow()
+        into WebPageClientImpl::createWindow() to make it more close to its protecting place.
+
+        * WebCoreSupport/ChromeClientBlackBerry.cpp:
+        (WebCore::ChromeClientBlackBerry::createWindow):
+
 2013-02-07  Mike Fenton  <[email protected]>
 
         [BlackBerry] Send type details with IMF mask as part of focus gained.

Modified: trunk/Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.cpp (142116 => 142117)


--- trunk/Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.cpp	2013-02-07 15:07:03 UTC (rev 142116)
+++ trunk/Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.cpp	2013-02-07 15:08:22 UTC (rev 142117)
@@ -236,9 +236,6 @@
         return 0;
 #endif
 
-    PageGroupLoadDeferrer deferrer(m_webPagePrivate->m_page, true);
-    TimerBase::fireTimersInNestedEventLoop();
-
     int x = features.xSet ? features.x : 0;
     int y = features.ySet ? features.y : 0;
     int width = features.widthSet? features.width : -1;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to