Implement supporting UI HRD-on-ChromeOS

chrome/browser/renderer_host/pepper/pepper_talk_host.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 CHROME_BROWSER_RENDERER_HOST_PEPPER_PEPPER_TALK_HOST_H_
 #define CHROME_BROWSER_RENDERER_HOST_PEPPER_PEPPER_TALK_HOST_H_
 
 #include "base/memory/weak_ptr.h"
 #include "ppapi/c/private/ppb_talk_private.h"
 #include "ppapi/host/resource_host.h"
@@ skipping 11 matching lines @@
 
 namespace chrome {
 
 class PepperTalkHost : public ppapi::host::ResourceHost {
  public:
   PepperTalkHost(content::BrowserPpapiHost* host,
                  PP_Instance instance,
                  PP_Resource resource);
   virtual ~PepperTalkHost();
 
+ private:
   // ResourceHost override.
   virtual int32_t OnResourceMessageReceived(
       const IPC::Message& msg,
       ppapi::host::HostMessageContext* context) OVERRIDE;
 
-  // Sends the reply.
-  void GotTalkPermission(ppapi::host::ReplyMessageContext reply);
-
- private:
-  int32_t OnRequestPermission(
-      ppapi::host::HostMessageContext* context,
-      PP_TalkPermission permission);
-  int32_t OnStartRemoting(
-      ppapi::host::HostMessageContext* context);
-  int32_t OnStopRemoting(
-      ppapi::host::HostMessageContext* context);
+  int32_t OnRequestPermission(ppapi::host::HostMessageContext* context,

[raymes, 2013/06/18 00:30:51]

nit: usually we have the message handlers grouped together.

[dcaiafa, 2013/06/18 16:36:40]

Done.

+                              PP_TalkPermission permission);
+  void OnRequestPermissionCompleted(ppapi::host::ReplyMessageContext reply);
+  int32_t OnStartRemoting(ppapi::host::HostMessageContext* context);
+  void OnStartRemotingCompleted(ppapi::host::ReplyMessageContext reply);
+  int32_t OnStopRemoting(ppapi::host::HostMessageContext* context);
+  void OnStopRemotingCompleted(ppapi::host::ReplyMessageContext reply);
+  void OnRemotingStopEvent();
 
   base::WeakPtrFactory<PepperTalkHost> weak_factory_;
   content::BrowserPpapiHost* browser_ppapi_host_;
+  bool remoting_started_;
 
   DISALLOW_COPY_AND_ASSIGN(PepperTalkHost);
 };
 
 }  // namespace chrome
 
 #endif  // CHROME_BROWSER_RENDERER_HOST_PEPPER_PEPPER_TALK_HOST_H_