cc:: Add remote mode to the compositor

cc/test/remote_channel_impl_for_test.cc

Index: cc/test/remote_channel_impl_for_test.cc
diff --git a/cc/test/remote_channel_impl_for_test.cc b/cc/test/remote_channel_impl_for_test.cc
new file mode 100644
index 0000000000000000000000000000000000000000..440aa5422aaba44bacd209985b26b0d8b76c604b
--- /dev/null
+++ b/cc/test/remote_channel_impl_for_test.cc
@@ -0,0 +1,43 @@
+// Copyright 2016 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 "cc/test/remote_channel_impl_for_test.h"
+
+#include "cc/test/proxy_impl_for_test.h"
+
+namespace cc {
+
+scoped_ptr<RemoteChannelImplForTest> RemoteChannelImplForTest::Create(
+    TestHooks* test_hooks,
+    LayerTreeHost* layer_tree_host,
+    RemoteProtoChannel* remote_proto_channel,
+    TaskRunnerProvider* task_runner_provider) {
+  return make_scoped_ptr(new RemoteChannelImplForTest(
+      test_hooks, layer_tree_host, remote_proto_channel, task_runner_provider));
+}
+
+RemoteChannelImplForTest::RemoteChannelImplForTest(
+    TestHooks* test_hooks,
+    LayerTreeHost* layer_tree_host,
+    RemoteProtoChannel* remote_proto_channel,
+    TaskRunnerProvider* task_runner_provider)
+    : RemoteChannelImpl(layer_tree_host,
+                        remote_proto_channel,
+                        task_runner_provider),
+      test_hooks_(test_hooks),
+      proxy_impl_for_test_(nullptr) {}
+
+scoped_ptr<ProxyImpl> RemoteChannelImplForTest::CreateProxyImpl(
+    ChannelImpl* channel_impl,
+    LayerTreeHost* layer_tree_host,
+    TaskRunnerProvider* task_runner_provider,
+    scoped_ptr<BeginFrameSource> external_begin_frame_source) {
+  scoped_ptr<ProxyImplForTest> proxy_impl = ProxyImplForTest::Create(
+      test_hooks_, channel_impl, layer_tree_host, task_runner_provider,
+      std::move(external_begin_frame_source));
+  proxy_impl_for_test_ = proxy_impl.get();
+  return std::move(proxy_impl);

[ericrk, 2016/01/20 22:32:29]

nit: from what I remember, std::move breaks RVO, so when returning a move-only
type that matches the return type, it's better to just return directly without
std::move.

[Khushal, 2016/01/22 01:03:24]

We need std::move here since the return needs to upcast ProxyImplForTest to
ProxyImpl. I remember because I had to return directly and the compiler started
yelling at me.

[ericrk, 2016/01/22 18:36:59]

oh, true, missed that.

+}
+
+}  // namespace cc