In dialogs & dialog-like bubbles, make the escape button just close

ui/views/window/dialog_client_view.cc

Index: ui/views/window/dialog_client_view.cc
diff --git a/ui/views/window/dialog_client_view.cc b/ui/views/window/dialog_client_view.cc
index eee682637d7989c4cf37fec65aafa7c347b8b6cd..358c81c39120b8afa346769ccd0d77a8c3ee1bc2 100644
--- a/ui/views/window/dialog_client_view.cc
+++ b/ui/views/window/dialog_client_view.cc
@@ -215,10 +215,6 @@ void DialogClientView::Layout() {
 bool DialogClientView::AcceleratorPressed(const ui::Accelerator& accelerator) {
   DCHECK_EQ(accelerator.key_code(), ui::VKEY_ESCAPE);
 
-  // If there's a cancel button, it handles escape.

[msw, 2016/05/10 18:23:32]

I like this behavior with the bookmark bubble, and I'm trying to think of other
cases where it might be nice to cancel (not just close) on Esc. It'd be really
nice to have a table listing dialogs where cancel isn't equivalent to close, and
listing the respective actions... can you find just a couple more examples?

-  if (cancel_button_)
-    return cancel_button_->AcceleratorPressed(accelerator);
-
   GetWidget()->Close();
   return true;
 }