MacViews: Remove aura dependency from menu_test_base

MacViews: Remove aura dependency from menu_test_base

The ui_controls_aura implementations either dispatch events via
aura::Window::GetHost() (which first does GetRootWindow()), find the
root window independently, or else ignore the window argument. So the
GetRootWindow call is not required here.

BUG=399191
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=287708

[tapted, 2014-08-05 13:46:18]

Hi sky, a small menu CL if you have time - please take a look.

[sky, 2014-08-05 17:18:33]

https://codereview.chromium.org/442633003/diff/1/chrome/browser/ui/views/menu...
File chrome/browser/ui/views/menu_test_base.cc (right):

https://codereview.chromium.org/442633003/diff/1/chrome/browser/ui/views/menu...
chrome/browser/ui/views/menu_test_base.cc:35: GetWidget()->GetNativeWindow(),
keycode, false, false,
This isn't necessarily the same thing. Does
ui_controls::SendKeyPressNotifyWhenDone figure out the root correctly?

[tapted, 2014-08-05 23:04:22]

https://codereview.chromium.org/442633003/diff/1/chrome/browser/ui/views/menu...
File chrome/browser/ui/views/menu_test_base.cc (right):

https://codereview.chromium.org/442633003/diff/1/chrome/browser/ui/views/menu...
chrome/browser/ui/views/menu_test_base.cc:35: GetWidget()->GetNativeWindow(),
keycode, false, false,
On 2014/08/05 17:18:32, sky wrote:
> This isn't necessarily the same thing. Does
> ui_controls::SendKeyPressNotifyWhenDone figure out the root correctly?

Yeah - I checked they all amount to the same thing:

aurawin first does HWND = window->GetHost() (and the first thing
aura::window::GetHost does is GetRootWindow).

aurax11 ignores the window argument. 

desktop_aurax11 also first does aura::window::GetHost()

factory_ash actually explicitly first calls window->GetRootWindow() as the first
thing

ozone does NOTIMPLEMENTED()

I think that's all of them :)

[sky, 2014-08-06 00:29:06]

LGTM then

[tapted, 2014-08-06 00:53:37]

The CQ bit was checked by tapted@chromium.org

[commit-bot: I haz the power, 2014-08-06 00:56:21]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tapted@chromium.org/442633003/1

[commit-bot: I haz the power, 2014-08-06 01:59:45]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_aosp on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/bu...)

[commit-bot: I haz the power, 2014-08-06 06:30:03]

Change committed as 287708