cc: Split ThreadProxy into ProxyMain and ProxyImpl

cc/trees/channel_main.h

Index: cc/trees/channel_main.h
diff --git a/cc/trees/channel_main.h b/cc/trees/channel_main.h
index 9c6689c1c4f4c6f9e993494fa725472e06df1046..6ff6fab114f19c54b80a4544314aab14f388d31c 100644
--- a/cc/trees/channel_main.h
+++ b/cc/trees/channel_main.h
@@ -9,10 +9,12 @@
 #include "cc/base/completion_event.h"
 #include "cc/input/top_controls_state.h"
 #include "cc/output/output_surface.h"
+#include "cc/scheduler/begin_frame_source.h"
 #include "cc/scheduler/commit_earlyout_reason.h"
 #include "cc/trees/proxy_common.h"
 
 namespace cc {
+
 // ChannelMain and ChannelImpl provide an abstract communication layer for
 // the main and impl side of the compositor.
 //
@@ -47,9 +49,15 @@ class CC_EXPORT ChannelMain {
   virtual void StartCommitOnImpl(CompletionEvent* completion,
                                  LayerTreeHost* layer_tree_host,
                                  bool hold_commit_for_activation) = 0;
-  virtual void InitializeImplOnImpl(CompletionEvent* completion,
-                                    LayerTreeHost* layer_tree_host) = 0;
-  virtual void LayerTreeHostClosedOnImpl(CompletionEvent* completion) = 0;
+
+  // Must be called before using the channel.
+  virtual void InitializeImpl(
+      CompletionEvent* completion,
+      LayerTreeHost* layer_tree_host,
+      scoped_ptr<BeginFrameSource> external_begin_frame_source) = 0;
+  // Must be called before deleting the channel.
+  virtual void CloseImpl(CompletionEvent* completion) = 0;
+  virtual bool IsInitialized() const = 0;

[Wez, 2015/11/10 01:41:50]

nit: Suggest separating the three methods with blank lines, otherwise it looks
like "Must be called before deleting the channel" may also apply to
IsInitialized().

[Khushal, 2015/11/11 04:17:47]

Done.

 
   virtual ~ChannelMain() {}
 };