Bug 124879 - [EFL] Layout tests need to be rebaselined.
Summary: [EFL] Layout tests need to be rebaselined.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-11-25 18:43 PST by Jongwoo Choi
Modified: 2013-11-28 07:24 PST (History)
6 users (show)

See Also:


Attachments
Patch (12.09 KB, patch)
2013-11-25 18:48 PST, Jongwoo Choi
no flags Details | Formatted Diff | Diff
Patch (5.41 KB, patch)
2013-11-26 16:14 PST, Jongwoo Choi
no flags Details | Formatted Diff | Diff
Patch (12.09 KB, patch)
2013-11-27 17:18 PST, Jongwoo 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 Jongwoo Choi 2013-11-25 18:43:04 PST
fast/table/011.html
fast/table/border-collapsing/004.html
fast/table/tableInsideCaption.html

Above tests need to be rebaselined results after r159747.
Comment 1 Jongwoo Choi 2013-11-25 18:48:13 PST
Created attachment 217856 [details]
Patch
Comment 2 Grzegorz Czajkowski 2013-11-26 00:21:13 PST
Comment on attachment 217856 [details]
Patch

Those tests definitely need to be rebaselined. Could you take care of png files as well? Those have been added at r140159 and it seems to be out of date. In addition, r159747 rebaselines more tests (at least for mac), don't we need to update them? Thanks
Comment 3 Michal Pakula vel Rutka 2013-11-26 10:17:08 PST
(In reply to comment #2)
> (From update of attachment 217856 [details])
> Those tests definitely need to be rebaselined. Could you take care of png files as well? Those have been added at r140159 and it seems to be out of date. In addition, r159747 rebaselines more tests (at least for mac), don't we need to update them? Thanks

The rests of tests were updated in r159753 and r159755.

The only test here which needs pixel expectation update is fast/table/border-collapsing/004.html, but the difference is not caused by r159747.
The difference is lack of scrollbar in current output and it is started to happen after r156472, precisely this change: http://trac.webkit.org/changeset/156472/trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp @2547. This affects not only this test but around 300-400 other ones causing identical image failures (lack of scrollbars in pixel expectations). 
I think we should leave old pixel expectation even if it is incorrect, in this and other tests and fix the bug with scrollbars.
Comment 4 Jongwoo Choi 2013-11-26 16:07:40 PST
(In reply to comment #3)
> (In reply to comment #2)
> > (From update of attachment 217856 [details] [details])
> > Those tests definitely need to be rebaselined. Could you take care of png files as well? Those have been added at r140159 and it seems to be out of date. In addition, r159747 rebaselines more tests (at least for mac), don't we need to update them? Thanks
> 
> The rests of tests were updated in r159753 and r159755.
> 
> The only test here which needs pixel expectation update is fast/table/border-collapsing/004.html, but the difference is not caused by r159747.
> The difference is lack of scrollbar in current output and it is started to happen after r156472, precisely this change: http://trac.webkit.org/changeset/156472/trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp @2547. This affects not only this test but around 300-400 other ones causing identical image failures (lack of scrollbars in pixel expectations). 
> I think we should leave old pixel expectation even if it is incorrect, in this and other tests and fix the bug with scrollbars.

Thank you for your comment.
I exclude two test expectations(fast/table/011.html and fast/table/tableInsideCaption.html) in this patch.
Comment 5 Jongwoo Choi 2013-11-26 16:14:24 PST
Created attachment 217910 [details]
Patch
Comment 6 Jongwoo Choi 2013-11-27 17:15:39 PST
According to Michal Pakula vel Rutka's messages, my first patch was ok.
So, I reupload my first patch.
Comment 7 Jongwoo Choi 2013-11-27 17:18:05 PST
Created attachment 217971 [details]
Patch
Comment 8 WebKit Commit Bot 2013-11-27 17:57:43 PST
Comment on attachment 217971 [details]
Patch

Rejecting attachment 217971 [details] from commit-queue.

jw0330.choi@samsung.com does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/contributors.json.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/contributors.json by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.
Comment 9 WebKit Commit Bot 2013-11-28 07:24:30 PST
Comment on attachment 217971 [details]
Patch

Clearing flags on attachment: 217971

Committed r159841: <http://trac.webkit.org/changeset/159841>
Comment 10 WebKit Commit Bot 2013-11-28 07:24:33 PST
All reviewed patches have been landed.  Closing bug.