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

Unified Diff: third_party/WebKit/Source/core/html/HTMLMediaElement.cpp

Issue 2890633003: [Blink,RemotePlayback] Use PresentationController for availability (Closed)
Patch Set: Limited to Blink only, rebased Created 3 years, 7 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/core/html/HTMLMediaElement.cpp
diff --git a/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp b/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
index b0d3bcc22673b62afac88b67ecbe2808fb572900..42399fdf37d5b19a3efcdc68f0e8caed6214ab35 100644
--- a/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
+++ b/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
@@ -1132,6 +1132,10 @@ void HTMLMediaElement::LoadResource(const WebMediaPlayerSource& source,
if (audio_source_node_)
audio_source_node_->OnCurrentSrcChanged(current_src_);
+ if (RuntimeEnabledFeatures::newRemotePlaybackPipelineEnabled() &&
+ RemotePlaybackClient()) {
+ RemotePlaybackClient()->SourceChanged(current_src_);
mark a. foltz 2017/05/31 21:16:41 Is the goal to track the currently playing source
+ }
BLINK_MEDIA_LOG << "loadResource(" << (void*)this << ") - current_src_ -> "
<< UrlForLoggingMedia(current_src_);
@@ -3146,8 +3150,10 @@ void HTMLMediaElement::RequestSeek(double time) {
void HTMLMediaElement::RemoteRouteAvailabilityChanged(
WebRemotePlaybackAvailability availability) {
- if (RemotePlaybackClient())
+ if (RemotePlaybackClient() &&
+ !RuntimeEnabledFeatures::newRemotePlaybackPipelineEnabled()) {
RemotePlaybackClient()->AvailabilityChanged(availability);
+ }
}
bool HTMLMediaElement::HasRemoteRoutes() const {

Powered by Google App Engine
This is Rietveld 408576698