Bug 45196

Summary: [Chromium] Make sure that the width of the menu list is always constant under layout tests.
Product: WebKit Reporter: Dimitri Glazkov (Google) <dglazkov>
Component: New BugsAssignee: Dimitri Glazkov (Google) <dglazkov>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, dpranke, eric, tony, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch
none
Now with correct test expectations. none

Dimitri Glazkov (Google)
Reported 2010-09-03 13:25:42 PDT
[Chromium] Make sure that the width of the menu list is always constant under layout tests.
Attachments
Patch (12.72 KB, patch)
2010-09-03 13:28 PDT, Dimitri Glazkov (Google) 		no flags
DetailsFormatted DiffDiff
Now with correct test expectations. (8.13 KB, patch)
2010-09-03 13:35 PDT, Dimitri Glazkov (Google) 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Dimitri Glazkov (Google)
Comment 1 2010-09-03 13:28:13 PDT
Created attachment 66537 [details] Patch
Dimitri Glazkov (Google)
Comment 2 2010-09-03 13:35:28 PDT
Created attachment 66538 [details] Now with correct test expectations.
Tony Chang
Comment 3 2010-09-03 13:37:53 PDT
Comment on attachment 66538 [details] Now with correct test expectations. Why doesn't test_shell have this problem?
Dimitri Glazkov (Google)
Comment 4 2010-09-03 13:41:20 PDT
It should, if you muck with themes -- this is not a DRT-only issue. I just happened to have a Win 7 theme different from that on the test box and noticed this.
Dimitri Glazkov (Google)
Comment 5 2010-09-03 13:43:10 PDT
Comment on attachment 66538 [details] Now with correct test expectations. Clearing flags on attachment: 66538 Committed r66768: <http://trac.webkit.org/changeset/66768>
Dimitri Glazkov (Google)
Comment 6 2010-09-03 13:43:15 PDT
All reviewed patches have been landed. Closing bug.
WebKit Review Bot
Comment 7 2010-09-04 00:18:44 PDT
http://trac.webkit.org/changeset/66768 might have broken Leopard Intel Debug (Tests) The following changes are on the blame list: http://trac.webkit.org/changeset/66768 http://trac.webkit.org/changeset/66767
Note You need to log in before you can comment on or make changes to this bug.