WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED CONFIGURATION CHANGED
Bug 64001
REGRESSION(
r90212
-
r90246
?): http/tests/cookies/third-party-cookie-relaxing.html fails on Snow Leopard
https://bugs.webkit.org/show_bug.cgi?id=64001
Summary
REGRESSION(r90212-r90246?): http/tests/cookies/third-party-cookie-relaxing.ht...
Eric Seidel (no email)
Reported
2011-07-06 09:33:43 PDT
http/tests/cookies/third-party-cookie-relaxing.html is failing under new-run-webkit-tests on the snow leopard bot
Attachments
Add attachment
proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1
2011-07-06 09:35:30 PDT
Example failure:
http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(Tests)/r90460%20(31134)/http/tests/cookies/third-party-cookie-relaxing-pretty-diff.html
--- /Volumes/Data/WebKit-BuildSlave/snowleopard-intel-release-tests/build/layout-test-results/http/tests/cookies/third-party-cookie-relaxing-expected.txt +++ /Volumes/Data/WebKit-BuildSlave/snowleopard-intel-release-tests/build/layout-test-results/http/tests/cookies/third-party-cookie-relaxing-actual.txt @@ -26,7 +26,7 @@ ALERT: Test stage 5 document.cookie is: ALERT:
http://localhost:8000/cookies/resources/cookie-utility.php?queryfunction=setFooCookie
ALERT: XHR response - Set the foo cookie -ALERT: Test stage 6 document.cookie is: foo=awesomevalue +ALERT: Test stage 6 document.cookie is: ALERT: Restricting to first party only cookies ALERT:
http://localhost:8000/cookies/resources/cookie-utility.php?queryfunction=deleteCookies
ALERT: XHR response - Deleted all cookies @@ -39,9 +39,9 @@ ALERT: Test stage 8 document.cookie is: ALERT:
http://localhost:8000/cookies/resources/cookie-utility.php?queryfunction=setFooCookie
ALERT: XHR response - Set the foo cookie -ALERT: Test stage 9 document.cookie is: foo=awesomevalue +ALERT: Test stage 9 document.cookie is: ALERT: Restricting to first party only cookies ALERT:
http://localhost:8000/cookies/resources/cookie-utility.php?queryfunction=setFooAndBarCookie
ALERT: XHR response - Set the foo and bar cookies -ALERT: Test stage 10 document.cookie is: bar=anotherawesomevalue; foo=awesomevalue +ALERT: Test stage 10 document.cookie is:
Eric Seidel (no email)
Comment 2
2011-07-06 09:37:19 PDT
new-run-webkit-tests http/tests/cookies/third-party-cookie-relaxing.html fails when run straight on my machine. Then again, so does: old-run-webkit-tests http/tests/cookies/third-party-cookie-relaxing.html
Eric Seidel (no email)
Comment 3
2011-07-06 09:39:59 PDT
It also fails for me with: old-run-webkit-tests http/tests/cookies and old-run-webkit-tests http I'm trying to understand how the test is supposed to work?
http://trac.webkit.org/browser/trunk/LayoutTests/http/tests/cookies/third-party-cookie-relaxing.html
Maybe there was a recent regression in behavior on Snow Leopard which happen to correspond with the NRWT transition? Maybe NRWT running cookie-tests in parallel manages to permanently "jam" the CF cookie store? Donno.
Eric Seidel (no email)
Comment 4
2011-07-06 09:42:44 PDT
Qt just added support for third-party cookies, and their results seem to match SL, in at least the 6 and 9 cases:
http://trac.webkit.org/browser/trunk/LayoutTests/platform/qt/http/tests/cookies/third-party-cookie-relaxing-expected.txt
Eric Seidel (no email)
Comment 5
2011-07-06 09:45:34 PDT
Bug 63545
is another cookie-related failure which occurs on my machine. This one of course occurs on the SL bot as well:
http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(Tests)/r90460%20(31134)/results.html
Eric Seidel (no email)
Comment 6
2011-07-06 10:14:20 PDT
Even if I move my ~/Library/Cookies/Cookies.plist file out of the way, the test still fails for me.
Eric Seidel (no email)
Comment 7
2011-07-06 10:14:43 PDT
Turns out this has nothing to do with NRWT, now that I look at the bot history (with webkit-patch failure-reason).
Eric Seidel (no email)
Comment 8
2011-07-06 10:19:51 PDT
I don't see any changes in that range which seem particularly related. Perhaps failure-reason is confused.
Eric Seidel (no email)
Comment 9
2011-07-06 10:34:08 PDT
The only possibly-related changes in that range are to the JIT. But I don't see how they could affect this test:
http://trac.webkit.org/changeset/90237
http://trac.webkit.org/changeset/90232
I checked the results. Unless build.webkit.org/results is lying, it's definitely this range. I've not tried using git-bisect, but I think the build was broken a bunch in there, so it may not answer our question.
Adam Roben (:aroben)
Comment 10
2011-07-06 12:46:51 PDT
TestFailures says this test is flaky:
r90315
(31042) (results.html): fail
r90310
(31041) (results.html): pass
r90308
(31040) (results.html): pass
r90288
(31027) (results.html): fail
r90285
(31026) (results.html): pass
r90284
(31025) (results.html): fail
r90280
(31024) (results.html): pass
r90279
(31023) (results.html): fail
r90273
(31022) (results.html): pass
r90271
(31019) (results.html): fail It always fails with the same diff:
http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(Tests)/r90315%20(31042)/http/tests/cookies/third-party-cookie-relaxing-pretty-diff.html
Alexey Proskuryakov
Comment 11
2020-06-25 17:56:23 PDT
This is no longer happening. Karl removed the expectation via
bug 213620
.
Radar WebKit Bug Importer
Comment 12
2020-06-25 17:57:17 PDT
<
rdar://problem/64783252
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug