Fixed drag and drop into and out of Browser Plugin.

content/public/browser/web_contents.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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
 
 #include "base/basictypes.h"
 #include "base/callback_forward.h"
 #include "base/process_util.h"
 #include "base/string16.h"
 #include "base/supports_user_data.h"
 #include "content/common/content_export.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/page_navigator.h"
 #include "content/public/browser/save_page_type.h"
 #include "content/public/browser/web_ui.h"
 #include "ipc/ipc_sender.h"
 #include "third_party/skia/include/core/SkColor.h"
+#include "third_party/WebKit/Source/WebKit/chromium/public/WebDragOperation.h"
 #include "ui/base/window_open_disposition.h"
 #include "ui/gfx/native_widget_types.h"
 #include "ui/gfx/size.h"
 
 namespace base {
 class TimeTicks;
 }
 
 namespace gfx {
 class Rect;
@@ skipping 306 matching lines @@
   virtual base::TimeTicks GetNewTabStartTime() const = 0;
 
   // Tells the tab to close now. The tab will take care not to close until it's
   // out of nested message loops.
   virtual void Close() = 0;
 
   // Notification that tab closing has started.  This can be called multiple
   // times, subsequent calls are ignored.
   virtual void OnCloseStarted() = 0;
 
-  // A render view-originated drag has ended. Informs the render view host and
-  // WebContentsDelegate.
+  // Informs the render view host and the BrowserPluginEmbedder, if present, of
+  // a Drag Source End.
+  virtual void DragSourceEndedAt(int client_x, int client_y, int screen_x,

[Fady Samuel, 2013/04/18 18:22:56]

Is this necessary?

[mthiesse, 2013/04/18 18:36:32]

Done.

+      int screen_y, WebKit::WebDragOperation operation) = 0;
+
+  // Informs the render view host and the BrowserPluginEmbedder, if present, of
+  // a Drag Source Move.
+  virtual void DragSourceMovedTo(int client_x, int client_y,

[Fady Samuel, 2013/04/18 18:22:56]

Is this necessary?

[mthiesse, 2013/04/18 18:36:32]

Done.

+                                 int screen_x, int screen_y) = 0;
+
+  // A render view-originated drag has ended. Informs the render view host, the
+  // WebContentsDelegate, and the BrowserPluginEmbedder, if present.

[Fady Samuel, 2013/04/18 18:22:56]

Please avoid referencing BrowserPluginEmbedder here as it's an implementation
detail. The notion of an EmbedderWebContents exists though, I think.

[mthiesse, 2013/04/18 18:36:32]

It's probably fine if I leave this comment as-is. Right?

   virtual void SystemDragEnded() = 0;
 
   // Notification the user has made a gesture while focus was on the
   // page. This is used to avoid uninitiated user downloads (aka carpet
   // bombing), see DownloadRequestLimiter for details.
   virtual void UserGestureDone() = 0;
 
   // Indicates if this tab was explicitly closed by the user (control-w, close
   // tab menu item...). This is false for actions that indirectly close the tab,
   // such as closing the window.  The setter is maintained by TabStripModel, and
@@ skipping 64 matching lines @@
 
  private:
   // This interface should only be implemented inside content.
   friend class WebContentsImpl;
   WebContents() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_