media: scaffolding and plumbing for MojoRenderer{Impl, Service}

media/mojo/services/renderer_unittest.cc

Index: media/mojo/services/renderer_unittest.cc
diff --git a/media/mojo/services/renderer_unittest.cc b/media/mojo/services/renderer_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..fbcf9f7fd81f67037af20e5548a0521e243ec1fe
--- /dev/null
+++ b/media/mojo/services/renderer_unittest.cc
@@ -0,0 +1,191 @@
+// Copyright 2014 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 "base/at_exit.h"
+#include "base/bind.h"
+#include "base/command_line.h"
+#include "base/message_loop/message_loop.h"
+#include "base/run_loop.h"
+#include "media/base/audio_decoder_config.h"
+#include "media/base/channel_layout.h"
+#include "media/base/demuxer.h"
+#include "media/base/sample_format.h"
+#include "media/base/video_decoder_config.h"
+#include "media/mojo/services/mojo_renderer_impl.h"
+#include "mojo/public/c/system/main.h"
+#include "mojo/public/cpp/application/application_delegate.h"
+#include "mojo/public/cpp/application/application_impl.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace {
+
+// This class is here to give the gtest class access to the
+// mojo::ApplicationImpl so that the tests can connect to other applications.
+class MojoRendererTestHelper : public mojo::ApplicationDelegate {
+ public:
+  MojoRendererTestHelper() : application_impl_(NULL) {}
+  virtual ~MojoRendererTestHelper() {}
+
+  // ApplicationDelegate implementation.
+  virtual void Initialize(mojo::ApplicationImpl* app) OVERRIDE {
+    application_impl_ = app;
+  }
+
+  mojo::ApplicationImpl* application_impl() { return application_impl_; }
+
+ private:
+  mojo::ApplicationImpl* application_impl_;
+
+  DISALLOW_COPY_AND_ASSIGN(MojoRendererTestHelper);
+};
+
+// TODO(tim): Reconcile this with mojo apptest framework when ready.
+MojoRendererTestHelper* g_test_delegate = NULL;
+
+// TODO(tim): Make media::FakeDemuxerStream support audio and use that for the
+// DemuxerStream implementation instead.
+class FakeDemuxer : public media::Demuxer,

[scherkus (not reviewing), 2014/09/09 20:35:33]

when xhwang@'s DemuxerStreamProvider CL lands, you might be able to replace this
with the existing FakeDemuxerStream inside of media/

there's always MockDemuxer{Stream} as well if gmock is your thing

[tim (not reviewing), 2014/09/10 23:08:31]

Well, FDS requires considerable work to support audio as well as it supports
video (if i understand correctly), so I did this because my needs are super
basic and it seemed weird to have a FDS implementation where the audio path is
an entirely lighter weight implementation than the video path in the same class.
 WDYT?

