Fixed drag and drop into and out of Browser Plugin.

content/browser/browser_plugin/browser_plugin_embedder.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_embedder.h"
 
 #include "base/command_line.h"
 #include "base/stl_util.h"
 #include "content/browser/browser_plugin/browser_plugin_guest.h"
 #include "content/browser/browser_plugin/browser_plugin_host_factory.h"
 #include "content/browser/renderer_host/render_view_host_impl.h"
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/common/browser_plugin_messages.h"
+#include "content/common/drag_messages.h"
 #include "content/common/gpu/gpu_messages.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_source.h"
 #include "content/public/browser/notification_types.h"
 #include "content/public/browser/user_metrics.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/common/result_codes.h"
 #include "content/public/common/url_constants.h"
 #include "net/base/escape.h"
@@ skipping 164 matching lines @@
 
 BrowserPluginGuest* BrowserPluginEmbedder::GetGuestByInstanceID(
     int instance_id) const {
   ContainerInstanceMap::const_iterator it =
       guest_web_contents_by_instance_id_.find(instance_id);
   if (it != guest_web_contents_by_instance_id_.end())
     return static_cast<WebContentsImpl*>(it->second)->GetBrowserPluginGuest();
   return NULL;
 }
 
+BrowserPluginGuest* BrowserPluginEmbedder::GetDraggingGuest() const {
+  ContainerInstanceMap::const_iterator it =
+      guest_web_contents_by_instance_id_.begin();
+  for (; it != guest_web_contents_by_instance_id_.end(); ++it) {
+    if (static_cast<WebContentsImpl*>(
+        it->second)->GetBrowserPluginGuest()->dragging())

[Fady Samuel, 2013/02/01 15:54:55]

If we're effectively only allowing one guest to be dragging/dragged over at any
one time, why is the state stored in BrowserPluginGuest and not
BrowserPluginEmbedder?

[mthiesse1, 2013/02/01 16:02:58]

We need a way to identify which guest is the one that's dragging/being dragged
over, and the embedder doesn't know this based on the messages it receives. The
DragHost messages don't include instance IDs.

[mthiesse, 2013/02/01 16:53:44]

Done.

+      return static_cast<WebContentsImpl*>(
+          it->second)->GetBrowserPluginGuest();
+  }
+  return NULL;
+}
+
+bool BrowserPluginEmbedder::IsDragOverGuest() const {
+  ContainerInstanceMap::const_iterator it =
+      guest_web_contents_by_instance_id_.begin();
+  for (; it != guest_web_contents_by_instance_id_.end(); ++it) {
+    if (static_cast<WebContentsImpl*>(
+        it->second)->GetBrowserPluginGuest()->drag_over())
+      return true;
+  }
+  return false;
+}
+
 void BrowserPluginEmbedder::DestroyGuestByInstanceID(int instance_id) {
   BrowserPluginGuest* guest = GetGuestByInstanceID(instance_id);
   if (guest) {
     WebContents* guest_web_contents = guest->GetWebContents();
 
     // Destroy the guest's web_contents.
     delete guest_web_contents;
     guest_web_contents_by_instance_id_.erase(instance_id);
   }
 }
@@ skipping 13 matching lines @@
 
 void BrowserPluginEmbedder::RenderViewDeleted(
     RenderViewHost* render_view_host) {
 }
 
 void BrowserPluginEmbedder::RenderViewGone(base::TerminationStatus status) {
   CleanUp();
 }
 
 bool BrowserPluginEmbedder::OnMessageReceived(const IPC::Message& message) {
+  if (message.type() == DragHostMsg_DragStopped::ID) {
+    BrowserPluginGuest* guest = GetDraggingGuest();
+    if (guest)
+      return guest->OnMessageReceived(message);
+  }
+  if (message.type() == DragHostMsg_UpdateDragCursor::ID){
+    // If the guest is being dragged over we want to mark this as handled so
+    // that we don't update the cursor twice, once by the guest, and once by the
+    // embedder (and have the cursor flicker between droppable and
+    // non-droppable).
+    return IsDragOverGuest();
+  }
   if (ShouldForwardToBrowserPluginGuest(message)) {
     int instance_id = 0;
     // All allowed messages must have instance_id as their first parameter.
     PickleIterator iter(message);
     bool success = iter.ReadInt(&instance_id);
     DCHECK(success);
     BrowserPluginGuest* guest = GetGuestByInstanceID(instance_id);
     if (guest && guest->OnMessageReceivedFromEmbedder(message))
       return true;
   }
@@ skipping 124 matching lines @@
                                              uint32 sync_point) {
   AcceleratedSurfaceMsg_BufferPresented_Params ack_params;
   ack_params.mailbox_name = mailbox_name;
   ack_params.sync_point = sync_point;
   RenderWidgetHostImpl::AcknowledgeBufferPresent(route_id,
                                                  gpu_host_id,
                                                  ack_params);
 }
 
 }  // namespace content