87208 2012-05-22 21:42:13 -0700 [Qt][Mac] http/tests/cookies/js-get-and-set-http-only-cookie.html failing since introduction in r119947 2014-02-03 03:21:00 -0800 1 1 1 Unclassified WebKit Page Loading 528+ (Nightly build) Mac Unspecified RESOLVED INVALID InRadar, MakingBotsRed P2 Normal --- 88760 86067 87008 1 slewis webkit-unassigned ap beidson cdumez jasonliuwebkit jberlin jochen kadam mrobinson ossy pnormand rakuco rwlbuis slewis webkit-bug-importer zan oldest_to_newest 631183 0 slewis 2012-05-22 21:42:13 -0700 http://trac.webkit.org/changeset/118105 added a new test: http/tests/cookies/js-get-and-set-http-only-cookie.php which is failing on Mac WK1 and WK2. http://build.webkit.org/results/Lion%20Release%20(Tests)/r118111%20(8569)/results.html --- /Volumes/Data/slave/lion-intel-release-tests/build/layout-test-results/http/tests/cookies/js-get-and-set-http-only-cookie-expected.txt +++ /Volumes/Data/slave/lion-intel-release-tests/build/layout-test-results/http/tests/cookies/js-get-and-set-http-only-cookie-actual.txt @@ -5,7 +5,7 @@ Check that we can't get or set httpOnly Cookies by JavaScript. PASS We can't get httpOnly cookies by JavaScript. -PASS We can't set httpOnly cookies by JavaScript. +FAIL We shouldn't set httpOnly cookies by JavaScript. PASS successfullyParsed is true TEST COMPLETE 631184 1 webkit-bug-importer 2012-05-22 21:42:28 -0700 <rdar://problem/11512237> 631217 2 slewis 2012-05-22 22:20:48 -0700 Checked in failing results http://trac.webkit.org/projects/webkit/changeset/118122 631285 3 ossy 2012-05-22 23:47:01 -0700 Same fail on GTK and Qt too. 631298 4 ossy 2012-05-22 23:59:11 -0700 I skipped it on Qt - r118132. 631304 5 zan 2012-05-23 00:05:47 -0700 It seems this test is also causing various failures (even crashes in GTK builders) in http/tests/cookies/multiple-cookies.html, which is the test that is run after this test. http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20-%20webkit.org&tests=http%2Ftests%2Fcookies%2Fmultiple-cookies.html I'll skip the test for Gtk. 632311 6 slewis 2012-05-23 19:09:06 -0700 This test can have side effects. Skipped in http://trac.webkit.org/changeset/118305 632353 7 jasonliuwebkit 2012-05-23 20:12:26 -0700 (In reply to comment #5) > It seems this test is also causing various failures (even crashes in GTK builders) in http/tests/cookies/multiple-cookies.html, which is the test that is run after this test. > > http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20-%20webkit.org&tests=http%2Ftests%2Fcookies%2Fmultiple-cookies.html > > I'll skip the test for Gtk. Do you know why multiple-cookies.html failed? I don't think my test case can cause this failing. 633959 8 zan 2012-05-25 08:35:00 -0700 (In reply to comment #7) > (In reply to comment #5) > > It seems this test is also causing various failures (even crashes in GTK builders) in http/tests/cookies/multiple-cookies.html, which is the test that is run after this test. > > > > http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20-%20webkit.org&tests=http%2Ftests%2Fcookies%2Fmultiple-cookies.html > > > > I'll skip the test for Gtk. > > Do you know why multiple-cookies.html failed? > > I don't think my test case can cause this failing. The stderr output of the crash was: 'Creating pipes for GWakeup: Too many open files' This perhaps signals a platform-specific problem, i.e. a bug in either the libsoup HTTP backend or libsoup itself. 641452 9 jasonliuwebkit 2012-06-05 05:28:16 -0700 The failure on mac is because of an actual bug in cookie jar. 641455 10 jochen 2012-06-05 05:31:58 -0700 To be more precise in WebCore/platform/mac/CookieJar.mm, it is not checked that a new cookie wouldn't override an existing httpOnly cookie. 645780 11 cdumez 2012-06-11 00:22:10 -0700 I will work on a fix for the SOUP-specific code in Bug 88760. This looks like a valid problem as we allow setting/overwriting httpOnly cookies from JavaScript. 645839 12 ossy 2012-06-11 02:02:14 -0700 Skipped on Qt by http://trac.webkit.org/changeset/119964/trunk/LayoutTests/platform/qt/Skipped Please unskip it with the proper fix. 709752 13 jberlin 2012-08-31 10:17:19 -0700 This test was removed in http://trac.webkit.org/changeset/118166 and reintroduced as http/tests/cookies/js-get-and-set-http-only-cookie.html, which does not appear to be suffering from the same problems. I am going to fix up the Skipped lists. 709755 14 jberlin 2012-08-31 10:19:58 -0700 Actually, it looks like the Qt Skipped list entry was for the .html version introduced in http://trac.webkit.org/changeset/119947. Updating the title to reflect that. 759989 15 ap 2012-11-06 13:21:28 -0800 Actually, this still fails on Mac, <rdar://problem/12427027>. 833088 16 kadam 2013-02-14 08:15:14 -0800 I checked this test but it still fails on Qt. 975463 17 jturcotte 2014-02-03 03:21:00 -0800 === Bulk closing of Qt bugs === If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary. If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.