Title: [191051] branches/safari-601-branch/Source/WebCore
Revision
191051
Author
matthew_han...@apple.com
Date
2015-10-14 11:49:30 -0700 (Wed, 14 Oct 2015)

Log Message

Merge r189979. rdar://problem/23075525

Modified Paths

Diff

Modified: branches/safari-601-branch/Source/WebCore/ChangeLog (191050 => 191051)


--- branches/safari-601-branch/Source/WebCore/ChangeLog	2015-10-14 18:49:28 UTC (rev 191050)
+++ branches/safari-601-branch/Source/WebCore/ChangeLog	2015-10-14 18:49:30 UTC (rev 191051)
@@ -1,5 +1,28 @@
 2015-10-13  Matthew Hanson  <matthew_han...@apple.com>
 
+        Merge r189979. rdar://problem/23075525
+
+    2015-09-18  Chris Dumez  <cdu...@apple.com>
+
+            WebContent crash in WebCore::MemoryPressureHandler::releaseCriticalMemory() with GuardMalloc when preparing to suspend
+            https://bugs.webkit.org/show_bug.cgi?id=149350
+
+            Reviewed by Antti Koivisto.
+
+            in MemoryPressureHandler::releaseCriticalMemory(), iterate over a copy of
+            Document::allDocuments() instead of iterating over allDocuments() directly.
+            Also make sure the Documents are ref'd inside the copy.
+
+            This is needed because clearing the StyleResolver of a Document may cause
+            Documents to be unref'd and removed from the allDocument() HashSet.
+
+            No new tests, already covered by existing tests.
+
+            * platform/MemoryPressureHandler.cpp:
+            (WebCore::MemoryPressureHandler::releaseCriticalMemory):
+
+2015-10-13  Matthew Hanson  <matthew_han...@apple.com>
+
         Merge r189421. rdar://problem/22802049
 
     2015-09-04  Myles C. Maxfield  <mmaxfi...@apple.com>

Modified: branches/safari-601-branch/Source/WebCore/platform/MemoryPressureHandler.cpp (191050 => 191051)


--- branches/safari-601-branch/Source/WebCore/platform/MemoryPressureHandler.cpp	2015-10-14 18:49:28 UTC (rev 191050)
+++ branches/safari-601-branch/Source/WebCore/platform/MemoryPressureHandler.cpp	2015-10-14 18:49:30 UTC (rev 191051)
@@ -129,7 +129,9 @@
 
     {
         ReliefLogger log("Discard StyleResolvers");
-        for (auto* document : Document::allDocuments())
+        Vector<RefPtr<Document>> documents;
+        copyToVector(Document::allDocuments(), documents);
+        for (auto& document : documents)
             document->clearStyleResolver();
     }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to