Bug 58147

Summary: [Chromium] Two new tests added by r83287 fail in chromium
Product: WebKit Reporter: Jian Li <jianli>
Component: WebCore Misc.Assignee: Leandro Graciá Gil <leandrogracia>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, steveblock
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Bug Depends on: 56586    
Bug Blocks: 56459    
Attachments:
Description Flags
Patch none

Attachments
Patch (1.68 KB, patch)
2011-04-12 05:33 PDT, Leandro Graciá Gil 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Leandro Graciá Gil
Comment 1 2011-04-12 05:33:49 PDT
Created attachment 89189 [details] Patch
Leandro Graciá Gil
Comment 2 2011-04-12 06:21:12 PDT
Planning to land this after this Chromium patch: http://codereview.chromium.org/6825072/
Steve Block
Comment 3 2011-04-12 06:22:34 PDT
Comment on attachment 89189 [details] Patch Can you link to the patch that added failing expectations for these tests?
Leandro Graciá Gil
Comment 4 2011-04-14 07:36:50 PDT
(In reply to comment #3) > (From update of attachment 89189 [details]) > Can you link to the patch that added failing expectations for these tests? The same one that added the tests: https://bugs.webkit.org/show_bug.cgi?id=56586 The problem is not in the patch or the tests themselves, but in an overrode feature in the chromium code. It should be fixed by now.
WebKit Commit Bot
Comment 5 2011-04-14 18:29:21 PDT
Comment on attachment 89189 [details] Patch Clearing flags on attachment: 89189 Committed r83925: <http://trac.webkit.org/changeset/83925>
WebKit Commit Bot
Comment 6 2011-04-14 18:29:25 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.