Changeset 57754 in webkit

Timestamp:

Apr 16, 2010 3:11:10 PM (14 years ago)

Author:

dumi@chromium.org

Message:

Make Safari correctly allocate 5MB of DB storage to all new
origins.
​https://bugs.webkit.org/show_bug.cgi?id=36671

Reviewed by Alexey Proskuryakov.

Eric fixed the same problem in
DatabaseTracker::fullPathForDatabaseNoLock() in r57128, but forgot
to fix it in DatabaseTracker::detailsForNameAndOrigin() too.

• storage/DatabaseTracker.cpp:

(WebCore::DatabaseTracker::detailsForNameAndOrigin):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• storage/DatabaseTracker.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-04-16  Dumitru Daniliuc  <dumi@chromium.org>
+
+        Reviewed by Alexey Proskuryakov.
+
+        Make Safari correctly allocate 5MB of DB storage to all new
+        origins.
+        https://bugs.webkit.org/show_bug.cgi?id=36671
+
+        Eric fixed the same problem in
+        DatabaseTracker::fullPathForDatabaseNoLock() in r57128, but forgot
+        to fix it in DatabaseTracker::detailsForNameAndOrigin() too.
+
+        * storage/DatabaseTracker.cpp:
+        (WebCore::DatabaseTracker::detailsForNameAndOrigin):
+
 2010-04-16  Kinuko Yasuda  <kinuko@chromium.org>
 

trunk/WebCore/storage/DatabaseTracker.cpp

@@ -376 +376 @@
 
         for (HashSet<ProposedDatabase*>::iterator iter = m_proposedDatabases.begin(); iter != m_proposedDatabases.end(); ++iter)
-            if ((*iter)->first == origin && (*iter)->second.name() == name) {
+            if ((*iter)->second.name() == name && (*iter)->first->equal(origin)) {
                 ASSERT((*iter)->second.thread() == currentThread());
                 return (*iter)->second;