MacViews: Fix accelerator handling while Omnibox is in focus.

ui/views/controls/textfield/textfield_unittest.cc

 // 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.
 
 #include "ui/views/controls/textfield/textfield.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <set>
@@ skipping 1234 matching lines @@
   EXPECT_TRUE(textfield_->key_handled());
   textfield_->clear();
 
   SendEndEvent(shift);
   EXPECT_TRUE(textfield_->key_received());
   EXPECT_TRUE(textfield_->key_handled());
   textfield_->clear();
 
   // F20 key won't be handled.
   SendKeyEvent(ui::VKEY_F20);
-#if defined(OS_MACOSX)
-  // On Mac, key combinations that don't map to editing commands are forwarded
-  // on to the next responder, usually ending up at the window, which will beep.

[tapted, 2016/11/23 02:09:30]

this is nice :)

-  EXPECT_FALSE(textfield_->key_received());

[themblsha, 2016/11/22 14:33:48]

The comment is outdated, as we try to DispatchKeyEvent using the InputMethod,
and key now is actually received.

-#else
   EXPECT_TRUE(textfield_->key_received());
-#endif
   EXPECT_FALSE(textfield_->key_handled());
   textfield_->clear();
 }
 
 // Tests that default key bindings are handled even with a delegate installed.
 TEST_F(TextfieldTest, OnKeyPressBinding) {
   InitTextfield();
 
 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
   // Install a TextEditKeyBindingsDelegateAuraLinux that does nothing.
@@ skipping 1736 matching lines @@
   ui::AXNodeData node_data_protected;
   node_data_protected.state = 0;
   textfield_->GetAccessibleNodeData(&node_data_protected);
   EXPECT_EQ(ui::AX_ROLE_TEXT_FIELD, node_data_protected.role);
   EXPECT_EQ(ASCIIToUTF16("********"),
             node_data_protected.GetString16Attribute(ui::AX_ATTR_VALUE));
   EXPECT_TRUE(node_data_protected.HasStateFlag(ui::AX_STATE_PROTECTED));
 }
 
 }  // namespace views