+                    public media::DemuxerStream {
+ public:
+  FakeDemuxer() {}
+  virtual ~FakeDemuxer() {}
+
+  // media::Demuxer implementation.
+  virtual void Initialize(media::DemuxerHost* host,
+                          const media::PipelineStatusCB& status_cb,
+                          bool enable_text_tracks) OVERRIDE {}
+
+  virtual void Seek(base::TimeDelta time,
+                    const media::PipelineStatusCB& status_cb) OVERRIDE {}
+
+  virtual void Stop() OVERRIDE {}
+
+  virtual media::DemuxerStream* GetStream(
+      media::DemuxerStream::Type type) OVERRIDE {
+    DCHECK_EQ(media::DemuxerStream::AUDIO, type);
+    return this;
+  }
+
+  virtual base::Time GetTimelineOffset() const OVERRIDE {
+    return base::Time();
+  }
+
+  virtual media::Demuxer::Liveness GetLiveness() const OVERRIDE {
+    return media::Demuxer::LIVENESS_UNKNOWN;
+  }
+
+  // media::DemuxerStream implementation.
+  virtual void Read(const ReadCB& read_cb) OVERRIDE {}
+
+  virtual media::AudioDecoderConfig audio_decoder_config() OVERRIDE {
+    media::AudioDecoderConfig config;
+    config.Initialize(media::kCodecAAC, media::kSampleFormatU8,
+        media::CHANNEL_LAYOUT_SURROUND, 48000, NULL, 0,
+        false, false, base::TimeDelta(), 0);
+    return config;
+  }
+
+  virtual media::VideoDecoderConfig video_decoder_config() OVERRIDE {
+    NOTREACHED();
+    return media::VideoDecoderConfig();
+  }
+
+  virtual media::DemuxerStream::Type type() OVERRIDE {
+    return media::DemuxerStream::AUDIO;
+  }
+
+  virtual void EnableBitstreamConverter() OVERRIDE {}
+
+  virtual bool SupportsConfigChanges() OVERRIDE {
+    return true;
+  }
+
+  virtual media::VideoRotation video_rotation() OVERRIDE {
+    NOTREACHED();
+    return media::VIDEO_ROTATION_0;
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(FakeDemuxer);
+};
+
+}  // namespace
+
+namespace media {
+
+class MojoRendererTest : public testing::Test {
+ public:
+  MojoRendererTest() : service_provider_(NULL) {}
+
+  virtual void SetUp() OVERRIDE {
+    demuxer_.reset(new FakeDemuxer());
+    service_provider_ =
+        g_test_delegate->application_impl()->ConnectToApplication(
+            "mojo:media_mojo_renderer_app")->GetServiceProvider();
+  }
+
+  mojo::ServiceProvider* service_provider() { return service_provider_; }
+  media::Demuxer* demuxer() { return demuxer_.get(); }

[scherkus (not reviewing), 2014/09/09 20:35:33]

remove media:: here and everywhere else

[tim (not reviewing), 2014/09/10 23:08:31]

Done.

+  scoped_refptr<base::SingleThreadTaskRunner> task_runner() {
+    return base::MessageLoop::current()->task_runner();
+  }
+
+ private:
+  scoped_ptr<media::Demuxer> demuxer_;
+  mojo::ServiceProvider* service_provider_;
+
+  DISALLOW_COPY_AND_ASSIGN(MojoRendererTest);
+};
+
+void ErrorCallback(media::PipelineStatus* output,
+                   media::PipelineStatus status) {
+  *output = status;
+}
+
+// Tests that a MojoRendererImpl can successfully establish communication
+// with a MojoRendererService and set up a MojoDemuxerStream
+// connection. The test also initializes a media::AudioRendererImpl which
+// will error-out expectedly due to lack of support for decoder selection.
+TEST_F(MojoRendererTest, BasicInitialize) {
+  MojoRendererImpl rimpl(task_runner(), demuxer(), service_provider());
+  PipelineStatus expected_error(PIPELINE_OK);
+  rimpl.Initialize(base::MessageLoop::current()->QuitClosure(),
+                   media::StatisticsCB(),
+                   base::Closure(),
+                   base::Bind(&ErrorCallback, &expected_error),
+                   media::BufferingStateCB(),
+                   media::Renderer::TimeDeltaCB());
+  base::MessageLoop::current()->Run();
+
+  // We expect an error during initialization because MojoRendererService
+  // doesn't initialize any decoders, which causes an error.
+  EXPECT_EQ(PIPELINE_ERROR_COULD_NOT_RENDER, expected_error);
+}
+
+}  // namespace media
+
+MojoResult MojoMain(MojoHandle shell_handle) {
+  base::CommandLine::Init(0, NULL);
+#if !defined(COMPONENT_BUILD)
+  base::AtExitManager at_exit;
+#endif
+
+  // TODO(tim): Reconcile this with apptest framework when it is ready.
+  scoped_ptr<mojo::ApplicationDelegate> delegate(new MojoRendererTestHelper());
+  g_test_delegate = static_cast<MojoRendererTestHelper*>(delegate.get());
+  {
+    base::MessageLoop loop;
+    mojo::ApplicationImpl impl(delegate.get(),
+        mojo::MakeScopedHandle(mojo::MessagePipeHandle(shell_handle)));
+
+    int argc = 0;
+    char** argv = NULL;
+    testing::InitGoogleTest(&argc, argv);
+    mojo_ignore_result(RUN_ALL_TESTS());
+  }
+
+  g_test_delegate = NULL;
+  delegate.reset();
+  return MOJO_RESULT_OK;
+}