MacViews: Implemented Drag & Drop

ui/views/widget/drop_helper.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 "ui/views/widget/drop_helper.h"
 
 #include "build/build_config.h"
 #include "ui/base/dragdrop/drag_drop_types.h"
 #include "ui/views/view.h"
 #include "ui/views/widget/widget.h"
@@ skipping 45 matching lines @@
   if (!drop_view)
     return ui::DragDropTypes::DRAG_NONE;
 
   if (drag_operation == ui::DragDropTypes::DRAG_NONE) {
     drop_view->OnDragExited();
     return ui::DragDropTypes::DRAG_NONE;
   }
 
   gfx::Point view_location(root_view_location);
   View* root_view = drop_view->GetWidget()->GetRootView();
+  root_view = root_view_;

[tapted, 2016/05/24 08:06:02]

Ah, BridgedNativeWidgetTestBase does some weird stuff with the root view. It's
really for testing some low-level interactions, and it's tricky to plumb
together.

Modelling the tests off NativeWidgetMacTest from native_widget_mac_unittest.mm
instead should get more of the Widget stuff plumbed through correctly.

[spqchan, 2016/05/26 01:56:55]

Done.

+
   View::ConvertPointToTarget(root_view, drop_view, &view_location);
   ui::DropTargetEvent drop_event(data, view_location, view_location,
                                  drag_operation);
   return drop_view->OnPerformDrop(drop_event);
 }
 
 View* DropHelper::CalculateTargetView(
     const gfx::Point& root_view_location,
     const OSExchangeData& data,
     bool check_can_drop) {
@@ skipping 72 matching lines @@
                                   drag_operation);
   return target_view_->OnDragUpdated(enter_event);
 }
 
 void DropHelper::NotifyDragExit() {
   if (target_view_)
     target_view_->OnDragExited();
 }
 
 }  // namespace views