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

Unified Diff: third_party/WebKit/Source/modules/mediasession/MediaSessionTest.cpp

Issue 1441883003: Hook up RendererMediaSessionManager with browser side (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address review comment from philipj: ActiveDOMObject Created 5 years 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/mediasession/MediaSessionTest.cpp
diff --git a/third_party/WebKit/Source/modules/mediasession/MediaSessionTest.cpp b/third_party/WebKit/Source/modules/mediasession/MediaSessionTest.cpp
index 8927e8aef7189185439b2fefdbab6889c8cc9000..bba8ca4a00ab92a8056cc3ef4e44f661c06e9a47 100644
--- a/third_party/WebKit/Source/modules/mediasession/MediaSessionTest.cpp
+++ b/third_party/WebKit/Source/modules/mediasession/MediaSessionTest.cpp
@@ -26,7 +26,7 @@ protected:
{
// The MediaSession takes ownership of the WebMediaSession, and the
// caller must take care to not end up with a stale pointer.
- return new MediaSession(adoptPtr(webMediaSession));
+ return new MediaSession(&document(), adoptPtr(webMediaSession));
}
Document& document() { return m_page->document(); }
@@ -41,6 +41,7 @@ class MockWebMediaSession : public WebMediaSession {
public:
MOCK_METHOD1(activate, void(WebMediaSessionActivateCallback*));
MOCK_METHOD1(deactivate, void(WebMediaSessionDeactivateCallback*));
+ bool hasPendingCallbacks() override { return false; }
};
class Helper {

Powered by Google App Engine
This is Rietveld 408576698