Plumb event flags (shift/alt/ctrl modifiers) for drag/drop events to WebKit.

content/browser/renderer_host/render_view_host_impl.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/renderer_host/render_view_host_impl.h"
 
 #include <set>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 485 matching lines @@
 }
 
 int RenderViewHostImpl::GetPendingRequestId() {
   return pending_request_id_;
 }
 
 void RenderViewHostImpl::DragTargetDragEnter(
     const WebDropData& drop_data,
     const gfx::Point& client_pt,
     const gfx::Point& screen_pt,
-    WebDragOperationsMask operations_allowed) {
+    WebDragOperationsMask operations_allowed,
+    int key_modifiers) {
   const int renderer_id = GetProcess()->GetID();
   ChildProcessSecurityPolicyImpl* policy =
       ChildProcessSecurityPolicyImpl::GetInstance();
 
   // The URL could have been cobbled together from any highlighted text string,
   // and can't be interpreted as a capability.
   WebDropData filtered_data(drop_data);
   FilterURL(policy, renderer_id, true, &filtered_data.url);
 
   // The filenames vector, on the other hand, does represent a capability to
@@ skipping 23 matching lines @@
 
   fileapi::IsolatedContext* isolated_context =
       fileapi::IsolatedContext::GetInstance();
   DCHECK(isolated_context);
   std::string filesystem_id = isolated_context->RegisterIsolatedFileSystem(
       filesets);
   policy->GrantAccessFileSystem(renderer_id, filesystem_id);
   filtered_data.filesystem_id = UTF8ToUTF16(filesystem_id);
 
   Send(new DragMsg_TargetDragEnter(GetRoutingID(), filtered_data, client_pt,
-                                   screen_pt, operations_allowed));
+                                   screen_pt, operations_allowed,
+                                   key_modifiers));
 }
 
 void RenderViewHostImpl::DragTargetDragOver(
     const gfx::Point& client_pt, const gfx::Point& screen_pt,

[sky, 2012/05/17 19:49:40]

wrap each param onto its own line.

[varunjain, 2012/05/17 21:14:04]

Done.

-    WebDragOperationsMask operations_allowed) {
+    WebDragOperationsMask operations_allowed,
+    int key_modifiers) {
   Send(new DragMsg_TargetDragOver(GetRoutingID(), client_pt, screen_pt,
-                                  operations_allowed));
+                                  operations_allowed, key_modifiers));
 }
 
 void RenderViewHostImpl::DragTargetDragLeave() {
   Send(new DragMsg_TargetDragLeave(GetRoutingID()));
 }
 
 void RenderViewHostImpl::DragTargetDrop(
-    const gfx::Point& client_pt, const gfx::Point& screen_pt) {
-  Send(new DragMsg_TargetDrop(GetRoutingID(), client_pt, screen_pt));
+    const gfx::Point& client_pt, const gfx::Point& screen_pt,
+    int key_modifiers) {

[sky, 2012/05/17 19:49:40]

wrap each param onto its own line.

[varunjain, 2012/05/17 21:14:04]

Done.

+  Send(new DragMsg_TargetDrop(GetRoutingID(), client_pt, screen_pt,
+                              key_modifiers));
 }
 
 void RenderViewHostImpl::DesktopNotificationPermissionRequestDone(
     int callback_context) {
   Send(new DesktopNotificationMsg_PermissionRequestDone(
       GetRoutingID(), callback_context));
 }
 
 void RenderViewHostImpl::DesktopNotificationPostDisplay(int callback_context) {
   Send(new DesktopNotificationMsg_PostDisplay(GetRoutingID(),
@@ skipping 1256 matching lines @@
   // can cause navigations to be ignored in OnMsgNavigate.
   is_waiting_for_beforeunload_ack_ = false;
   is_waiting_for_unload_ack_ = false;
 }
 
 void RenderViewHostImpl::ClearPowerSaveBlockers() {
   STLDeleteValues(&power_save_blockers_);
 }
 
 }  // namespace content