[RemotePlayback] Stop remote playback when disableRemotePlayback is set

third_party/WebKit/Source/modules/remoteplayback/RemotePlayback.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "modules/remoteplayback/RemotePlayback.h"
 
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "bindings/modules/v8/RemotePlaybackAvailabilityCallback.h"
 #include "core/HTMLNames.h"
 #include "core/dom/DOMException.h"
@@ skipping 249 matching lines @@
 
 void RemotePlayback::remotePlaybackDisabled() {
   if (m_promptPromiseResolver) {
     m_promptPromiseResolver->reject(DOMException::create(
         InvalidStateError, "disableRemotePlayback attribute is present."));
     m_promptPromiseResolver = nullptr;
   }
 
   m_availabilityCallbacks.clear();
 
   // TODO(avayvod): stop remote playback too. https://crbug.com/657566.

[hubbe, 2016/11/02 21:17:45]

remove todo?

[whywhat, 2016/11/04 15:01:06]

Done.

+  if (m_state != WebRemotePlaybackState::Disconnected)
+    m_mediaElement->requestRemotePlaybackStop();

[mlamouri (slow - plz ping), 2016/11/03 16:12:40]

What would happen if the state was `connecting` at this point?

[whywhat, 2016/11/04 15:01:06]

Replied.

 }
 
 void RemotePlayback::setV8ReferencesForCallbacks(
     v8::Isolate* isolate,
     const v8::Persistent<v8::Object>& wrapper) {
   for (auto callback : m_availabilityCallbacks.values())
     callback->setWrapperReference(isolate, wrapper);
 }
 
 DEFINE_TRACE(RemotePlayback) {
   visitor->trace(m_availabilityCallbacks);
   visitor->trace(m_promptPromiseResolver);
   visitor->trace(m_mediaElement);
   EventTargetWithInlineData::trace(visitor);
 }
 
 DEFINE_TRACE_WRAPPERS(RemotePlayback) {
   for (auto callback : m_availabilityCallbacks.values()) {
     visitor->traceWrappers(callback);
   }
   EventTargetWithInlineData::traceWrappers(visitor);
 }
 
 }  // namespace blink