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

Unified Diff: Source/core/layout/LayoutMediaControls.cpp

Issue 1082533002: Support text track selection in video controls (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Rebase 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
Index: Source/core/layout/LayoutMediaControls.cpp
diff --git a/Source/core/layout/LayoutMediaControls.cpp b/Source/core/layout/LayoutMediaControls.cpp
index f969a22290ffcd206d4f22ceed5935cbcd1a464a..c65576e328015aca1cad7d6f25111aee3bf79471 100644
--- a/Source/core/layout/LayoutMediaControls.cpp
+++ b/Source/core/layout/LayoutMediaControls.cpp
@@ -375,6 +375,16 @@ static bool paintMediaCastButton(LayoutObject* object, const PaintInfo& paintInf
}
}
+static bool paintMediaTrackSelectionCheckmark(LayoutObject* object, const PaintInfo& paintInfo, const IntRect& rect)
+{
+ HTMLMediaElement* mediaElement = toParentMediaElement(object);
+ if (!mediaElement)
+ return false;
+
+ static Image* mediaTrackSelectionCheckmark = platformResource("mediaplayerTrackSelectionCheckmark");
+ return paintMediaButton(paintInfo.context, rect, mediaTrackSelectionCheckmark);
+}
+
bool LayoutMediaControls::paintMediaControlsPart(MediaControlElementType part, LayoutObject* object, const PaintInfo& paintInfo, const IntRect& rect)
{
switch (part) {
@@ -404,12 +414,15 @@ bool LayoutMediaControls::paintMediaControlsPart(MediaControlElementType part, L
case MediaOverlayCastOffButton:
case MediaOverlayCastOnButton:
return paintMediaCastButton(object, paintInfo, rect);
+ case MediaTrackSelectionCheckmark:
+ return paintMediaTrackSelectionCheckmark(object, paintInfo, rect);
case MediaVolumeSliderContainer:
case MediaTimelineContainer:
case MediaCurrentTimeDisplay:
case MediaTimeRemainingDisplay:
case MediaControlsPanel:
case MediaHideClosedCaptionsButton:
+ case MediaTextTrackList:
case MediaFullScreenVolumeSlider:
case MediaFullScreenVolumeSliderThumb:
ASSERT_NOT_REACHED();

Powered by Google App Engine
This is Rietveld 408576698