views: don't send OnKeypressUnhandled for all keys

ui/views/controls/textfield/textfield.cc

Index: ui/views/controls/textfield/textfield.cc
diff --git a/ui/views/controls/textfield/textfield.cc b/ui/views/controls/textfield/textfield.cc
index 4642d23fc89a89633cc84ff72fbb5011959a7b9d..3eddb18f9b22e41073d740b8403cb3885c13f8d9 100644
--- a/ui/views/controls/textfield/textfield.cc
+++ b/ui/views/controls/textfield/textfield.cc
@@ -684,9 +684,6 @@ bool Textfield::OnKeyPressed(const ui::KeyEvent& event) {
     ExecuteTextEditCommand(edit_command);
     handled = true;
   }
-
-  if (!handled)

[sky, 2016/10/06 15:58:12]

Should this instead check handled || event.handled() ||
event.stopped_propagation()?

[Elly Fong-Jones, 2016/10/06 16:20:05]

I don't think so - for example, when you tab away from the textfield, the event
is handled further up the view hierarchy (as I understand it), so by the time
OnKeyPressed runs, the event hasn't been handled yet, but will be in future.

-    OnKeypressUnhandled();
   return handled;
 }