Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1418)

Unified Diff: Source/core/paint/ThemePainter.cpp

Issue 1115203003: De-virtualize media control painting methods (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Mac doesn't want cast button Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/paint/ThemePainter.h ('k') | Source/core/paint/ThemePainterDefault.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/paint/ThemePainter.cpp
diff --git a/Source/core/paint/ThemePainter.cpp b/Source/core/paint/ThemePainter.cpp
index 9b6ea695280c9d5a07eb279c06833d83364ffa9b..ee7194d39fca4c6c451f9e567bc8b988c3f8fb41 100644
--- a/Source/core/paint/ThemePainter.cpp
+++ b/Source/core/paint/ThemePainter.cpp
@@ -32,6 +32,7 @@
#include "core/layout/LayoutMeter.h"
#include "core/layout/LayoutTheme.h"
#include "core/layout/LayoutView.h"
+#include "core/paint/MediaControlsPainter.h"
#include "core/paint/PaintInfo.h"
#include "core/style/ComputedStyle.h"
#include "platform/graphics/GraphicsContextStateSaver.h"
@@ -113,37 +114,32 @@ bool ThemePainter::paint(LayoutObject* o, const PaintInfo& paintInfo, const IntR
return paintSliderThumb(o, paintInfo, r);
case MediaEnterFullscreenButtonPart:
case MediaExitFullscreenButtonPart:
- return paintMediaFullscreenButton(o, paintInfo, r);
+ return MediaControlsPainter::paintMediaFullscreenButton(o, paintInfo, r);
case MediaPlayButtonPart:
- return paintMediaPlayButton(o, paintInfo, r);
+ return MediaControlsPainter::paintMediaPlayButton(o, paintInfo, r);
case MediaOverlayPlayButtonPart:
- return paintMediaOverlayPlayButton(o, paintInfo, r);
+ return MediaControlsPainter::paintMediaOverlayPlayButton(o, paintInfo, r);
case MediaMuteButtonPart:
- return paintMediaMuteButton(o, paintInfo, r);
+ return MediaControlsPainter::paintMediaMuteButton(o, paintInfo, r);
case MediaToggleClosedCaptionsButtonPart:
- return paintMediaToggleClosedCaptionsButton(o, paintInfo, r);
+ return MediaControlsPainter::paintMediaToggleClosedCaptionsButton(o, paintInfo, r);
case MediaSliderPart:
- return paintMediaSliderTrack(o, paintInfo, r);
+ return MediaControlsPainter::paintMediaSlider(o, paintInfo, r);
case MediaSliderThumbPart:
- return paintMediaSliderThumb(o, paintInfo, r);
+ return MediaControlsPainter::paintMediaSliderThumb(o, paintInfo, r);
case MediaVolumeSliderContainerPart:
- return paintMediaVolumeSliderContainer(o, paintInfo, r);
+ return true;
case MediaVolumeSliderPart:
- return paintMediaVolumeSliderTrack(o, paintInfo, r);
+ return MediaControlsPainter::paintMediaVolumeSlider(o, paintInfo, r);
case MediaVolumeSliderThumbPart:
- return paintMediaVolumeSliderThumb(o, paintInfo, r);
+ return MediaControlsPainter::paintMediaVolumeSliderThumb(o, paintInfo, r);
case MediaFullScreenVolumeSliderPart:
- return paintMediaFullScreenVolumeSliderTrack(o, paintInfo, r);
case MediaFullScreenVolumeSliderThumbPart:
- return paintMediaFullScreenVolumeSliderThumb(o, paintInfo, r);
case MediaTimeRemainingPart:
- return paintMediaTimeRemaining(o, paintInfo, r);
case MediaCurrentTimePart:
- return paintMediaCurrentTime(o, paintInfo, r);
case MediaControlsBackgroundPart:
- return paintMediaControlsBackground(o, paintInfo, r);
+ return true;
case MediaCastOffButtonPart:
- return paintMediaCastButton(o, paintInfo, r);
case MediaOverlayCastOffButtonPart:
return paintMediaCastButton(o, paintInfo, r);
case MenulistButtonPart:
« no previous file with comments | « Source/core/paint/ThemePainter.h ('k') | Source/core/paint/ThemePainterDefault.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698