[Chromoting] Refactoring DesktopEnvironment and moving screen/audio recorders to ClientSession.

remoting/host/event_executor.h

Index: remoting/host/event_executor.h
diff --git a/remoting/host/event_executor.h b/remoting/host/event_executor.h
index c660eb3601289c52c0adab639b78618b8faa178c..1032367346f1de96c20e0f763b5f814826d36a79 100644
--- a/remoting/host/event_executor.h
+++ b/remoting/host/event_executor.h
@@ -28,11 +28,11 @@ class EventExecutor : public protocol::ClipboardStub,
       scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner);
 
   // Initialises any objects needed to execute events.
-  virtual void OnSessionStarted(
+  virtual void Start(
       scoped_ptr<protocol::ClipboardStub> client_clipboard) = 0;
 
-  // Destroys any objects constructed by Start().
-  virtual void OnSessionFinished() = 0;
+  // Destroys any objects constructed by Start() and deletes |this|.
+  virtual void StopAndDelete() = 0;

[alexeypa (please no reviews), 2012/09/05 22:53:30]

I  am going to make EventExecutor a ref-counted object and get rid of this
instance of StopAndDelete. I'd like to do it in a follow up CL. I tired to make
it a part of this CL but the change turned out to be too large.

[Wez, 2012/09/06 00:13:23]

Let's discuss whether or not EventExecutor should be ref-counted off-CL!

 };
 
 }  // namespace remoting