Bug 85210

Summary: Integrate IETC CSS : flexbox tests
Product: WebKit Reporter: Dave Tharp <dtharp>
Component: CSSAssignee: Dave Tharp <dtharp>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, aestes, bdakin, rakuco, tomz, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
URL: http://samples.msdn.microsoft.com/ietestcenter/#css3flexbox
Bug Depends on: 85596    
Bug Blocks: 81526, 85666    
Attachments:
Description Flags
Patch
none
Patch abarth: review+

Dave Tharp
Reported 2012-04-30 11:05:55 PDT
Integrate the 24 IETC flexbox tests. Three of these are failures and will have separate bugs written for them.
Attachments
Patch (129.81 KB, patch)
2012-04-30 14:48 PDT, Dave Tharp 		no flags
DetailsFormatted DiffDiff
Patch (129.91 KB, patch)
2012-05-02 15:30 PDT, Dave Tharp 		abarth: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Dave Tharp
Comment 1 2012-04-30 14:48:27 PDT
Created attachment 139522 [details] Patch
WebKit Review Bot
Comment 2 2012-04-30 14:52:58 PDT
Attachment 139522 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/iete..." exit_code: 1 LayoutTests/platform/gtk/test_expectations.txt:517: More specific entry on line 330 overrides line 517 fast/workers/storage/use-same-database-in-page-and-workers.html [test/expectations] [5] Total errors found: 1 in 54 files If any of these errors are false positives, please file a bug against check-webkit-style.
Dave Tharp
Comment 3 2012-04-30 14:55:44 PDT
Style failure due to existing issue in gtk/test_expectations.txt : 517: More specific entry on line 330 overrides line 517 fast/workers/storage/use-same-database-in-page-and-workers.html. I'm inclined to let the gtk gardener deal with this rather than attempting a fix myself, but am open to suggestion.
Dave Tharp
Comment 4 2012-05-02 09:28:18 PDT
Notes on this patch: - 24 total tests, 3 fail. - bugs have been written for the failures - All tests have correct reference tests - EWS failure due to existing problem in gtk test_expectations.txt, and also due to churn in the chromium test_expectations.txt. I will deal with these issues when I land the patch.
Dave Tharp
Comment 5 2012-05-02 15:30:18 PDT
Created attachment 139893 [details] Patch
Dave Tharp
Comment 6 2012-05-02 15:31:30 PDT
(In reply to comment #5) > Created an attachment (id=139893) [details] > Patch Rebased to include fix (by koz) to gtk test_expectations.txt so we don't faile the style check.
Tom Zakrajsek
Comment 7 2012-05-03 14:48:22 PDT
Committed r116018: <http://trac.webkit.org/changeset/116018>
Andy Estes
Comment 8 2012-05-04 14:08:07 PDT
None of these reference tests pass on the Mac. The test looks visually identical to the reference, but ImageDiff detects a difference between the color drawn by background-image and the color in the reference image. Might this be because the images have an sRGB color profile?
Andy Estes
Comment 9 2012-05-04 14:14:26 PDT
I filed <https://bugs.webkit.org/show_bug.cgi?id=85666> about this.
Dave Tharp
Comment 10 2012-05-07 10:33:36 PDT
I had assumed the green in the pngs used in the MS tests were the same color and background "green". In fact, all my testing on my system (Linux: QT,GTK, and Chromium) went just fine. It's weird that QT on the bots also failed on minor pixel differences (can't repro this locally). Anyway, I've reworked the reference tests to use the pngs where necessary, so the pixel differences should disappear. I have repro'ed the issue on another developer's Mac, and will upload a patch as soon as I can get him to test the new ref tests.
Note You need to log in before you can comment on or make changes to this bug.