Add file thread in MessageHostContext.

remoting/host/chromoting_host_context.h

Index: remoting/host/chromoting_host_context.h
diff --git a/remoting/host/chromoting_host_context.h b/remoting/host/chromoting_host_context.h
index d1b87ed6b18776b4608ec88458fb782cbbdcbac9..466394bcc134603704684d1fa6bf8cf33386d2eb 100644
--- a/remoting/host/chromoting_host_context.h
+++ b/remoting/host/chromoting_host_context.h
@@ -19,8 +19,7 @@ namespace remoting {
 class ChromotingHostContext {
  public:
   // Create a context.
-  ChromotingHostContext(base::MessageLoopProxy* io_message_loop,
-                        base::MessageLoopProxy* ui_message_loop);
+  ChromotingHostContext(base::MessageLoopProxy* ui_message_loop);
   virtual ~ChromotingHostContext();
 
   // TODO(ajwong): Move the Start method out of this class. Then
@@ -32,12 +31,12 @@ class ChromotingHostContext {
 
   virtual JingleThread* jingle_thread();
 
-  virtual base::MessageLoopProxy* io_message_loop();
-  virtual base::MessageLoopProxy* ui_message_loop();
   virtual MessageLoop* main_message_loop();
   virtual MessageLoop* encode_message_loop();
   virtual base::MessageLoopProxy* network_message_loop();
   virtual MessageLoop* desktop_message_loop();
+  virtual base::MessageLoopProxy* ui_message_loop();

[Wez, 2012/04/17 01:14:59]

Why have you moved this?

[Sergey Ulanov, 2012/04/17 01:28:09]

So that the main message loop is the first in the list.

+  virtual MessageLoop* file_message_loop();
 
  private:
   FRIEND_TEST_ALL_PREFIXES(ChromotingHostContextTest, StartAndStop);
@@ -45,7 +44,8 @@ class ChromotingHostContext {
   // A thread that hosts all network operations.
   JingleThread jingle_thread_;
 
-  // A thread that hosts ChromotingHost and performs rate control.
+  // TODO(sergeyu): The "main" thread is used just by the
+  // capturer. Should it be renamed to capturer_thread?

[Wez, 2012/04/17 01:14:59]

I agree that the name is unhelpful but let's discuss this offline.

[Sergey Ulanov, 2012/04/17 01:28:09]

Changed the comment to "Consider renaming it."

   base::Thread main_thread_;
 
   // A thread that hosts all encode operations.
@@ -55,7 +55,9 @@ class ChromotingHostContext {
   // This is NOT a Chrome-style UI thread.
   base::Thread desktop_thread_;
 
-  scoped_refptr<base::MessageLoopProxy> io_message_loop_;
+  // Thread for blocking IO operations.
+  base::Thread file_thread_;
+
   scoped_refptr<base::MessageLoopProxy> ui_message_loop_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromotingHostContext);