Remove MediaControls methods needed for the Cast button

third_party/WebKit/Source/modules/remoteplayback/RemotePlaybackTest.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/ExceptionState.h"
 #include "bindings/core/v8/V8BindingForTesting.h"
 #include "bindings/modules/v8/RemotePlaybackAvailabilityCallback.h"
 #include "core/dom/DocumentUserGestureToken.h"
@@ skipping 38 matching lines @@
 
 class RemotePlaybackTest : public ::testing::Test {
  protected:
   void CancelPrompt(RemotePlayback* remote_playback) {
     remote_playback->PromptCancelled();
   }
 
   void SetState(RemotePlayback* remote_playback, WebRemotePlaybackState state) {
     remote_playback->StateChanged(state);
   }
+
+  bool HasAvailabilityCallbacks(RemotePlayback* remote_playback) {
+    return !remote_playback->availability_callbacks_.IsEmpty();
+  }
 };
 
 TEST_F(RemotePlaybackTest, PromptCancelledRejectsWithNotAllowedError) {
   V8TestingScope scope;
 
   auto page_holder = DummyPageHolder::Create();
 
   HTMLMediaElement* element =
       HTMLVideoElement::Create(page_holder->GetDocument());
   RemotePlayback* remote_playback =
@@ skipping 197 matching lines @@
   // Runs pending promises.
   v8::MicrotasksScope::PerformCheckpoint(scope.GetIsolate());
 
   // Verify mock expectations explicitly as the mock objects are garbage
   // collected.
   ::testing::Mock::VerifyAndClear(resolve);
   ::testing::Mock::VerifyAndClear(reject);
   ::testing::Mock::VerifyAndClear(callback_function);
 }
 
+TEST_F(RemotePlaybackTest, RemovingFromDocumentRemovesListenersAndCallbacks) {

[mlamouri (slow - plz ping), 2017/04/25 12:19:21]

This should be a media_controls/ test, shouldn't it?

[whywhat, 2017/04/25 15:45:37]

I guess so :)

+  V8TestingScope scope;
+
+  auto page_holder = DummyPageHolder::Create();
+
+  HTMLMediaElement* element =
+      HTMLVideoElement::Create(page_holder->GetDocument());
+  element->SetBooleanAttribute(HTMLNames::controlsAttr, true);
+  page_holder->GetDocument().body()->AppendChild(element);
+
+  RemotePlayback* remote_playback =
+      HTMLMediaElementRemotePlayback::remote(*element);
+
+  EXPECT_TRUE(remote_playback->HasEventListeners());
+  EXPECT_TRUE(HasAvailabilityCallbacks(remote_playback));
+
+  WeakPersistent<HTMLMediaElement> weak_persistent_video = element;
+  {
+    Persistent<HTMLMediaElement> persistent_video = element;
+    page_holder->GetDocument().body()->setInnerHTML("");
+
+    // When removed from the document, the event listeners should have been
+    // dropped.
+    EXPECT_FALSE(remote_playback->HasEventListeners());
+    EXPECT_FALSE(HasAvailabilityCallbacks(remote_playback));
+  }
+
+  testing::RunPendingTasks();
+
+  ThreadState::Current()->CollectAllGarbage();
+
+  // It has been GC'd.
+  EXPECT_EQ(nullptr, weak_persistent_video);
+}
+
+TEST_F(RemotePlaybackTest, ReInsertingInDocumentRestoresListenersAndCallbacks) {

[mlamouri (slow - plz ping), 2017/04/25 12:19:21]

ditto

+  V8TestingScope scope;
+
+  auto page_holder = DummyPageHolder::Create();
+
+  HTMLMediaElement* element =
+      HTMLVideoElement::Create(page_holder->GetDocument());
+  element->SetBooleanAttribute(HTMLNames::controlsAttr, true);
+  page_holder->GetDocument().body()->AppendChild(element);
+
+  RemotePlayback* remote_playback =
+      HTMLMediaElementRemotePlayback::remote(*element);
+
+  // This should be a no-op. We keep a reference on the media element to avoid
+  // an unexpected GC.
+  {
+    Persistent<HTMLMediaElement> video_holder = element;
+    page_holder->GetDocument().body()->RemoveChild(element);
+    page_holder->GetDocument().body()->AppendChild(video_holder.Get());
+    EXPECT_TRUE(remote_playback->HasEventListeners());
+    EXPECT_TRUE(HasAvailabilityCallbacks(remote_playback));
+  }
+}
+
 }  // namespace blink