Remote assistance on Chrome OS Part IV - It2MeHost

remoting/host/chromoting_host_context.h

 // Copyright (c) 2012 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 REMOTING_HOST_CHROMOTING_HOST_CONTEXT_H_
 #define REMOTING_HOST_CHROMOTING_HOST_CONTEXT_H_
 
 #include "base/gtest_prod_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 
 namespace net {
 class URLRequestContextGetter;
 }  // namespace net
 
+namespace policy {
+class PolicyService;
+}  // namespace policy
+
 namespace remoting {
 
 class AutoThreadTaskRunner;
 
 // A class that manages threads and running context for the chromoting host
 // process.  This class is virtual only for testing purposes (see below).
 class ChromotingHostContext {
  public:
   ~ChromotingHostContext();
 
   // Create threads and URLRequestContextGetter for use by a host.
   // During shutdown the caller should tear-down the ChromotingHostContext and
   // then continue to run until |ui_task_runner| is no longer referenced.
   // NULL is returned if any threads fail to start.
   static scoped_ptr<ChromotingHostContext> Create(
       scoped_refptr<AutoThreadTaskRunner> ui_task_runner);
 
+  // Must be called on the UI thread of the browser process.
+  static scoped_ptr<ChromotingHostContext> CreateForChromeOS(
+      scoped_refptr<AutoThreadTaskRunner> ui_task_runner,
+      scoped_refptr<net::URLRequestContextGetter> url_request_context_getter,
+      policy::PolicyService* policy_service);

[Wez, 2014/10/17 17:58:00]

What are the lifetime/ownership properties of |policy_service|?

[kelvinp, 2014/10/20 00:21:17]

Done.

+
   // Task runner for the thread used for audio capture and encoding.
   scoped_refptr<AutoThreadTaskRunner> audio_task_runner();
 
   // Task runner for the thread that is used for blocking file
   // IO. This thread is used by the URLRequestContext to read proxy
   // configuration and by NatConfig to read policy configs.
   scoped_refptr<AutoThreadTaskRunner> file_task_runner();
 
   // Task runner for the thread that is used by the InputInjector.
   //
@@ skipping 11 matching lines @@
 
   // Task runner for the thread used by the ScreenRecorder to capture
   // the screen.
   scoped_refptr<AutoThreadTaskRunner> video_capture_task_runner();
 
   // Task runner for the thread used to encode video streams.
   scoped_refptr<AutoThreadTaskRunner> video_encode_task_runner();
 
   scoped_refptr<net::URLRequestContextGetter> url_request_context_getter();
 
+  policy::PolicyService* policy_service();

[Wez, 2014/10/17 17:58:00]

Wouldn't it be cleaner to push the PolicyWatcher out into the context, so that
calling code can initialize it appropriately for the platform?

[Wez, 2014/10/17 17:58:00]

Add a comment to clarify how this is used / why it's needed?

[kelvinp, 2014/10/20 00:21:17]

Done.

[kelvinp, 2014/10/20 00:21:17]

Done.

+
  private:
-  ChromotingHostContext(AutoThreadTaskRunner* ui_task_runner);
+  ChromotingHostContext(
+      AutoThreadTaskRunner* ui_task_runner,
+      AutoThreadTaskRunner* file_task_runner,
+      AutoThreadTaskRunner* input_task_runner,
+      AutoThreadTaskRunner* network_task_runner,
+      AutoThreadTaskRunner* video_capture_task_runner,

[Wez, 2014/10/17 17:58:00]

Why are these bare pointers?

[kelvinp, 2014/10/20 00:21:17]

Done.

+      net::URLRequestContextGetter* url_request_context_getter,
+      policy::PolicyService* policy_service
+  );
+
+  // Verify that all threads have started.
+  bool VerifyInitialized();

[Wez, 2014/10/17 17:58:00]

Under what circumstances could the threads _not_ be started? They're passed in
to the ctor, after all?

[Wez, 2014/10/17 17:58:00]

AreThreadsStarted()?

[kelvinp, 2014/10/20 00:21:17]

Done.

+
+  // Caller-supplied UI thread. This is usually the application main thread.
+  scoped_refptr<AutoThreadTaskRunner> ui_task_runner_;
 
   // Thread for audio capture and encoding.
   scoped_refptr<AutoThreadTaskRunner> audio_task_runner_;
 
   // Thread for I/O operations.
   scoped_refptr<AutoThreadTaskRunner> file_task_runner_;
 
   // Thread for input injection.
   scoped_refptr<AutoThreadTaskRunner> input_task_runner_;
 
   // Thread for network operations.
   scoped_refptr<AutoThreadTaskRunner> network_task_runner_;
 
-  // Caller-supplied UI thread. This is usually the application main thread.
-  scoped_refptr<AutoThreadTaskRunner> ui_task_runner_;
-
   // Thread for screen capture.
   scoped_refptr<AutoThreadTaskRunner> video_capture_task_runner_;
 
   // Thread for video encoding.
   scoped_refptr<AutoThreadTaskRunner> video_encode_task_runner_;
 
   // Serves URLRequestContexts that use the network and UI task runners.
   scoped_refptr<net::URLRequestContextGetter> url_request_context_getter_;
 
+  policy::PolicyService* policy_service_;
+
   DISALLOW_COPY_AND_ASSIGN(ChromotingHostContext);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_CHROMOTING_HOST_CONTEXT_H_