PPAPI: Refactor renderer side of browser host creation

ppapi/proxy/ppapi_messages.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.
 
 // Multiply-included message header, no traditional include guard.
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 411 matching lines @@
 // The list of switches can be found in:
 //   components/nacl/browser/nacl_process_host.cc
 IPC_STRUCT_TRAITS_BEGIN(ppapi::PpapiNaClPluginArgs)
   IPC_STRUCT_TRAITS_MEMBER(off_the_record)
   IPC_STRUCT_TRAITS_MEMBER(permissions)
   IPC_STRUCT_TRAITS_MEMBER(keepalive_throttle_interval_milliseconds)
   IPC_STRUCT_TRAITS_MEMBER(switch_names)
   IPC_STRUCT_TRAITS_MEMBER(switch_values)
 IPC_STRUCT_TRAITS_END()
 
+IPC_STRUCT_TRAITS_BEGIN(ppapi::proxy::CompletedBrowserResourceHosts)
+  IPC_STRUCT_TRAITS_MEMBER(sequence_num)

[Tom Sepez, 2014/11/14 20:33:58]

nit: some truly compulsive folks might want you to not abbreviate and write out
"sequence_number" instead of "sequence_num."  I'm not one of them.

[dmichael (off chromium), 2014/11/17 22:26:20]

I can sympathize. Changed.

+  IPC_STRUCT_TRAITS_MEMBER(host_ids)
+IPC_STRUCT_TRAITS_END()
+
 #if !defined(OS_NACL) && !defined(NACL_WIN64)
 
 IPC_STRUCT_TRAITS_BEGIN(ppapi::proxy::PPPDecryptor_Buffer)
   IPC_STRUCT_TRAITS_MEMBER(resource)
   IPC_STRUCT_TRAITS_MEMBER(handle)
   IPC_STRUCT_TRAITS_MEMBER(size)
 IPC_STRUCT_TRAITS_END()
 
 #endif  // !defined(OS_NACL) && !defined(NACL_WIN64)
 
@@ skipping 887 matching lines @@
     int /* child_process_id */,
     ppapi::proxy::ResourceMessageCallParams /* params */,
     PP_Instance /* instance */,
     std::vector<IPC::Message> /* nested_msgs */)
 
 // This message is sent from the browser to the renderer when it has created
 // ResourceHosts for the renderer. It contains the sequence number that was sent
 // in the request and the IDs of the pending ResourceHosts which were created in
 // the browser. These IDs are only useful for the plugin which can attach to the
 // ResourceHosts in the browser.
-IPC_MESSAGE_ROUTED2(
+IPC_MESSAGE_ROUTED1(
     PpapiHostMsg_CreateResourceHostsFromHostReply,
-    int32_t /* sequence */,
-    std::vector<int> /* pending_host_ids */)
+    ppapi::proxy::CompletedBrowserResourceHosts /* completed_hosts */)
 
 //-----------------------------------------------------------------------------
 // Messages for resources using call/reply above.
 
 // Broker ----------------------------------------------------------------------
 IPC_MESSAGE_CONTROL0(PpapiHostMsg_Broker_Create)
 
 // Queries whether the plugin has permission to connect to the Pepper broker.
 // The response is contained in the error value of the
 // ResourceMessageReplyParams in the reply message.
@@ skipping 921 matching lines @@
 IPC_MESSAGE_CONTROL1(PpapiHostMsg_Talk_RequestPermission,
                      PP_TalkPermission /* permission */)
 IPC_MESSAGE_CONTROL0(PpapiPluginMsg_Talk_RequestPermissionReply)
 IPC_MESSAGE_CONTROL0(PpapiHostMsg_Talk_StartRemoting)
 IPC_MESSAGE_CONTROL0(PpapiPluginMsg_Talk_StartRemotingReply)
 IPC_MESSAGE_CONTROL0(PpapiHostMsg_Talk_StopRemoting)
 IPC_MESSAGE_CONTROL0(PpapiPluginMsg_Talk_StopRemotingReply)
 IPC_MESSAGE_CONTROL1(PpapiPluginMsg_Talk_NotifyEvent, PP_TalkEvent /* event */)
 
 #endif  // !defined(OS_NACL) && !defined(NACL_WIN64)