Title: [140220] trunk/Source/WebKit2
Revision
140220
Author
a...@apple.com
Date
2013-01-18 16:54:36 -0800 (Fri, 18 Jan 2013)

Log Message

        <rdar://problem/13042826> REGRESSION (r139985): Full Screen Youtube loses keyboard and mouse control
        https://bugs.webkit.org/show_bug.cgi?id=107191

        Reviewed by Darin Adler.

        Moved initialization from platformInitializePluginProcess to platformInitializeProcess,
        because the former is called too late. In r139985, I used a correct location for
        WebProcess, but not for PluginProcess.

        * PluginProcess/mac/PluginProcessMac.mm:
        (WebKit::PluginProcess::platformInitializeProcess):
        (WebKit::PluginProcess::platformInitializePluginProcess):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (140219 => 140220)


--- trunk/Source/WebKit2/ChangeLog	2013-01-19 00:43:58 UTC (rev 140219)
+++ trunk/Source/WebKit2/ChangeLog	2013-01-19 00:54:36 UTC (rev 140220)
@@ -1,3 +1,18 @@
+2013-01-18  Alexey Proskuryakov  <a...@apple.com>
+
+        <rdar://problem/13042826> REGRESSION (r139985): Full Screen Youtube loses keyboard and mouse control
+        https://bugs.webkit.org/show_bug.cgi?id=107191
+
+        Reviewed by Darin Adler.
+
+        Moved initialization from platformInitializePluginProcess to platformInitializeProcess,
+        because the former is called too late. In r139985, I used a correct location for
+        WebProcess, but not for PluginProcess.
+
+        * PluginProcess/mac/PluginProcessMac.mm:
+        (WebKit::PluginProcess::platformInitializeProcess):
+        (WebKit::PluginProcess::platformInitializePluginProcess):
+
 2013-01-18  Brady Eidson  <beid...@apple.com>
 
         Rework NetworkProcess resource load identifiers.

Modified: trunk/Source/WebKit2/PluginProcess/mac/PluginProcessMac.mm (140219 => 140220)


--- trunk/Source/WebKit2/PluginProcess/mac/PluginProcessMac.mm	2013-01-19 00:43:58 UTC (rev 140219)
+++ trunk/Source/WebKit2/PluginProcess/mac/PluginProcessMac.mm	2013-01-19 00:54:36 UTC (rev 140220)
@@ -272,6 +272,8 @@
 
 void PluginProcess::platformInitializeProcess(const ChildProcessInitializationParameters&)
 {
+    RunLoop::setUseApplicationRunLoopOnMainRunLoop();
+
 #if defined(__i386__)
     // Initialize the shim.
     initializeShim();
@@ -340,8 +342,6 @@
 
 void PluginProcess::platformInitializePluginProcess(const PluginProcessCreationParameters& parameters)
 {
-    RunLoop::setUseApplicationRunLoopOnMainRunLoop();
-
     m_compositingRenderServerPort = parameters.acceleratedCompositingPort.port();
 
     NSString *applicationName = [NSString stringWithFormat:WEB_UI_STRING("%@ (%@ Internet plug-in)",
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to