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

Unified Diff: content/browser/media/session/media_session_controller_unittest.cc

Issue 1698933004: Make MediaSession a runtime-enabled feature on Desktop. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 9 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: content/browser/media/session/media_session_controller_unittest.cc
diff --git a/content/browser/media/android/media_session_controller_unittest.cc b/content/browser/media/session/media_session_controller_unittest.cc
similarity index 94%
rename from content/browser/media/android/media_session_controller_unittest.cc
rename to content/browser/media/session/media_session_controller_unittest.cc
index be56ccb31fb0b3bd72bde5daa8887a33da0e2e2e..7ed5a4a9d75fe0a2f54e038674cb3e8a22424781 100644
--- a/content/browser/media/android/media_session_controller_unittest.cc
+++ b/content/browser/media/session/media_session_controller_unittest.cc
@@ -3,9 +3,9 @@
// found in the LICENSE file.
#include "base/tuple.h"
-#include "content/browser/media/android/media_session.h"
-#include "content/browser/media/android/media_session_controller.h"
-#include "content/browser/media/android/media_web_contents_observer_android.h"
+#include "content/browser/media/media_web_contents_observer.h"
+#include "content/browser/media/session/media_session.h"
+#include "content/browser/media/session/media_session_controller.h"
#include "content/common/media/media_player_delegate_messages.h"
#include "content/test/test_render_view_host.h"
#include "content/test/test_web_contents.h"
@@ -18,7 +18,6 @@ class MediaSessionControllerTest : public RenderViewHostImplTestHarness {
void SetUp() override {
RenderViewHostImplTestHarness::SetUp();
id_ = WebContentsObserver::MediaPlayerId(contents()->GetMainFrame(), 0);
- observer_ = MediaWebContentsObserverAndroid::FromWebContents(contents());
controller_ = CreateController();
}
@@ -31,8 +30,8 @@ class MediaSessionControllerTest : public RenderViewHostImplTestHarness {
protected:
scoped_ptr<MediaSessionController> CreateController() {
- return scoped_ptr<MediaSessionController>(
- new MediaSessionController(id_, observer_));
+ return scoped_ptr<MediaSessionController>(new MediaSessionController(
+ id_, contents()->media_web_contents_observer()));
}
MediaSession* media_session() { return MediaSession::Get(contents()); }
@@ -99,7 +98,6 @@ class MediaSessionControllerTest : public RenderViewHostImplTestHarness {
}
WebContentsObserver::MediaPlayerId id_;
- MediaWebContentsObserverAndroid* observer_;
scoped_ptr<MediaSessionController> controller_;
};

Powered by Google App Engine
This is Rietveld 408576698