Title: [132402] trunk/Source/WebKit/mac
Revision
132402
Author
timo...@apple.com
Date
2012-10-24 14:37:19 -0700 (Wed, 24 Oct 2012)

Log Message

Fix a crash seen during the Inspector tests on the WebKit1 bots.

Reviewed by Filip Pizlo.

* WebCoreSupport/WebInspectorClient.mm:
(WebInspectorFrontendClient::bringToFront): Use the window from the WebView since m_windowController's window
is not the same when the Inspector is docked.

Modified Paths

Diff

Modified: trunk/Source/WebKit/mac/ChangeLog (132401 => 132402)


--- trunk/Source/WebKit/mac/ChangeLog	2012-10-24 21:36:57 UTC (rev 132401)
+++ trunk/Source/WebKit/mac/ChangeLog	2012-10-24 21:37:19 UTC (rev 132402)
@@ -1,3 +1,13 @@
+2012-10-24  Timothy Hatcher  <timo...@apple.com>
+
+        Fix a crash seen during the Inspector tests on the WebKit1 bots.
+
+        Reviewed by Filip Pizlo.
+
+        * WebCoreSupport/WebInspectorClient.mm:
+        (WebInspectorFrontendClient::bringToFront): Use the window from the WebView since m_windowController's window
+        is not the same when the Inspector is docked.
+
 2012-10-24  Brady Eidson  <beid...@apple.com>
 
         Add a strategy for loader customization.

Modified: trunk/Source/WebKit/mac/WebCoreSupport/WebInspectorClient.mm (132401 => 132402)


--- trunk/Source/WebKit/mac/WebCoreSupport/WebInspectorClient.mm	2012-10-24 21:36:57 UTC (rev 132401)
+++ trunk/Source/WebKit/mac/WebCoreSupport/WebInspectorClient.mm	2012-10-24 21:37:19 UTC (rev 132402)
@@ -200,8 +200,13 @@
 void WebInspectorFrontendClient::bringToFront()
 {
     updateWindowTitle();
+
     [m_windowController.get() showWindow:nil];
-    [[m_windowController.get() window] makeFirstResponder:[m_windowController.get() webView]];
+
+    // Use the window from the WebView since m_windowController's window
+    // is not the same when the Inspector is docked.
+    WebView *webView = [m_windowController.get() webView];
+    [[webView window] makeFirstResponder:webView];
 }
 
 void WebInspectorFrontendClient::closeWindow()
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to