| Index: media/mojo/services/media_mojo_unittest.cc
|
| diff --git a/media/mojo/services/media_mojo_unittest.cc b/media/mojo/services/media_mojo_unittest.cc
|
| index 2c1955a21d4203d0d44d50c7ae1fee7da11ad501..8f4d52067e18a287bfd0d4fdc3a58ce06fe05fe2 100644
|
| --- a/media/mojo/services/media_mojo_unittest.cc
|
| +++ b/media/mojo/services/media_mojo_unittest.cc
|
| @@ -48,13 +48,7 @@ class MockRendererClient : public mojom::RendererClient {
|
| 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
|
| - // If we need to use this mock method before the gmock bug gets fixed, we
|
| - // would need to define a MOCK_METHOD using non movable type which calls this.
|
| - void OnVideoNaturalSizeChange(mojo::SizePtr size) override {}
|
| + MOCK_METHOD1(OnVideoNaturalSizeChange, void(const gfx::Size& size));
|
|
|
| private:
|
| DISALLOW_COPY_AND_ASSIGN(MockRendererClient);
|
|
|