Bug 194854 - [GTK] Gardening for unexpected passes
Summary: [GTK] Gardening for unexpected passes
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-02-20 05:22 PST by Diego Pino
Modified: 2019-02-21 04:30 PST (History)
3 users (show)

See Also:


Attachments
Patch (8.06 KB, patch)
2019-02-20 05:22 PST, Diego Pino
no flags Details | Formatted Diff | Diff
Patch (8.06 KB, patch)
2019-02-20 05:48 PST, Diego Pino
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Pino 2019-02-20 05:22:09 PST
[GTK] Gardening for unexpected passes
Comment 1 Diego Pino 2019-02-20 05:22:29 PST
Created attachment 362493 [details]
Patch
Comment 2 Diego Pino 2019-02-20 05:23:15 PST
Summary:

Expected to crash, but passed: (1)

webgl/1.0.3/conformance/extensions/webgl-draw-buffers.html:
	The test is passing since r241120 ([WPE] Implement GStreamer based holepunch).
	The test only passes when using --display-server=xvfb. It's still failing for --display-server=xorg.

Expected to timeout, but passed: (2)

http/tests/contentextensions/block-cookies-in-image-load-in-onunload.html
http/tests/contentextensions/block-image-load-in-onunload.html:
	The tests are passing since r241283 ([GTK][WPE] Add content extensions support in WKTR and unskip layout tests  https://bugs.webkit.org/show_bug.cgi?id=193622).

Expected to fail, but passed: (23)

http/tests/cache/disk-cache/redirect-chain-limits.html:
	Fixed somewhere between [r241656-r241659]. Likely fixed due to r241656 (Use dumpJSConsoleLogInStdErr=true webkit-test-runner option for non-imported tests instead of using DumpJSConsoleLogInStdErr expectation in TestExpectations https://bugs.webkit.org/show_bug.cgi?id=194586)
http/tests/contentextensions/block-cookies-in-csp-report.php
http/tests/contentextensions/block-cookies-in-ping.html
http/tests/contentextensions/block-cookies-send.html
http/tests/contentextensions/block-csp-report.php
http/tests/contentextensions/block-everything-unless-domain-redirect.php
http/tests/contentextensions/block-ping.html
http/tests/contentextensions/fetch-redirect-blocked.html
http/tests/contentextensions/hide-on-csp-report.php
http/tests/contentextensions/hide-on-ping-with-ping-that-redirects.html
http/tests/contentextensions/hide-on-ping.html
http/tests/contentextensions/main-resource-redirect-blocked.php
http/tests/contentextensions/main-resource-redirect-error.html
http/tests/contentextensions/reload-without-contentextensions.html:
	The tests are passing since r241283 ([GTK][WPE] Add content extensions support in WKTR and unskip layout tests https://bugs.webkit.org/show_bug.cgi?id=193622
imported/w3c/web-platform-tests/media-source/mediasource-addsourcebuffer-mode.html:
	The test is passing since r241585 (Decoding media-capabilities configuration initial support https://bugs.webkit.org/show_bug.cgi?id=191191).
webgl/1.0.2/conformance/context/context-attribute-preserve-drawing-buffer.html:
	The test is passing since r241117 ([JSC] Use BufferInternal single character StringImpl for SmallStrings https://bugs.webkit.org/show_bug.cgi?id=194374)
http/wpt/offscreen-canvas/transferToImageBitmap-webgl.html:
imported/blink/compositing/draws-content/webgl-simple-background.html:
fast/canvas/webgl/match-page-color-space.html:
webgl/1.0.2/conformance/canvas/buffer-offscreen-test.html:
webgl/webgl-texture-image-buffer-reuse.html:
webgl/webgl-box-shadow.html:
webgl/webgl-border.html:
	The test is passing since r241120 ([WPE] Implement GStreamer based holepunch https://bugs.webkit.org/show_bug.cgi?id=193715)
Comment 3 Adrian Perez 2019-02-20 05:40:56 PST
Comment on attachment 362493 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=362493&action=review

Patch looks good, just one comment below.

> LayoutTests/ChangeLog:6
> +        Reviewed by NOBODY (OOPS!).

As this is a gardening patch, it does not need review and you change it to:

    Unreviewed test gardening.

Then set “cq?”, and leave “r” empty. Once the EWS bots have passed, you
can then switch to “cq+” and let the commit-queue bot land the patch.

(Alternatively: Use “webkit-patch land-safely”, which automates the above)
Comment 4 Diego Pino 2019-02-20 05:48:35 PST
Created attachment 362494 [details]
Patch
Comment 5 WebKit Commit Bot 2019-02-21 04:29:23 PST
Comment on attachment 362494 [details]
Patch

Clearing flags on attachment: 362494

Committed r241864: <https://trac.webkit.org/changeset/241864>
Comment 6 WebKit Commit Bot 2019-02-21 04:29:24 PST
All reviewed patches have been landed.  Closing bug.
Comment 7 Radar WebKit Bug Importer 2019-02-21 04:30:28 PST
<rdar://problem/48272136>