Title: [152340] trunk/Source/WebKit2
Revision
152340
Author
[email protected]
Date
2013-07-02 22:54:31 -0700 (Tue, 02 Jul 2013)

Log Message

[WK2] Invalidate FontCache before purging MemoryCache upon WebProcess termination/closure
https://bugs.webkit.org/show_bug.cgi?id=118280

Reviewed by Darin Adler.

Invalidate the FontCache before disabling and purging the MemoryCache in WebProcess::didClose
and WebProcess::terminate. This frees up additional references to objects that were held
by the FontCache, reducing the amount of 'LEAK: *' output when exiting MiniBrowser or
WebKitTestRunner in debug builds.

* WebProcess/WebProcess.cpp:
(WebKit::WebProcess::terminate):
(WebKit::WebProcess::didClose):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (152339 => 152340)


--- trunk/Source/WebKit2/ChangeLog	2013-07-03 05:30:03 UTC (rev 152339)
+++ trunk/Source/WebKit2/ChangeLog	2013-07-03 05:54:31 UTC (rev 152340)
@@ -1,3 +1,19 @@
+2013-07-02  Zan Dobersek  <[email protected]>
+
+        [WK2] Invalidate FontCache before purging MemoryCache upon WebProcess termination/closure
+        https://bugs.webkit.org/show_bug.cgi?id=118280
+
+        Reviewed by Darin Adler.
+
+        Invalidate the FontCache before disabling and purging the MemoryCache in WebProcess::didClose
+        and WebProcess::terminate. This frees up additional references to objects that were held
+        by the FontCache, reducing the amount of 'LEAK: *' output when exiting MiniBrowser or
+        WebKitTestRunner in debug builds.
+
+        * WebProcess/WebProcess.cpp:
+        (WebKit::WebProcess::terminate):
+        (WebKit::WebProcess::didClose):
+
 2013-07-02  Jessie Berlin  <[email protected]>
 
         The callback for WKPageGetPlugInInformation needs info about whether or not there are any

Modified: trunk/Source/WebKit2/WebProcess/WebProcess.cpp (152339 => 152340)


--- trunk/Source/WebKit2/WebProcess/WebProcess.cpp	2013-07-03 05:30:03 UTC (rev 152339)
+++ trunk/Source/WebKit2/WebProcess/WebProcess.cpp	2013-07-03 05:54:31 UTC (rev 152340)
@@ -616,6 +616,7 @@
 {
 #ifndef NDEBUG
     gcController().garbageCollectNow();
+    fontCache()->invalidate();
     memoryCache()->setDisabled(true);
 #endif
 
@@ -668,6 +669,7 @@
     pages.clear();
 
     gcController().garbageCollectSoon();
+    fontCache()->invalidate();
     memoryCache()->setDisabled(true);
 #endif    
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to