Pepper: Add routing ID information to Connection.

content/renderer/pepper/pepper_in_process_router.cc

 // 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.
 
 #include "content/renderer/pepper/pepper_in_process_router.h"
 
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
 #include "content/renderer/pepper/renderer_ppapi_host_impl.h"
 #include "ipc/ipc_message.h"
@@ skipping 40 matching lines @@
 }
 
 IPC::Sender* PepperInProcessRouter::GetPluginToRendererSender() {
   return plugin_to_host_router_.get();
 }
 
 IPC::Sender* PepperInProcessRouter::GetRendererToPluginSender() {
   return host_to_plugin_router_.get();
 }
 
-ppapi::proxy::Connection PepperInProcessRouter::GetPluginConnection() {
+ppapi::proxy::Connection PepperInProcessRouter::GetPluginConnection(
+    PP_Instance instance) {

[yzshen1, 2013/07/23 17:44:01]

GetRoutingIDForWidget() doesn't always return the same routing ID for an
instance:
"""
This will take into account the current Flash fullscreen state, so if there is a
Flash fullscreen instance active, this will return the routing ID of the
fullscreen widget.
"""

I think you always want the routing ID of the render view hosting the plugin,
instead of the fullscreen render widget, right?

[teravest, 2013/07/23 17:59:51]

Changed to get the routing ID from the render view.

+  int routing_id = host_impl_->GetRoutingIDForWidget(instance);
   return ppapi::proxy::Connection(dummy_browser_channel_.get(),
-                                  plugin_to_host_router_.get());
+                                  plugin_to_host_router_.get(),
+                                  routing_id);
 }
 
 bool PepperInProcessRouter::SendToHost(IPC::Message* msg) {
   scoped_ptr<IPC::Message> message(msg);
 
   if (!message->is_sync()) {
     bool result = host_impl_->GetPpapiHost()->OnMessageReceived(*message);
     DCHECK(result) << "The message was not handled by the host.";
     return true;
   }
@@ skipping 53 matching lines @@
           reply_params.pp_resource());
   if (!resource) {
     // The resource could have been destroyed while the async processing was
     // pending. Just drop the message.
     return;
   }
   resource->OnReplyReceived(reply_params, nested_msg);
 }
 
 }  // namespace content