[chromoting] Fix crash when generating access code

remoting/host/desktop_environment.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_DESKTOP_ENVIRONMENT_H_
 #define REMOTING_HOST_DESKTOP_ENVIRONMENT_H_
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/time.h"
 #include "remoting/base/scoped_thread_proxy.h"
 
 namespace remoting {
 
 class Capturer;
 class ChromotingHost;
 class ChromotingHostContext;
 
 namespace protocol {
 class HostEventStub;
 };
 
 class DesktopEnvironment {
  public:
-  static scoped_ptr<DesktopEnvironment> Create(ChromotingHostContext* context);
+  static scoped_ptr<DesktopEnvironment> Create(

[alexeypa (please no reviews), 2012/03/21 23:39:12]

Add a comment saying that this one creates DesktopEnvironemnt in a host plugin.

+      ChromotingHostContext* context);
+
+  // Create a DesktopEnvironment used in a service process.

[alexeypa (please no reviews), 2012/03/21 23:39:12]

nit: Creates ...

[Alpha Left Google, 2012/03/21 23:43:27]

Done.

+  static scoped_ptr<DesktopEnvironment> CreateForService(
+      ChromotingHostContext* context);
   static scoped_ptr<DesktopEnvironment> CreateFake(

[alexeypa (please no reviews), 2012/03/21 23:39:12]

nit: either add an empty line between CreateForService and CreateFake or remove
one between Create and CreateForService.

[Alpha Left Google, 2012/03/21 23:43:27]

There should be an empty line before comments. Added an empty line.

       ChromotingHostContext* context,
       scoped_ptr<Capturer> capturer,
       scoped_ptr<protocol::HostEventStub> event_executor);
 
   virtual ~DesktopEnvironment();
 
   void set_host(ChromotingHost* host) { host_ = host; }
 
   Capturer* capturer() const { return capturer_.get(); }
   protocol::HostEventStub* event_executor() const {
@@ skipping 17 matching lines @@
 
   // Executes input and clipboard events received from the client.
   scoped_ptr<protocol::HostEventStub> event_executor_;
 
   DISALLOW_COPY_AND_ASSIGN(DesktopEnvironment);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_DESKTOP_ENVIRONMENT_H_