Title: [95013] trunk/Source/WebCore
Revision
95013
Author
joep...@webkit.org
Date
2011-09-12 21:41:42 -0700 (Mon, 12 Sep 2011)

Log Message

Possible ASSERT(!storageTracker) in Lazily Initialized StorageTracker
https://bugs.webkit.org/show_bug.cgi?id=67957

Reviewed by Darin Adler.

The storageTracker singleton may already have been created
through StorageTracker::tracker before initializeTracker
gets called. Update the ASSERT in this case to check that
there is no client for the tracker.

No test, this depends on how the port initializes the tracker.

* storage/StorageTracker.cpp:
(WebCore::StorageTracker::initializeTracker):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (95012 => 95013)


--- trunk/Source/WebCore/ChangeLog	2011-09-13 04:15:59 UTC (rev 95012)
+++ trunk/Source/WebCore/ChangeLog	2011-09-13 04:41:42 UTC (rev 95013)
@@ -1,3 +1,20 @@
+2011-09-12  Joseph Pecoraro  <joep...@webkit.org>
+
+        Possible ASSERT(!storageTracker) in Lazily Initialized StorageTracker
+        https://bugs.webkit.org/show_bug.cgi?id=67957
+
+        Reviewed by Darin Adler.
+
+        The storageTracker singleton may already have been created
+        through StorageTracker::tracker before initializeTracker
+        gets called. Update the ASSERT in this case to check that
+        there is no client for the tracker.
+
+        No test, this depends on how the port initializes the tracker.
+
+        * storage/StorageTracker.cpp:
+        (WebCore::StorageTracker::initializeTracker):
+
 2011-09-12  Adam Klein  <ad...@chromium.org>
 
         Fix out-of-bounds access in Gradient::sortStopsIfNecessary

Modified: trunk/Source/WebCore/storage/StorageTracker.cpp (95012 => 95013)


--- trunk/Source/WebCore/storage/StorageTracker.cpp	2011-09-13 04:15:59 UTC (rev 95012)
+++ trunk/Source/WebCore/storage/StorageTracker.cpp	2011-09-13 04:41:42 UTC (rev 95013)
@@ -51,7 +51,7 @@
 void StorageTracker::initializeTracker(const String& storagePath, StorageTrackerClient* client)
 {
     ASSERT(isMainThread());
-    ASSERT(!storageTracker);
+    ASSERT(!storageTracker || !storageTracker->m_client);
     
     if (!storageTracker)
         storageTracker = new StorageTracker(storagePath);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to