Bug 58149

Summary: Some elements in meter-styles.html are hiding its content part with non-content-area.
Product: WebKit Reporter: Hajime Morrita <morrita>
Component: Layout and RenderingAssignee: Hajime Morrita <morrita>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, tkent
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 50661    
Attachments:
Description Flags
Patch tkent: review+

Hajime Morrita
Reported 2011-04-08 11:55:21 PDT
Too large-sized border and padding hide the content area, which we need to ensure its visual. http://trac.webkit.org/changeset/82899 changed the layout scheme of <meter>, that made this test less useful. We need to update the test.
Attachments
Patch (183.93 KB, patch)
2011-04-08 12:01 PDT, Hajime Morrita 		tkent: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Hajime Morrita
Comment 1 2011-04-08 12:01:51 PDT
Created attachment 88853 [details] Patch
Dimitri Glazkov (Google)
Comment 3 2011-04-22 14:04:21 PDT
(In reply to comment #2) > What about the second <meter> in meter-element.html? Its background bar is not rendered. > http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=meter-element.html%2Cmeter-styles.html&showExpectations=true&showLargeExpectations=true&group=%40ToT%20-%20chromium.org Morrita-san, have you answered this question? Would love to close this bug.
Hajime Morrita
Comment 4 2011-04-23 13:10:19 PDT
I'm sorry I missed Kent-san's comment. And this actually broken.... I filed Bug 59281 and will fix shortly.
Kent Tamura
Comment 5 2011-04-26 16:46:22 PDT
Comment on attachment 88853 [details] Patch ok
Hajime Morrita
Comment 6 2011-05-11 22:10:09 PDT
Committed r86307: <http://trac.webkit.org/changeset/86307>
Note You need to log in before you can comment on or make changes to this bug.