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

Unified Diff: content/child/blink_platform_impl.cc

Issue 1079323002: Support text track selection in video controls (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase and lgtm nits Created 4 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 | « content/app/strings/content_strings.grd ('k') | third_party/WebKit/LayoutTests/media/media-controls.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/child/blink_platform_impl.cc
diff --git a/content/child/blink_platform_impl.cc b/content/child/blink_platform_impl.cc
index fbd0731fa71294f78308d11eede5a148a471ae75..3d432ccfe73c02bbbb5e3fc6e7efd13ce5ea8c7a 100644
--- a/content/child/blink_platform_impl.cc
+++ b/content/child/blink_platform_impl.cc
@@ -337,6 +337,10 @@ static int ToMessageID(WebLocalizedString::Name name) {
return IDS_FORM_INPUT_WEEK_TEMPLATE;
case WebLocalizedString::WeekNumberLabel:
return IDS_FORM_WEEK_NUMBER_LABEL;
+ case WebLocalizedString::TextTracksNoLabel:
+ return IDS_MEDIA_TRACKS_NO_LABEL;
+ case WebLocalizedString::TextTracksOff:
+ return IDS_MEDIA_TRACKS_OFF;
// This "default:" line exists to avoid compile warnings about enum
// coverage when we add a new symbol to WebLocalizedString.h in WebKit.
// After a planned WebKit patch is landed, we need to add a case statement
@@ -673,6 +677,24 @@ const DataResource kDataResources[] = {
{"mediaplayerOverlayPlayNew",
IDR_MEDIAPLAYER_OVERLAY_PLAY_BUTTON_NEW,
ui::SCALE_FACTOR_100P},
+ {"mediaplayerTrackSelectionCheckmark",
+ IDR_MEDIAPLAYER_TRACKSELECTION_CHECKMARK,
+ ui::SCALE_FACTOR_100P},
+ {"mediaplayerTrackSelectionCheckmarkNew",
+ IDR_MEDIAPLAYER_TRACKSELECTION_CHECKMARK_NEW,
+ ui::SCALE_FACTOR_100P},
+ {"mediaplayerClosedCaptionsIcon",
+ IDR_MEDIAPLAYER_CLOSEDCAPTIONS_ICON,
+ ui::SCALE_FACTOR_100P},
+ {"mediaplayerClosedCaptionsIconNew",
+ IDR_MEDIAPLAYER_CLOSEDCAPTIONS_ICON_NEW,
+ ui::SCALE_FACTOR_100P},
+ {"mediaplayerSubtitlesIcon",
+ IDR_MEDIAPLAYER_SUBTITLES_ICON,
+ ui::SCALE_FACTOR_100P},
+ {"mediaplayerSubtitlesIconNew",
+ IDR_MEDIAPLAYER_SUBTITLES_ICON_NEW,
+ ui::SCALE_FACTOR_100P},
{"searchCancel", IDR_SEARCH_CANCEL, ui::SCALE_FACTOR_100P},
{"searchCancelPressed", IDR_SEARCH_CANCEL_PRESSED, ui::SCALE_FACTOR_100P},
{"searchMagnifier", IDR_SEARCH_MAGNIFIER, ui::SCALE_FACTOR_100P},
« no previous file with comments | « content/app/strings/content_strings.grd ('k') | third_party/WebKit/LayoutTests/media/media-controls.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698