Title: [188781] branches/safari-601.1.46-branch/Source/WebCore
Revision
188781
Author
[email protected]
Date
2015-08-21 14:53:16 -0700 (Fri, 21 Aug 2015)

Log Message

Merged r187758.  rdar://problem/22375884

Modified Paths

Diff

Modified: branches/safari-601.1.46-branch/Source/WebCore/ChangeLog (188780 => 188781)


--- branches/safari-601.1.46-branch/Source/WebCore/ChangeLog	2015-08-21 21:52:01 UTC (rev 188780)
+++ branches/safari-601.1.46-branch/Source/WebCore/ChangeLog	2015-08-21 21:53:16 UTC (rev 188781)
@@ -1,5 +1,22 @@
 2015-08-21  Babak Shafiei  <[email protected]>
 
+        Merge r187758.
+
+    2015-08-03  Anders Carlsson  <[email protected]>
+
+            Safari is failing to remove SQLite Databases via Remove All Website Data
+            https://bugs.webkit.org/show_bug.cgi?id=147584
+            rdar://problem/22095006
+
+            Reviewed by Brady Eidson.
+
+            If we end up deleting every single database for an origin, make sure to also delete the origin.
+
+            * Modules/webdatabase/DatabaseTracker.cpp:
+            (WebCore::DatabaseTracker::deleteDatabasesModifiedSince):
+
+2015-08-21  Babak Shafiei  <[email protected]>
+
         Merge r187685.
 
     2015-07-31  Myles C. Maxfield  <[email protected]>

Modified: branches/safari-601.1.46-branch/Source/WebCore/Modules/webdatabase/DatabaseTracker.cpp (188780 => 188781)


--- branches/safari-601.1.46-branch/Source/WebCore/Modules/webdatabase/DatabaseTracker.cpp	2015-08-21 21:52:01 UTC (rev 188780)
+++ branches/safari-601.1.46-branch/Source/WebCore/Modules/webdatabase/DatabaseTracker.cpp	2015-08-21 21:53:16 UTC (rev 188781)
@@ -862,6 +862,8 @@
         if (!databaseNamesForOrigin(origin.get(), databaseNames))
             continue;
 
+        size_t deletedDatabases = 0;
+
         for (auto& databaseName : databaseNames) {
             auto fullPath = fullPathForDatabase(origin.get(), databaseName, false);
 
@@ -873,7 +875,11 @@
                 continue;
 
             deleteDatabase(origin.get(), databaseName);
+            ++deletedDatabases;
         }
+
+        if (deletedDatabases == databaseNames.size())
+            deleteOrigin(origin.get());
     }
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to