Attachment 341474 Details for Bug 185835 – Patch for landing

[patch] Patch for landing

bug-185835-20180528230542.patch (text/plain), 5.14 KB, created by Sihui Liu on 2018-05-28 23:05:43 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Sihui Liu

Created: 2018-05-28 23:05:43 PDT

Size: 5.14 KB

patch

obsolete

>Subversion Revision: 232251
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 1c12d7c09b521f2727d699238f934a2ed6112c7f..46fc514c7af61e8cd3e2c40b37ffb59e6cbbe644 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,17 @@
>+2018-05-28  Sihui Liu  <sihui_liu@apple.com>
>+
>+        Unable to remove IndexedDB Databases with Cocoa API removeDataOfTypes
>+        https://bugs.webkit.org/show_bug.cgi?id=185835
>+        <rdar://problem/39142257>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Fix a wrong if condition: databases should be closed and deleted if websiteDataTypes contains 
>+        WebsiteDataType::IndexedDBDatabases.
>+
>+        * StorageProcess/StorageProcess.cpp:
>+        (WebKit::StorageProcess::deleteWebsiteDataForOrigins):
>+
> 2018-05-28  Sam Weinig  <sam@webkit.org>
> 
>         Modernize SVGRenderStyleDefs.h
>diff --git a/Source/WebKit/StorageProcess/StorageProcess.cpp b/Source/WebKit/StorageProcess/StorageProcess.cpp
>index 5782a2ae64ee59db9e1cabce335d04b29a3b4219..421ce76c65b3c1b7168aa2636b628a05db6ba422 100644
>--- a/Source/WebKit/StorageProcess/StorageProcess.cpp
>+++ b/Source/WebKit/StorageProcess/StorageProcess.cpp
>@@ -349,7 +349,7 @@ void StorageProcess::deleteWebsiteDataForOrigins(PAL::SessionID sessionID, Optio
> #endif
> 
> #if ENABLE(INDEXED_DATABASE)
>-    if (!websiteDataTypes.contains(WebsiteDataType::IndexedDBDatabases))
>+    if (websiteDataTypes.contains(WebsiteDataType::IndexedDBDatabases))
>         idbServer(sessionID).closeAndDeleteDatabasesForOrigins(securityOrigins, [callbackAggregator = WTFMove(callbackAggregator)] { });
> #endif
> }
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index ab7261f1a377c362039757370ea4d9f33b2d1423..8f8327be375b1eb5dfef733b5381875a46165e6d 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,16 @@
>+2018-05-28  Sihui Liu  <sihui_liu@apple.com>
>+
>+        Unable to remove IndexedDB Databases with Cocoa API removeDataOfTypes
>+        https://bugs.webkit.org/show_bug.cgi?id=185835
>+        <rdar://problem/39142257>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Add API test coverage.
>+
>+        * TestWebKitAPI/Tests/WebKitCocoa/IndexedDBPersistence.mm:
>+        (TEST):
>+
> 2018-05-28  Thibault Saunier  <tsaunier@igalia.com>
> 
>         [GStreamer] Handle changes in the "drm-preferred-decryption-system-id" NEED_CONTEXT message.
>diff --git a/Tools/TestWebKitAPI/Tests/WebKitCocoa/IndexedDBPersistence.mm b/Tools/TestWebKitAPI/Tests/WebKitCocoa/IndexedDBPersistence.mm
>index da6f924d15c940519d349ddd58449e3e978f8454..afa944611c4f8f5917fdf06bafbc2c4bed290461 100644
>--- a/Tools/TestWebKitAPI/Tests/WebKitCocoa/IndexedDBPersistence.mm
>+++ b/Tools/TestWebKitAPI/Tests/WebKitCocoa/IndexedDBPersistence.mm
>@@ -37,6 +37,7 @@
> 
> #if WK_API_ENABLED
> 
>+static bool readyToContinue;
> static bool receivedScriptMessage;
> static RetainPtr<WKScriptMessage> lastScriptMessage;
> 
>@@ -93,4 +94,47 @@ TEST(IndexedDB, IndexedDBPersistence)
>     EXPECT_WK_STREQ(@"2 TestObjectStore", string3.get());
> }
> 
>+TEST(IndexedDB, IndexedDBDataRemoval)
>+{
>+    auto websiteDataTypes = adoptNS([[NSSet alloc] initWithArray:@[WKWebsiteDataTypeIndexedDBDatabases]]);
>+
>+    readyToContinue = false;
>+    [[WKWebsiteDataStore defaultDataStore] removeDataOfTypes:websiteDataTypes.get() modifiedSince:[NSDate distantPast] completionHandler:^() {
>+        readyToContinue = true;
>+    }];
>+    TestWebKitAPI::Util::run(&readyToContinue);
>+
>+    readyToContinue = false;
>+    [[WKWebsiteDataStore defaultDataStore] fetchDataRecordsOfTypes:websiteDataTypes.get() completionHandler:^(NSArray<WKWebsiteDataRecord *> *dataRecords) {
>+        readyToContinue = true;
>+        ASSERT_EQ(0u, dataRecords.count);
>+    }];
>+    TestWebKitAPI::Util::run(&readyToContinue);
>+
>+    receivedScriptMessage = false;
>+    auto handler = adoptNS([[IndexedDBMessageHandler alloc] init]);
>+    auto configuration = adoptNS([[WKWebViewConfiguration alloc] init]);
>+    [[configuration userContentController] addScriptMessageHandler:handler.get() name:@"testHandler"];
>+    auto webView = adoptNS([[WKWebView alloc] initWithFrame:NSMakeRect(0, 0, 800, 600) configuration:configuration.get()]);
>+    NSURLRequest *request = [NSURLRequest requestWithURL:[[NSBundle mainBundle] URLForResource:@"IndexedDBPersistence-1" withExtension:@"html" subdirectory:@"TestWebKitAPI.resources"]];
>+    [webView loadRequest:request];
>+    TestWebKitAPI::Util::run(&receivedScriptMessage);
>+
>+    readyToContinue = false;
>+    [[WKWebsiteDataStore defaultDataStore] fetchDataRecordsOfTypes:websiteDataTypes.get() completionHandler:^(NSArray<WKWebsiteDataRecord *> *dataRecords) {
>+        ASSERT_EQ(1u, dataRecords.count);
>+        [[WKWebsiteDataStore defaultDataStore] removeDataOfTypes:websiteDataTypes.get() forDataRecords:dataRecords completionHandler:^() {
>+            readyToContinue = true;
>+        }];
>+    }];
>+    TestWebKitAPI::Util::run(&readyToContinue);
>+
>+    readyToContinue = false;
>+    [[WKWebsiteDataStore defaultDataStore] fetchDataRecordsOfTypes:websiteDataTypes.get() completionHandler:^(NSArray<WKWebsiteDataRecord *> *dataRecords) {
>+        readyToContinue = true;
>+        ASSERT_EQ(0u, dataRecords.count);
>+    }];
>+    TestWebKitAPI::Util::run(&readyToContinue);
>+}
>+
> #endif

Actions: View | Formatted Diff | Diff

Attachments on bug 185835: 340883 | 341030 | 341474 | 341475