The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/aa336ae9f9b653825af2c70bdf59593fd58e886a

commit aa336ae9f9b653825af2c70bdf59593fd58e886a
Author: karandeepb <karandeepb@chromium.org>
Date: Mon Apr 11 01:20:33 2016

MacViews: Fix MenuRunnerTests which fail with asan enabled.

This CL fixes the following tests which fail due to use-after-free errors when
asan is enabled -
  -MenuRunnerTest.AsynchronousKeyEventHandling
  -MenuRunnerTest.LatinMnemonic
  -MenuRunnerTest.NonLatinMnemonic

These fail on the events dispatched by ui::test::EventGenerator. The reason for
the failures is that the call to TerminateNestedMessageLoopIfNecessary() in
MenuController::OnWillDispatchKeyEvent is unsafe, since the MenuController might
by deleted by the calls to SelectByChar(..) or OnKeyDown(..). These failures are
not experienced on platforms other than Mac, due to the differences in event
processing pipelines.

This CL fixes the tests by checking that the current MenuController instance is
alive before calling TerminateNestedMessageLoopIfNecessary().

BUG= 601315 

Review URL: https://codereview.chromium.org/1863393006

Cr-Commit-Position: refs/heads/master@{#386323}

[modify] https://crrev.com/aa336ae9f9b653825af2c70bdf59593fd58e886a/ui/views/controls/menu/menu_controller.cc