Add support for passing an arbitrary parameter to an IPC message handler. The motivation is for Web…

chrome/renderer/pepper/pepper_extensions_common_host.cc

 // Copyright (c) 2013 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.
 
 #include "chrome/renderer/pepper/pepper_extensions_common_host.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/values.h"
 #include "chrome/renderer/extensions/chrome_v8_context.h"
@@ skipping 60 matching lines @@
   if (!context)
     return NULL;
 
   return new PepperExtensionsCommonHost(
       host, instance, resource, context->pepper_request_proxy());
 }
 
 int32_t PepperExtensionsCommonHost::OnResourceMessageReceived(
     const IPC::Message& msg,
     ppapi::host::HostMessageContext* context) {
-  IPC_BEGIN_MESSAGE_MAP(PepperExtensionsCommonHost, msg)
-  PPAPI_DISPATCH_HOST_RESOURCE_CALL(PpapiHostMsg_ExtensionsCommon_Post, OnPost)
-  PPAPI_DISPATCH_HOST_RESOURCE_CALL(PpapiHostMsg_ExtensionsCommon_Call, OnCall)
-  IPC_END_MESSAGE_MAP()
+  PPAPI_BEGIN_MESSAGE_MAP(PepperExtensionsCommonHost, msg)

[Tom Sepez, 2014/05/13 18:48:40]

Could we split this PPAPI_ cleanup into a separate CL that could go first, just
to reduce the heft of this one?

[jam, 2014/05/13 20:31:40]

Done: https://codereview.chromium.org/281803003/

+    PPAPI_DISPATCH_HOST_RESOURCE_CALL(PpapiHostMsg_ExtensionsCommon_Post,
+                                      OnPost)
+    PPAPI_DISPATCH_HOST_RESOURCE_CALL(PpapiHostMsg_ExtensionsCommon_Call,
+                                      OnCall)
+  PPAPI_END_MESSAGE_MAP()
   return PP_ERROR_FAILED;
 }
 
 void PepperExtensionsCommonHost::OnResponseReceived(
     ppapi::host::ReplyMessageContext reply_context,
     bool success,
     const base::ListValue& response,
     const std::string& /* error */) {
   reply_context.params.set_result(success ? PP_OK : PP_ERROR_FAILED);
   SendReply(reply_context, PpapiPluginMsg_ExtensionsCommon_CallReply(response));
@@ skipping 16 matching lines @@
   std::string error;
   bool success = pepper_request_proxy_->StartRequest(
       base::Bind(&PepperExtensionsCommonHost::OnResponseReceived,
                  weak_factory_.GetWeakPtr(),
                  context->MakeReplyMessageContext()),
       request_name,
       args,
       &error);
   return success ? PP_OK_COMPLETIONPENDING : PP_ERROR_FAILED;
 }