Introduce WebMediaSession

third_party/WebKit/Source/modules/mediasession/MediaSessionTest.cpp

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
new file mode 100644
index 0000000000000000000000000000000000000000..16a4809ed29d658bdf61b5cda9735c7ce4cefd30
--- /dev/null
+++ b/third_party/WebKit/Source/modules/mediasession/MediaSessionTest.cpp
@@ -0,0 +1,70 @@
+// Copyright 2015 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 "config.h"
+#include "modules/mediasession/MediaSession.h"
+
+#include "core/dom/Document.h"
+#include "core/testing/DummyPageHolder.h"
+#include "public/platform/modules/mediasession/WebMediaSession.h"
+#include <gmock/gmock.h>
+#include <gtest/gtest.h>
+
+using testing::_;
+
+namespace blink {
+namespace {
+
+class MockWebMediaSession : public WebMediaSession {
+public:
+    MOCK_METHOD1(activate, void(WebMediaSessionCommandCallback*));
+    MOCK_METHOD1(deactivate, void(WebMediaSessionCommandCallback*));
+};
+
+class MediaSessionTest : public ::testing::Test {
+protected:
+    MediaSessionTest()
+        : m_page(DummyPageHolder::create(IntSize(1, 1)))
+    {}
+
+    MediaSession* createMediaSession(WebMediaSession* webMediaSession)
+    {
+        // The MediaSession takes ownership of the WebMediaSession, and the
+        // caller must take care to not end up with a stale pointer.
+        return MediaSession::createForTesting(adoptPtr(webMediaSession));
+    }
+
+    Document& document() { return m_page->document(); }
+    ScriptState* mainScriptState() { return ScriptState::forMainWorld(document().frame()); }
+private:
+    OwnPtr<DummyPageHolder> m_page;
+};
+
+#if OS(ANDROID)
+TEST_F(MediaSessionTest, Activate)

[mlamouri (slow - plz ping), 2015/09/28 15:16:03]

Can you follow the pattern we usually do in Chromium:

#if OS(ANDROID)
#define MAYBE_Activate Activate
#define MAYBE_Deactivate Deactivate
#else
#define MAYBE_Activate DISABLED_Activate
#define MAYBE_Deactivate DISABLED_Deactivate
#endif

TEST_F(MediaSessionTest, MAYBE_Activate)
{
...
}

TEST_F(MediaSessionTest, MAYBE_Deactivate)

{
...
}

[davve, 2015/10/06 12:05:13]

Done.

+#else
+TEST_F(MediaSessionTest, DISABLED_Activate)
+#endif
+{
+    MockWebMediaSession* mockWebMediaSession = new MockWebMediaSession;
+    MediaSession* mediaSession = createMediaSession(mockWebMediaSession);
+    EXPECT_CALL(*mockWebMediaSession, activate(_));

[mlamouri (slow - plz ping), 2015/09/28 15:16:03]

nit: leave empty line before, remove the one after.

Arguably these tests don't tests much butI guess they could be used for better
things later.

[davve, 2015/10/06 12:05:13]

Done.

+
+    mediaSession->activate(mainScriptState());
+}
+
+#if OS(ANDROID)
+TEST_F(MediaSessionTest, Deactivate)
+#else
+TEST_F(MediaSessionTest, DISABLED_Deactivate)
+#endif
+{
+    MockWebMediaSession* mockWebMediaSession = new MockWebMediaSession;
+    MediaSession* mediaSession = createMediaSession(mockWebMediaSession);
+    EXPECT_CALL(*mockWebMediaSession, deactivate(_));

[mlamouri (slow - plz ping), 2015/09/28 15:16:03]

nit: leave empty line before.

[davve, 2015/10/06 12:05:13]

Done.

+    mediaSession->deactivate(mainScriptState());
+}
+
+} // namespace
+} // namespace blink