Title: [136636] trunk/Source/WebCore
Revision
136636
Author
commit-qu...@webkit.org
Date
2012-12-04 21:18:55 -0800 (Tue, 04 Dec 2012)

Log Message

Web Inspector: Resources: domain cookies are not shown for sub-sub domain pages.
https://bugs.webkit.org/show_bug.cgi?id=104016

Patch by Eugene Klyuchnikov <eus...@chromium.org> on 2012-12-04
Reviewed by Pavel Feldman.

Fixed regexp to allow zero or more subdomain prefix (was zero or one).

* inspector/front-end/CookieParser.js:
(WebInspector.Cookies.cookieDomainMatchesResourceDomain): Fixed regexp.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (136635 => 136636)


--- trunk/Source/WebCore/ChangeLog	2012-12-05 05:15:11 UTC (rev 136635)
+++ trunk/Source/WebCore/ChangeLog	2012-12-05 05:18:55 UTC (rev 136636)
@@ -1,3 +1,15 @@
+2012-12-04  Eugene Klyuchnikov  <eus...@chromium.org>
+
+        Web Inspector: Resources: domain cookies are not shown for sub-sub domain pages.
+        https://bugs.webkit.org/show_bug.cgi?id=104016
+
+        Reviewed by Pavel Feldman.
+
+        Fixed regexp to allow zero or more subdomain prefix (was zero or one).
+
+        * inspector/front-end/CookieParser.js:
+        (WebInspector.Cookies.cookieDomainMatchesResourceDomain): Fixed regexp.
+
 2012-12-04  Alpha Lam  <hc...@chromium.org>
 
         Not reviewed. Build fix.

Modified: trunk/Source/WebCore/inspector/front-end/CookieParser.js (136635 => 136636)


--- trunk/Source/WebCore/inspector/front-end/CookieParser.js	2012-12-05 05:15:11 UTC (rev 136635)
+++ trunk/Source/WebCore/inspector/front-end/CookieParser.js	2012-12-05 05:18:55 UTC (rev 136636)
@@ -422,5 +422,5 @@
 {
     if (cookieDomain.charAt(0) !== '.')
         return resourceDomain === cookieDomain;
-    return !!resourceDomain.match(new RegExp("^([^\\.]+\\.)?" + cookieDomain.substring(1).escapeForRegExp() + "$", "i"));
+    return !!resourceDomain.match(new RegExp("^([^\\.]+\\.)*" + cookieDomain.substring(1).escapeForRegExp() + "$", "i"));
 }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to