Bug 93006

Summary: [Chromium] Layout Tests tables/mozilla/bugs/bug27038*.html are failing
Product: WebKit Reporter: Alexander Pavlov (apavlov) <apavlov>
Component: Layout and RenderingAssignee: Julien Chaffraix <jchaffraix>
Status: RESOLVED FIXED    
Severity: Normal CC: webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch for landing none

Alexander Pavlov (apavlov)
Reported 2012-08-02 10:25:25 PDT
Bug 37244 patch fixed the tests tables/mozilla/bugs/bug27038-1.html tables/mozilla/bugs/bug27038-2.html but they were broken once again some time ago, which went unnoticed due to the fact that their IMAGE+TEXT expectations had never been removed from the TestExpectations file.
Attachments
Patch for landing (112.18 KB, patch)
2012-09-08 08:02 PDT, Julien Chaffraix
no flags
Julien Chaffraix
Comment 1 2012-09-08 07:58:08 PDT
The history of those 2 lines is a bit more complex. bug 37244 did the proper rebaselining. However part of it was rolled out as part of bug 81826 and the approach taken for this bug was to disable the tests on all platforms until I come around and rebaseline them. Unfortunately I wrongly copied the TestExpectations entry from other ports that hadn't rebaselined them thus the Chromium entries are pointing at bug 37244 and not bug 81826. The new baselines are expected so I will just rebaseline the tests.
Julien Chaffraix
Comment 2 2012-09-08 08:02:37 PDT
Created attachment 162958 [details] Patch for landing
WebKit Review Bot
Comment 3 2012-09-08 08:24:25 PDT
Comment on attachment 162958 [details] Patch for landing Clearing flags on attachment: 162958 Committed r127971: <http://trac.webkit.org/changeset/127971>
WebKit Review Bot
Comment 4 2012-09-08 08:24:28 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.