[webkit-reviews] review requested: [Bug 29161] Implement default media UI for mac chromium. : [Attachment 39401] Reimplement default media UI for Mac Chromium to match the style

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Sep 10 17:49:27 PDT 2009


Albert J. Wong <ajwong at chromium.org> has asked	for review:
Bug 29161: Implement default media UI for mac chromium.
https://bugs.webkit.org/show_bug.cgi?id=29161

Attachment 39401: Reimplement default media UI for Mac Chromium to match the
style
https://bugs.webkit.org/attachment.cgi?id=39401&action=review

------- Additional Comments from Albert J. Wong <ajwong at chromium.org>
of the Windows and Linux versions.  Also breaks the dependency
on the internal wk* functions that were previously used to
render the media controller widgets.
https://bugs.webkit.org/show_bug.cgi?id=29161

Patch by Albert J. Wong <ajwong at chromium.org> on 2009-09-10
Reviewed by NOBODY (OOPS!).

No media layout tests are currently enabled in Mac Chromium, so
nothing needs rebaselineing, etc.

* css/mediaControlsChromium.css:
* rendering/RenderThemeChromiumMac.h:
* rendering/RenderThemeChromiumMac.mm:
(WebCore::mediaElementParent):
(WebCore::RenderThemeChromiumMac::extraMediaControlsStyleSheet):
(WebCore::mediaSliderThumbImage):
(WebCore::mediaVolumeSliderThumbImage):
(WebCore::RenderThemeChromiumMac::paintSliderTrack):
(WebCore::RenderThemeChromiumMac::adjustSliderThumbSize):
(WebCore::RenderThemeChromiumMac::paintMediaButtonInternal):
(WebCore::RenderThemeChromiumMac::paintMediaPlayButton):
(WebCore::RenderThemeChromiumMac::paintMediaMuteButton):
(WebCore::RenderThemeChromiumMac::paintMediaSliderTrack):
(WebCore::RenderThemeChromiumMac::paintMediaVolumeSliderTrack):
(WebCore::RenderThemeChromiumMac::paintMediaSliderThumb):
(WebCore::RenderThemeChromiumMac::paintMediaVolumeSliderThumb):
(WebCore::RenderThemeChromiumMac::paintMediaControlsBackground):
* rendering/RenderThemeChromiumSkia.cpp:
(WebCore::RenderThemeChromiumSkia::adjustSliderThumbSize):
---
 5 files changed, 252 insertions(+), 98 deletions(-)


More information about the webkit-reviews mailing list