MacViews: Fix gn check fail for mac_views_browser on Aura dependencies.

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/logging.h"
 #include "base/trace_event/trace_event.h"
 #include "build/build_config.h"
-#include "ui/aura/window_observer.h"
 #include "ui/events/gestures/gesture_recognizer.h"
 #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"
 
-#if !defined(OS_MACOSX)
-#include "ui/aura/window.h"
+#if defined(USE_AURA) && !defined(OS_MACOSX)
+#include "ui/aura/window.h"           // nogncheck
+#include "ui/aura/window_observer.h"  // nogncheck
 #endif
 
 namespace views {
 
 namespace internal {
 
 #if !defined(OS_MACOSX)
 // This class adds itself as the pre target handler for the |window|
 // passed in. It currently handles touch events and forwards them to the
 // controller. Reason for this approach is views does not get raw touch
 // events which we need to determine if a touch happened outside the bounds
 // of the menu.
 class PreMenuEventDispatchHandler : public ui::EventHandler,

[tapted, 2016/09/19 03:23:36]

This should be moved to a file like
ui/views/controls/menu/pre_menu_event_dispatch_handler_aura.cc

                                     aura::WindowObserver {
  public:
   PreMenuEventDispatchHandler(const MenuController* controller,
                               SubmenuView* submenu,
                               aura::Window* window)
       : menu_controller_(const_cast<MenuController*>(controller)),
         submenu_(submenu),
         window_(window) {
     window_->AddPreTargetHandler(this);
     window_->AddObserver(this);
@@ skipping 190 matching lines @@
   }
   menu_controller->OnDragComplete(should_close);
 
   // We may have lost capture in the drag and drop, but are remaining open.
   // Return capture so we get MouseCaptureLost events.
   if (!should_close)
     native_widget_private()->SetCapture();
 }
 
 }  // namespace views