cc: Split ThreadProxy into ProxyMain and ProxyImpl

cc/trees/channel_main.h

 // 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.
 
 #ifndef CC_TREES_CHANNEL_MAIN_H_
 #define CC_TREES_CHANNEL_MAIN_H_
 
 #include "cc/base/cc_export.h"
 #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.
 //
 // The communication sequence between the 2 sides is:
 //
 // LayerTreeHost<-->ProxyMain<-->ChannelMain
 //                                      |
 //                                      |
 //                               ChannelImpl<-->ProxyImpl<-->LayerTreeHostImpl
 
@@ skipping 14 matching lines @@
   virtual void FinishGLOnImpl(CompletionEvent* completion) = 0;
   virtual void MainFrameWillHappenOnImplForTesting(
       CompletionEvent* completion,
       bool* main_frame_will_happen) = 0;
   virtual void SetNeedsRedrawOnImpl(const gfx::Rect& damage_rect) = 0;
   virtual void SetNeedsCommitOnImpl() = 0;
   virtual void BeginMainFrameAbortedOnImpl(CommitEarlyOutReason reason) = 0;
   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() {}
 };
 
 }  // namespace cc
 
 #endif  // CC_TREES_CHANNEL_MAIN_H_