MacViews: Fix failing Menu Runner Cocoa Tests.

ui/views/controls/menu/menu_runner_cocoa_unittest.mm

 // Copyright 2014 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.
 
 #import "ui/views/controls/menu/menu_runner_impl_cocoa.h"
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/macros.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 111 matching lines @@
     MenuRunner::RunResult result = runner_->RunMenuAt(
         parent_, nullptr, anchor, MENU_ANCHOR_TOPLEFT, MenuRunner::COMBOBOX);
 
     // Ensure the anchor view is removed.
     EXPECT_EQ(1u, [[parent_->GetNativeView() subviews] count]);
     return result;
   }
 
   void MenuCancelCallback() {
     runner_->Cancel();
-    EXPECT_FALSE(runner_->IsRunning());
+    // For a syncronous menu, MenuRunner::IsRunning() should return true
+    // immediately after MenuRunner::Cancel() since the menu message loop has
+    // not yet terminated. It has only been marked for termination.
+    EXPECT_TRUE(runner_->IsRunning());
   }
 
   void MenuDeleteCallback() {
     runner_->Release();
     runner_ = nullptr;
   }
 
   void MenuCancelAndDeleteCallback() {
     runner_->Cancel();
     runner_->Release();
@@ skipping 111 matching lines @@
   // In RTL, Cocoa messes up the positioning unless the anchor rectangle is
   // offset to the right of the view. The offset for the checkmark is also
   // skipped, to give a better match to native behavior.
   base::i18n::SetICUDefaultLocale("he");
   RunMenuAt(anchor_rect);
   EXPECT_EQ(combobox_rect.right(), last_anchor_frame_.origin.x);
 }
 
 }  // namespace test
 }  // namespace views