Bug 206639
| Summary: | [GTK][WPE] http/tests/contentextensions/block-cookies-basic.html is flaky | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Lauro Moura <lmoura> |
| Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | bugs-noreply, dpino |
| Priority: | P2 | ||
| Version: | Other | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Lauro Moura
Started flakiness between r249708 and r249813
Sample diff:
@@ -1,4 +1,4 @@
This image resource sets a cookie. But based on the content rules, setting cookie should be blocked.
-document.cookie should be empty, and its value is ''
+document.cookie should be empty, and its value is 'cookieSetAsFirstParty=1'
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Lauro Moura
The same happens to /http/tests/contentextensions/block-cookies-send.html starking flakiness in the same range.
--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/http/tests/contentextensions/block-cookies-send-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/http/tests/contentextensions/block-cookies-send-actual.txt
@@ -1,6 +1,6 @@
This test loads a resource that sets a cookie, then verifies that the cookie is not sent out when it should be blocked.
-After the image, document.cookie is 'foo=awesomevalue'
-After the iframe, document.cookie is 'foo=awesomevalue'
+After the image, document.cookie is 'cookieSetAsFirstParty=1; foo=awesomevalue'
+After the iframe, document.cookie is 'cookieSetAsFirstParty=1; foo=awesomevalue'
Lauro Moura
The same is happening to WPE.
Both tests started flakiness in r249622.
[r249607-r249616] NOERROR
[r249620-r249621] NOERROR
r249622 TEXT (Expected: PASS)
r249624 NOERROR
Diego Pino
This test is no longer flaky:
https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2Fcontentextensions%2Fblock-cookies-basic.html&platform=WPE&platform=GTK
Test expectations were updated in 287857@main. There are no more test failures file under this bug. Closing bug.