MacViews: Adds NativeWidgetMac and gets views target compiling on Mac.

ui/views/controls/menu/menu_host.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/controls/menu/menu_host.h"
 
 #include "base/auto_reset.h"
 #include "base/debug/trace_event.h"
 #include "ui/events/gestures/gesture_recognizer.h"

[tapted, 2014/05/21 09:40:29]

would need to guard this too (but that breaks apply_patch right now)

 #include "ui/gfx/path.h"
 #include "ui/native_theme/native_theme.h"
 #include "ui/views/controls/menu/menu_controller.h"
 #include "ui/views/controls/menu/menu_host_root_view.h"
 #include "ui/views/controls/menu/menu_item_view.h"
 #include "ui/views/controls/menu/menu_scroll_view_container.h"
 #include "ui/views/controls/menu/submenu_view.h"
 #include "ui/views/round_rect_painter.h"
 #include "ui/views/widget/native_widget_private.h"
 #include "ui/views/widget/widget.h"
@@ skipping 45 matching lines @@
 }
 
 void MenuHost::ShowMenuHost(bool do_capture) {
   // Doing a capture may make us get capture lost. Ignore it while we're in the
   // process of showing.
   base::AutoReset<bool> reseter(&ignore_capture_lost_, true);
   ShowInactive();
   if (do_capture) {
     // Cancel existing touches, so we don't miss some touch release/cancel
     // events due to the menu taking capture.
+#ifdef USE_AURA
     ui::GestureRecognizer::Get()->TransferEventsTo(NULL, NULL);
+#endif

[tapted, 2014/05/21 09:40:29]

This is to resolve

Undefined symbols for architecture i386:
  "ui::GestureRecognizer::Get()", referenced from:
      views::MenuHost::ShowMenuHost(bool) in views.menu_host.o

I wonder if we can just move this into NativeWidgetAura::SetCapture(), or give
that an argument..

[Andre, 2014/05/21 17:13:31]

That symbol was defined the last time I tried to build views.
Looks like gesture_recognizer_impl.cc was changed to be excluded for !aura a few
days ago,
https://codereview.chromium.org/282593002/diff/120001/ui/events/events.gyp

I agree, NativeWidgetAura::SetCapture seems like a better home for this code.

[tapted, 2014/05/22 12:50:14]

I poked at this a bit. Tried out just compiling the gesture stuff again the way
it was last week (crrev/291373002). But there are patches in flight that suggest
more aura-specific stuff on the way. Touch input on desktop OSX isn't currently
a priority AFAIK, so an empty stub felt right for now.

     native_widget_private()->SetCapture();
   }
 }
 
 void MenuHost::HideMenuHost() {
   ignore_capture_lost_ = true;
   ReleaseMenuHostCapture();
   Hide();
   ignore_capture_lost_ = false;
 }
@@ skipping 45 matching lines @@
   if (destroying_)
     return;
 
   MenuController* menu_controller =
       submenu_->GetMenuItem()->GetMenuController();
   if (menu_controller && !menu_controller->drag_in_progress())
     menu_controller->CancelAll();
 }
 
 }  // namespace views