Unreviewed, rolling out r227964.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 2 Feb 2018 13:54:14 +0000 (13:54 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 2 Feb 2018 13:54:14 +0000 (13:54 +0000)
https://bugs.webkit.org/show_bug.cgi?id=182423

It broke several unit tests (Requested by KaL on #webkit).

Reverted changeset:

"[SOUP] Ensure domain is valid when converting a WebCore
Cookie to Soup"
https://bugs.webkit.org/show_bug.cgi?id=182328
https://trac.webkit.org/changeset/227964

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@228006 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/network/soup/CookieSoup.cpp

index 8038922..ef905fe 100644 (file)
@@ -1,3 +1,17 @@
+2018-02-02  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r227964.
+        https://bugs.webkit.org/show_bug.cgi?id=182423
+
+        It broke several unit tests (Requested by KaL on #webkit).
+
+        Reverted changeset:
+
+        "[SOUP] Ensure domain is valid when converting a WebCore
+        Cookie to Soup"
+        https://bugs.webkit.org/show_bug.cgi?id=182328
+        https://trac.webkit.org/changeset/227964
+
 2018-02-01  Chris Dumez  <cdumez@apple.com>
 
         When SW install fails, null out registration.installing before setting worker state to "redundant"
index 4cc5565..eec169b 100644 (file)
@@ -60,13 +60,8 @@ SoupCookie* Cookie::toSoupCookie() const
     if (name.isNull() || value.isNull() || domain.isNull() || path.isNull())
         return nullptr;
 
-    // soup_cookie_new() will handle the given domain as a hostname if it doesn't start with '.'.
-    auto cookieDomain = domain.utf8();
-    if (cookieDomain.length() && !g_hostname_is_ip_address(cookieDomain.data()) && cookieDomain.data()[0] != '.')
-        cookieDomain = makeString('.', domain).utf8();
-
     SoupCookie* soupCookie = soup_cookie_new(name.utf8().data(), value.utf8().data(),
-        cookieDomain.data(), path.utf8().data(), -1);
+        domain.utf8().data(), path.utf8().data(), -1);
 
     soup_cookie_set_http_only(soupCookie, httpOnly);
     soup_cookie_set_secure(soupCookie, secure);