Attachment 339240 Details for Bug 185041 – Patch

[patch] Patch

bug-185041-20180501162931.patch (text/plain), 3.53 KB, created by Sihui Liu on 2018-05-01 16:29:32 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Sihui Liu

Created: 2018-05-01 16:29:32 PDT

Size: 3.53 KB

patch

obsolete

>Subversion Revision: 231178
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index cba905e6179e4a78f79a07354fd7a9e518740329..594eb27f4274b3640be60440d49ecf93dbde7726 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,14 @@
>+2018-04-30  Sihui Liu  <sihui_liu@apple.com>
>+
>+        [WKHTTPCookieStore getAllCookies] returns inconsistent creation time
>+        https://bugs.webkit.org/show_bug.cgi?id=185041
>+        <rdar://problem/34684214>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * TestWebKitAPI/Tests/WebKitCocoa/WKHTTPCookieStore.mm:
>+        (TEST):
>+
> 2018-04-30  Mark Lam  <mark.lam@apple.com>
> 
>         Apply PtrTags to the MetaAllocator and friends.
>diff --git a/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKHTTPCookieStore.mm b/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKHTTPCookieStore.mm
>index 9d5d7ecd6dc2a4719cd7d49fcc0a254ecd619798..6e7537f64fa826f3ceeba1b8619013987bbb0759 100644
>--- a/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKHTTPCookieStore.mm
>+++ b/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKHTTPCookieStore.mm
>@@ -33,6 +33,7 @@
> #import <WebKit/WKWebsiteDataStorePrivate.h>
> #import <WebKit/_WKWebsiteDataStoreConfiguration.h>
> #import <wtf/RetainPtr.h>
>+#import <wtf/Seconds.h>
> 
> #if WK_API_ENABLED
> 
>@@ -310,6 +311,59 @@ TEST(WebKit, WKHTTPCookieStoreHttpOnly)
>     [cookies release];
> }
> 
>+TEST(WebKit, WKHTTPCookieStoreCreationTime) 
>+{   
>+    WKWebsiteDataStore* dataStore = [WKWebsiteDataStore defaultDataStore];
>+
>+    auto configuration = adoptNS([[WKWebViewConfiguration alloc] init]);
>+    configuration.get().websiteDataStore = dataStore;
>+    auto webView = adoptNS([[WKWebView alloc] initWithFrame:NSMakeRect(0, 0, 800, 600) configuration:configuration.get()]);
>+
>+    [webView loadHTMLString:@"WebKit Test" baseURL:[NSURL URLWithString:@"http://webkit.org"]];
>+    [webView _test_waitForDidFinishNavigation];
>+
>+    [dataStore removeDataOfTypes:[WKWebsiteDataStore allWebsiteDataTypes] modifiedSince:[NSDate distantPast] completionHandler:[] {
>+        gotFlag = true;
>+    }];
>+    TestWebKitAPI::Util::run(&gotFlag);
>+    gotFlag = false;
>+
>+    globalCookieStore = dataStore.httpCookieStore;
>+
>+    RetainPtr<NSHTTPCookie> cookie = [NSHTTPCookie cookieWithProperties:@{
>+        NSHTTPCookiePath: @"/path",
>+        NSHTTPCookieName: @"CookieName",
>+        NSHTTPCookieValue: @"CookieValue",
>+        NSHTTPCookieDomain: @".www.webkit.org",
>+    }];
>+
>+    [globalCookieStore setCookie:cookie.get() completionHandler:[]() {
>+        gotFlag = true;
>+    }];
>+    TestWebKitAPI::Util::run(&gotFlag);
>+    gotFlag = false;
>+
>+    RetainPtr<NSNumber> creationTime = nil;
>+    [globalCookieStore getAllCookies:[creationTimePtr = &creationTime](NSArray<NSHTTPCookie *> *cookies) {
>+        ASSERT_EQ(1u, cookies.count);
>+        *creationTimePtr = [cookies objectAtIndex:0].properties[@"Created"];
>+        gotFlag = true;
>+    }];
>+    TestWebKitAPI::Util::run(&gotFlag);
>+    gotFlag = false;
>+
>+    sleep(1_s);
>+
>+    [globalCookieStore getAllCookies:^(NSArray<NSHTTPCookie *> *cookies) {
>+        ASSERT_EQ(1u, cookies.count);
>+        RetainPtr<NSNumber> creationTime2 = [cookies objectAtIndex:0].properties[@"Created"];
>+        EXPECT_TRUE([creationTime.get() isEqual:creationTime2.get()]);
>+        gotFlag = true;
>+    }];
>+    TestWebKitAPI::Util::run(&gotFlag);
>+    gotFlag = false;
>+}
>+
> // FIXME: This should be removed once <rdar://problem/35344202> is resolved and bots are updated.
> #if (PLATFORM(MAC) && __MAC_OS_X_VERSION_MAX_ALLOWED <= 101301) || (PLATFORM(IOS) && __IPHONE_OS_VERSION_MAX_ALLOWED <= 110102)
> TEST(WebKit, WKHTTPCookieStoreNonPersistent)

Actions: View | Formatted Diff | Diff

Attachments on bug 185041: 339171 | 339212 | 339240 | 339803 | 341302