Fix message routing for BrowserPlugin in iframe

content/browser/browser_plugin/browser_plugin_guest.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/browser/browser_plugin/browser_plugin_guest.h"
 
 #include <algorithm>
 
 #include "base/message_loop/message_loop.h"
 #include "base/pickle.h"
@@ skipping 377 matching lines @@
 void BrowserPluginGuest::SendMessageToEmbedder(IPC::Message* msg) {
   if (!attached()) {
     // Some pages such as data URLs, javascript URLs, and about:blank
     // do not load external resources and so they load prior to attachment.
     // As a result, we must save all these IPCs until attachment and then
     // forward them so that the embedder gets a chance to see and process
     // the load events.
     pending_messages_.push_back(linked_ptr<IPC::Message>(msg));
     return;
   }
-  msg->set_routing_id(owner_web_contents_->GetRoutingID());

[lazyboy, 2014/12/16 23:57:30]

The comment about this routing_id in browser_plugin_guest.h has gone stale,
please update.

[Charlie Reis, 2014/12/17 00:06:44]

What is it set to instead?

(Is this still for routed messages, or is it for control messages now?)

[Fady Samuel, 2014/12/17 19:30:35]

These messages are control messages now. All BrowserPluginMsgs have instance_id
as their first parameter which is used for routing to the appropriate
BrowserPlugin.

[Fady Samuel, 2014/12/17 19:30:35]

Done.

   owner_web_contents_->Send(msg);
 }
 
 void BrowserPluginGuest::DragSourceEndedAt(int client_x, int client_y,
     int screen_x, int screen_y, blink::WebDragOperation operation) {
   web_contents()->GetRenderViewHost()->DragSourceEndedAt(client_x, client_y,
       screen_x, screen_y, operation);
 }
 
 void BrowserPluginGuest::EndSystemDrag() {
@@ skipping 437 matching lines @@
 void BrowserPluginGuest::OnImeCompositionRangeChanged(
       const gfx::Range& range,
       const std::vector<gfx::Rect>& character_bounds) {
   static_cast<RenderWidgetHostViewBase*>(
       web_contents()->GetRenderWidgetHostView())->ImeCompositionRangeChanged(
           range, character_bounds);
 }
 #endif
 
 }  // namespace content