Refactor menu controller to isolate aura dependency.

ui/views/widget/native_widget_aura.cc

Index: ui/views/widget/native_widget_aura.cc
diff --git a/ui/views/widget/native_widget_aura.cc b/ui/views/widget/native_widget_aura.cc
index f428259e841da0f65f44462004cb82b306ce2bb2..57b04f06e46db41f551963cedacaaf7278d87382 100644
--- a/ui/views/widget/native_widget_aura.cc
+++ b/ui/views/widget/native_widget_aura.cc
@@ -34,6 +34,7 @@
 #include "ui/views/widget/widget_aura_utils.h"
 #include "ui/views/widget/widget_delegate.h"
 #include "ui/views/widget/window_reorderer.h"
+#include "ui/wm/core/shadow_types.h"
 #include "ui/wm/core/window_util.h"
 #include "ui/wm/public/activation_client.h"
 #include "ui/wm/public/drag_drop_client.h"
@@ -176,6 +177,9 @@ void NativeWidgetAura::InitNativeWidget(const Widget::InitParams& params) {
 
   window_reorderer_.reset(new WindowReorderer(window_,
       GetWidget()->GetRootView()));
+
+  if (!params.has_dropshadow)

[tapted, 2014/05/04 23:43:39]

This might be a different kind of shadow. I think the wm::ShadowType is the
thing CrOS (and Mac!) do when a window gains focus -- it's a soft shadow around
all sides that gets darker when it's the main window. Whereas the drop shadow is
a hard/thin/projected shadow that's just on the right and bottom of the window
(and menus actually have them :).

.. or at least that's the conclusion I made when I was poking at this
dependency.

[Andre, 2014/05/05 22:41:29]

OK, I added Widget::SetHasShadow to replace this.

+    SetShadowType(window_, wm::SHADOW_TYPE_NONE);
 }
 
 NonClientFrameView* NativeWidgetAura::CreateNonClientFrameView() {