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

Unified Diff: third_party/WebKit/Source/modules/media_controls/MediaControlsImpl.h

Issue 2782373002: Remove MediaControls methods needed for the Cast button (Closed)
Patch Set: Unified watch availability code. Created 3 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: third_party/WebKit/Source/modules/media_controls/MediaControlsImpl.h
diff --git a/third_party/WebKit/Source/modules/media_controls/MediaControlsImpl.h b/third_party/WebKit/Source/modules/media_controls/MediaControlsImpl.h
index d761f237a4a66c798583b11414f53dcf78d029ce..e4b4d912310689e1c6945b71b80a027bd66610c4 100644
--- a/third_party/WebKit/Source/modules/media_controls/MediaControlsImpl.h
+++ b/third_party/WebKit/Source/modules/media_controls/MediaControlsImpl.h
@@ -83,13 +83,6 @@ class MODULES_EXPORT MediaControlsImpl final : public HTMLDivElement,
// HTMLTrackElement failed to load because there is no web exposed way to
// be notified on the TextTrack object. See https://crbug.com/669977
void OnTrackElementFailedToLoad() override { OnTextTracksAddedOrRemoved(); }
- // TODO(mlamouri): the following methods will be able to become private when
- // the controls have to modules/ and have access to RemotePlayback.
- void OnRemotePlaybackAvailabilityChanged() override {
- RefreshCastButtonVisibility();
- }
- void OnRemotePlaybackConnecting() override { StartedCasting(); }
- void OnRemotePlaybackDisconnected() override { StoppedCasting(); }
// TODO(mlamouri): this method is needed in order to notify the controls that
// the attribute have changed.
void OnDisableRemotePlaybackAttributeChanged() override {
@@ -196,8 +189,6 @@ class MODULES_EXPORT MediaControlsImpl final : public HTMLDivElement,
bool ContainsRelatedTarget(Event*);
// Internal cast related methods.
- void StartedCasting();
- void StoppedCasting();
void RefreshCastButtonVisibility();
void RefreshCastButtonVisibilityWithoutUpdate();

Powered by Google App Engine
This is Rietveld 408576698