[Webkit-unassigned] [Bug 40280] <meter> should allow styling for each gauge-level and component

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jun 14 19:59:08 PDT 2010


Kent Tamura <tkent at chromium.org> changed:

           What    |Removed                     |Added
  Attachment #58632|review?                     |review-
               Flag|                            |

--- Comment #4 from Kent Tamura <tkent at chromium.org>  2010-06-14 19:59:07 PST ---
(From update of attachment 58632)
 +          - Disabled pixel tests for HTMLMeterElement is temporality disabled.
This sentence looks curious.  Maybe either of "disabled" is not needed?
typo: temporality -> temporarily

 +          window.setTimeout(function(evt) {
What's the intention of this setTimeout()?

 +  fast/dom/HTMLMeterElement/meter-element.html
Because Qt has ENABLE_METER_TAG, we don't need to skip the test.  we can:
 - Commit this change
 - Qt buildbot runs
 - Pick the Qt test result
 - Commit it

 +                  A7AA66D611C5ED6A001D8C8C /* RenderIndicator.cpp in Sources */,
Should be sorted.

 +      return IntRect(borderLeft() + paddingLeft(), borderTop() + paddingTop(), lround(width() - borderLeft() - paddingLeft() - borderRight() - paddingRight()), height()  - borderTop() - paddingTop() - borderBottom() - paddingBottom());
We should have a test for this.  e.g.
meter {
  border-left-width: 100px;
  border-right-width: 1px;
and the meter value is just a half.

 +  bool RenderTheme::canPaintControlsFor(RenderObject* renderObject) const
Can we merge this into RenderTheme::isControlStyled()?

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list