Bug 183129

Summary:

Rewrite frameset-flattening-simple to make expectation clearer

Product:

WebKit

Reporter:

Frédéric Wang (:fredw) <fred.wang>

Component:

Tools / Tests

Assignee:

Frédéric Wang (:fredw) <fred.wang>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

don.olmstead, ews-watchlist, jfernandez, lforschler, rniwa, simon.fraser, tonikitoo, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

See Also:

https://bugs.webkit.org/show_bug.cgi?id=71655

Attachments:

Description	Flags
Patch	none
Archive of layout-test-results from ews101 for mac-sierra	none
Patch	none
Archive of layout-test-results from ews200 for win-future	none
Patch	jfernandez: review+
Patch for landing	none

Description Frédéric Wang (:fredw) 2018-02-26 07:43:04 PST

frameset-flattening-simple test description says that the iframe should be 800x wide and should not have any scrollbars, however it is a bit difficult to know the size when checking the PNG reference and moreover the scrollbars of the main frame is visible, which is a bit confusing.

Comment 1 Frédéric Wang (:fredw) 2018-02-26 07:48:42 PST

Created attachment 334615 [details]
Patch

Comment 2 EWS Watchlist 2018-02-26 08:49:18 PST

Comment on attachment 334615 [details]
Patch

Attachment 334615 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/6675944

New failing tests:
fast/frames/flattening/frameset-flattening-simple.html

Comment 3 EWS Watchlist 2018-02-26 08:49:19 PST

Created attachment 334618 [details]
Archive of layout-test-results from ews101 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101  Port: mac-sierra  Platform: Mac OS X 10.12.6

Comment 4 Frédéric Wang (:fredw) 2018-02-26 09:15:46 PST

Created attachment 334621 [details]
Patch

Comment 5 EWS Watchlist 2018-02-26 10:58:00 PST

Comment on attachment 334621 [details]
Patch

Attachment 334621 [details] did not pass win-ews (win):
Output: http://webkit-queues.webkit.org/results/6677510

New failing tests:
fast/frames/flattening/frameset-flattening-simple.html

Comment 6 EWS Watchlist 2018-02-26 10:58:11 PST

Created attachment 334628 [details]
Archive of layout-test-results from ews200 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews200  Port: win-future  Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit

Comment 7 Frédéric Wang (:fredw) 2018-02-26 11:07:37 PST

Created attachment 334630 [details]
Patch

Comment 8 Frédéric Wang (:fredw) 2018-02-27 06:40:17 PST

Created attachment 334689 [details]
Patch for landing

Comment 9 Frédéric Wang (:fredw) 2018-02-27 06:50:31 PST

@Don: I plan to land this later but FYI the patch fails to apply on WinCairo due to changes in the PNG expectation. I think this already happened in the past for another bug.

Comment 10 Don Olmstead 2018-02-27 10:55:16 PST

(In reply to Frédéric Wang (:fredw) from comment #9)
> @Don: I plan to land this later but FYI the patch fails to apply on WinCairo
> due to changes in the PNG expectation. I think this already happened in the
> past for another bug.

Yea I have a few outstanding bugs on our EWS to take care of and will also update git.

Comment 11 Frédéric Wang (:fredw) 2018-03-13 06:44:37 PDT

Committed r229576: <https://trac.webkit.org/changeset/229576>

Comment 12 Radar WebKit Bug Importer 2018-03-13 06:46:20 PDT

<rdar://problem/38415260>