Title: [163603] trunk/Source/WebKit2
Revision
163603
Author
[email protected]
Date
2014-02-06 18:41:10 -0800 (Thu, 06 Feb 2014)

Log Message

Crash in MiniBrowser when using Command-W to close a WK2 window
https://bugs.webkit.org/show_bug.cgi?id=128355
<rdar://problem/16008531>

Reviewed by Simon Fraser.

Don't set the API clients to nullptr when closing the page, just create new empty ones instead.
Too much code depends on the clients never being null.

* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::close):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (163602 => 163603)


--- trunk/Source/WebKit2/ChangeLog	2014-02-07 02:30:11 UTC (rev 163602)
+++ trunk/Source/WebKit2/ChangeLog	2014-02-07 02:41:10 UTC (rev 163603)
@@ -1,3 +1,17 @@
+2014-02-06  Anders Carlsson  <[email protected]>
+
+        Crash in MiniBrowser when using Command-W to close a WK2 window
+        https://bugs.webkit.org/show_bug.cgi?id=128355
+        <rdar://problem/16008531>
+
+        Reviewed by Simon Fraser.
+
+        Don't set the API clients to nullptr when closing the page, just create new empty ones instead.
+        Too much code depends on the clients never being null.
+
+        * UIProcess/WebPageProxy.cpp:
+        (WebKit::WebPageProxy::close):
+
 2014-02-03  Tim Horton  <[email protected]>
 
         [wk2] iOS should use accelerated drawing by default, except in the simulator

Modified: trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp (163602 => 163603)


--- trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp	2014-02-07 02:30:11 UTC (rev 163602)
+++ trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp	2014-02-07 02:41:10 UTC (rev 163603)
@@ -578,10 +578,10 @@
 
     resetState();
 
-    m_loaderClient = nullptr;
-    m_policyClient = nullptr;
+    m_loaderClient = std::make_unique<API::LoaderClient>();
+    m_policyClient = std::make_unique<API::PolicyClient>();
     m_formClient.initialize(0);
-    m_uiClient = nullptr;
+    m_uiClient = std::make_unique<API::UIClient>();
 #if PLATFORM(EFL)
     m_uiPopupMenuClient.initialize(0);
 #endif
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to