Bug 94687 - [EFL] Update pixel test expectations for layout test fast/forms/
Summary: [EFL] Update pixel test expectations for layout test fast/forms/
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: KwangYong Choi
URL:
Keywords:
Depends on:
Blocks: 94188
  Show dependency treegraph
 
Reported: 2012-08-22 03:28 PDT by KwangYong Choi
Modified: 2012-08-23 12:26 PDT (History)
4 users (show)

See Also:


Attachments
Patch 1 (1.61 MB, patch)
2012-08-22 04:09 PDT, KwangYong Choi
no flags Details | Formatted Diff | Diff
Patch 2 (1.61 MB, patch)
2012-08-22 04:10 PDT, KwangYong Choi
no flags Details | Formatted Diff | Diff
Patch 3 (712.02 KB, patch)
2012-08-22 04:11 PDT, KwangYong Choi
no flags Details | Formatted Diff | Diff
Patch (deleted)
2012-08-22 19:20 PDT, KwangYong Choi
no flags Details | Formatted Diff | Diff
Patch (deleted)
2012-08-23 05:35 PDT, KwangYong Choi
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description KwangYong Choi 2012-08-22 03:28:27 PDT
[EFL] Update pixel test expectations for layout test fast/forms/
Comment 1 KwangYong Choi 2012-08-22 04:09:31 PDT
Created attachment 159899 [details]
Patch 1

Uploaded pixel expectations are little bit different from original ones.

Bugzilla has upload limit, so I splited patch to 3 pieces. Is it possible to upload like this?
Comment 2 KwangYong Choi 2012-08-22 04:10:23 PDT
Created attachment 159900 [details]
Patch 2
Comment 3 KwangYong Choi 2012-08-22 04:11:00 PDT
Created attachment 159901 [details]
Patch 3
Comment 4 KwangYong Choi 2012-08-22 19:20:54 PDT
Created attachment 160068 [details]
Patch

with big file option.
Comment 5 Raphael Kubo da Costa (:rakuco) 2012-08-22 19:33:36 PDT
fast/forms/listbox-hit-test-zoomed-expected.png looks wrong; it should be skipped and a bug report should be created.

Ditto for menulist-restrict-line-height-expected.png, fast/forms/search-cancel-button-style-sharing-expected.png and fast/forms/search-rtl-expected.png.

fast/forms/menulist-narrow-width-expected.png, fast/forms/select-baseline-expected.png and fast/forms/menulist-style-color-expected.png all seem to be suffering from the same regression. Perhaps fast/forms/select-style-expected.png is having the same bug as well.

The scrollbar isn't being fully rendered in fast/forms/textarea-scroll-height-expected.png and fast/forms/textarea-scrolled-type-expected.png (I guess that's bug 85484).
Comment 6 KwangYong Choi 2012-08-23 00:07:51 PDT
(In reply to comment #5)
> fast/forms/listbox-hit-test-zoomed-expected.png looks wrong; it should be skipped and a bug report should be created.
> 
You're right. Test 2 is failing. I will make this test skip.

> Ditto for menulist-restrict-line-height-expected.png, fast/forms/search-cancel-button-style-sharing-expected.png and fast/forms/search-rtl-expected.png.
> 
For fast/forms/menulist-restrict-line-height.html, text in the popup button is clipped. Need skip.

For fast/forms/search-cancel-button-style-sharing.html and fast/forms/search-rtl.html, cancel mark is not displayed on search box. Need skip.

> fast/forms/menulist-narrow-width-expected.png, fast/forms/select-baseline-expected.png and fast/forms/menulist-style-color-expected.png all seem to be suffering from the same regression. Perhaps fast/forms/select-style-expected.png is having the same bug as well.
> 
These tests have styling problem. Need skip.

> The scrollbar isn't being fully rendered in fast/forms/textarea-scroll-height-expected.png and fast/forms/textarea-scrolled-type-expected.png (I guess that's bug 85484).

These tests have scroll bar displaying problem. Need skip.
Comment 7 KwangYong Choi 2012-08-23 05:35:49 PDT
Created attachment 160136 [details]
Patch

The wrong test expectations will be removed by bug 94798.
Comment 8 Gyuyoung Kim 2012-08-23 05:48:17 PDT
Anyone double checking ?
Comment 9 Gyuyoung Kim 2012-08-23 06:33:09 PDT
Rebased .png flies pass --pixel test in my linux box.
Comment 10 Raphael Kubo da Costa (:rakuco) 2012-08-23 12:07:53 PDT
Comment on attachment 160136 [details]
Patch

Looks OK now, cq+'ing.
Comment 11 WebKit Review Bot 2012-08-23 12:26:09 PDT
Comment on attachment 160136 [details]
Patch

Clearing flags on attachment: 160136

Committed r126466: <http://trac.webkit.org/changeset/126466>
Comment 12 WebKit Review Bot 2012-08-23 12:26:13 PDT
All reviewed patches have been landed.  Closing bug.