Moves video frame callbacks from VideoFrameCompositor to Renderer.

media/mojo/services/media_mojo_unittest.cc

 // 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 <stdint.h>
 
 #include <memory>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 21 matching lines @@
 namespace {
 
 #if defined(ENABLE_MOJO_CDM)
 const char kClearKeyKeySystem[] = "org.w3.clearkey";
 const char kInvalidKeySystem[] = "invalid.key.system";
 #endif
 const char kSecurityOrigin[] = "http://foo.com";
 
 class MockRendererClient : public interfaces::RendererClient {
  public:
-  MockRendererClient(){};
-  ~MockRendererClient() override{};
+  MockRendererClient() {}
+  ~MockRendererClient() override {}
 
   // interfaces::RendererClient implementation.
   MOCK_METHOD2(OnTimeUpdate, void(int64_t time_usec, int64_t max_time_usec));
   MOCK_METHOD1(OnBufferingStateChange, void(interfaces::BufferingState state));
   MOCK_METHOD0(OnEnded, void());
   MOCK_METHOD0(OnError, void());
+  MOCK_METHOD1(OnVideoOpacityChange, void(bool opaque));
+
+  // TODO(alokp): gmock does not support move-only function arguments.
+  // Convert this into MOCK_METHOD after gmock implements this feature.
+  // https://github.com/google/googletest/issues/395
+  void OnVideoNaturalSizeChange(mojo::SizePtr size) override {}

[xhwang, 2016/05/17 06:24:13]

You can define a MOCK_METHOD using non movable type and call it here to achieve
the same result.

[alokp, 2016/05/17 16:25:49]

Acknowledged. I will append this to the comment. It can be done if we need to
use the mock method.

 
  private:
   DISALLOW_COPY_AND_ASSIGN(MockRendererClient);
 };
 
 class MediaShellTest : public shell::test::ShellTest {
  public:
   MediaShellTest()
       : ShellTest("exe:media_mojo_unittests"),
         renderer_client_binding_(&renderer_client_),
@@ skipping 119 matching lines @@
   // close the connection.
   EXPECT_CALL(*this, ConnectionClosed())
       .Times(Exactly(1))
       .WillOnce(Invoke(run_loop_.get(), &base::RunLoop::Quit));
   service_factory_.reset();
 
   run_loop_->Run();
 }
 
 }  // namespace media