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

Unified Diff: Source/core/rendering/RenderMediaControls.cpp

Issue 291163004: Implement media cast buttons (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Respond to comments Created 6 years, 3 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
Index: Source/core/rendering/RenderMediaControls.cpp
diff --git a/Source/core/rendering/RenderMediaControls.cpp b/Source/core/rendering/RenderMediaControls.cpp
index 6288aa3969d8087ddf1e4f2898fa61c128ea5b9a..e28c8730cc2fe532b8dd65cd9c8a46ca339fae92 100644
--- a/Source/core/rendering/RenderMediaControls.cpp
+++ b/Source/core/rendering/RenderMediaControls.cpp
@@ -335,7 +335,39 @@ static bool paintMediaToggleClosedCaptionsButton(RenderObject* object, const Pai
return paintMediaButton(paintInfo.context, rect, mediaClosedCaptionButtonDisabled);
}
+static bool paintMediaCastButton(RenderObject* object, const PaintInfo& paintInfo, const IntRect& rect)
+{
+ HTMLMediaElement* mediaElement = toParentMediaElement(object);
+ if (!mediaElement)
+ return false;
+
+ static Image* mediaCastOnButton = platformResource("mediaplayerCastOn");
+ static Image* mediaCastOffButton = platformResource("mediaplayerCastOff");
+
+ if (mediaElement->isPlayingRemotely()) {
+ return paintMediaButton(paintInfo.context, rect, mediaCastOnButton);
+ }
+
+ return paintMediaButton(paintInfo.context, rect, mediaCastOffButton);
+
+}
+static bool paintMediaOverlayCastButton(RenderObject* object, const PaintInfo& paintInfo, const IntRect& rect)
acolwell GONE FROM CHROMIUM 2014/09/10 23:35:23 This looks identical to the method above. Why do w
aberent 2014/09/11 16:38:19 Done.
+{
+ HTMLMediaElement* mediaElement = toParentMediaElement(object);
+ if (!mediaElement)
+ return false;
+
+ static Image* mediaCastOnButton = platformResource("mediaplayerCastOn");
+ static Image* mediaCastOffButton = platformResource("mediaplayerCastOff");
+
+ if (mediaElement->isPlayingRemotely()) {
+ return paintMediaButton(paintInfo.context, rect, mediaCastOnButton);
+ }
+
+ return paintMediaButton(paintInfo.context, rect, mediaCastOffButton);
+
+}
bool RenderMediaControls::paintMediaControlsPart(MediaControlElementType part, RenderObject* object, const PaintInfo& paintInfo, const IntRect& rect)
{
@@ -361,6 +393,12 @@ bool RenderMediaControls::paintMediaControlsPart(MediaControlElementType part, R
return paintMediaFullscreenButton(object, paintInfo, rect);
case MediaOverlayPlayButton:
return paintMediaOverlayPlayButton(object, paintInfo, rect);
+ case MediaCastOffButton:
+ case MediaCastOnButton:
+ return paintMediaCastButton(object, paintInfo, rect);
+ case MediaOverlayCastOffButton:
+ case MediaOverlayCastOnButton:
+ return paintMediaOverlayCastButton(object, paintInfo, rect);
case MediaVolumeSliderContainer:
case MediaTimelineContainer:
case MediaCurrentTimeDisplay:

Powered by Google App Engine
This is Rietveld 408576698