Handle Escape key on constrained window.

chrome/browser/ui/views/constrained_web_dialog_delegate_views.cc

 // Copyright (c) 2012 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 "chrome/browser/ui/webui/constrained_web_dialog_delegate_base.h"
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
 #include "chrome/browser/ui/views/constrained_window_views.h"
 #include "chrome/browser/ui/webui/web_dialog_delegate.h"
 #include "chrome/browser/ui/webui/web_dialog_ui.h"
+#include "content/public/browser/native_web_keyboard_event.h"
 #include "content/public/browser/web_contents.h"
+#include "ui/aura/event.h"
+#include "ui/aura/window.h"
 #include "ui/gfx/size.h"
 #include "ui/views/controls/webview/webview.h"
 #include "ui/views/view.h"
+#include "ui/views/widget/native_widget_aura.h"
 #include "ui/views/widget/widget_delegate.h"
 
 using content::WebContents;
 
 namespace {
 
 class ConstrainedWebDialogDelegateViews
     : public ConstrainedWebDialogDelegateBase {
  public:
   ConstrainedWebDialogDelegateViews(
@@ skipping 10 matching lines @@
     }
   }
 
   virtual ~ConstrainedWebDialogDelegateViews() {}
 
   // WebDialogWebContentsDelegate interface.
   virtual void CloseContents(WebContents* source) OVERRIDE {
     window()->CloseConstrainedWindow();
   }
 
+  // contents::WebContentsDelegate
+  virtual void HandleKeyboardEvent(
+      const content::NativeWebKeyboardEvent& event) OVERRIDE {
+#if defined(USE_AURA)
+    aura::KeyEvent aura_event(event.os_event->native_event(), false);
+    window()->GetNativeWindow()->delegate()->OnKeyEvent(&aura_event);
+#elif defined(OS_WIN)
+    // TODO(yoshiki): Fill this when this is necessary.
+    NOTIMPLEMENTED();
+#endif
+  }
+
  private:
   DISALLOW_COPY_AND_ASSIGN(ConstrainedWebDialogDelegateViews);
 };
 
 }  // namespace
 
 class ConstrainedWebDialogDelegateViewViews
     : public views::WebView,
       public ConstrainedWebDialogDelegate,
       public views::WidgetDelegate {
@@ skipping 46 matching lines @@
   }
   virtual string16 GetWindowTitle() const OVERRIDE {
     return impl_->closed_via_webui() ? string16() :
         GetWebDialogDelegate()->GetDialogTitle();
   }
   virtual views::View* GetContentsView() OVERRIDE {
     return this;
   }
 
   // views::WebView overrides.
+  virtual bool
+      AcceleratorPressed(const ui::Accelerator& accelerator) OVERRIDE {

[kochi, 2012/05/30 07:17:31]

style nit: newline after open paren.
virtual bool AcceleratorPressed(
    const ui::Accelerator& accelerator) OVERRIDE {

[yoshiki, 2012/05/30 08:43:15]

Done.

+    // Pressing ESC closes the dialog.
+    DCHECK_EQ(ui::VKEY_ESCAPE, accelerator.key_code());
+    if (GetWidget())
+      GetWidget()->Close();
+    return true;
+  };

[kochi, 2012/05/30 07:17:31]

nit: a newline after this line.

[yoshiki, 2012/05/30 08:43:15]

Other inherited methods are listed without blank line (above
views::WidgetDelegate interface methods). I think newline is not necessarily to
keep consistency.

On 2012/05/30 07:17:31, Takayoshi Kochi wrote:

   virtual gfx::Size GetPreferredSize() OVERRIDE {
     gfx::Size size;
     if (!impl_->closed_via_webui())
       GetWebDialogDelegate()->GetDialogSize(&size);
     return size;
   }
 
  private:
   scoped_ptr<ConstrainedWebDialogDelegateViews> impl_;
 
   DISALLOW_COPY_AND_ASSIGN(ConstrainedWebDialogDelegateViewViews);
 };
 
 ConstrainedWebDialogDelegateViewViews::ConstrainedWebDialogDelegateViewViews(
     Profile* profile,
     WebDialogDelegate* delegate,
     WebDialogWebContentsDelegate* tab_delegate)
     : views::WebView(profile),
       impl_(new ConstrainedWebDialogDelegateViews(profile,
                                                   delegate,
                                                   tab_delegate)) {
   SetWebContents(tab()->web_contents());
+
+  // Pressing the ESC key will close the dialog.

[kochi, 2012/05/30 07:17:31]

probably "the" is not necessary.
You can reuse the same phrase on line 131.

[yoshiki, 2012/05/30 08:43:15]

Done.

+  set_allow_accelerators(true);
+  AddAccelerator(ui::Accelerator(ui::VKEY_ESCAPE, ui::EF_NONE));

[kochi, 2012/05/30 07:17:31]

Don't we need to RemoveAccelerator()?
(I'm not sure)

[yoshiki, 2012/05/30 08:43:15]

I think it is not necessarily because WebDialogView class uses the same way but
has not RemoveAccelerator() (in chrome/browser/ui/views/web_dialog_view.cc).

On 2012/05/30 07:17:31, Takayoshi Kochi wrote:

 }
 
 ConstrainedWebDialogDelegateViewViews::~ConstrainedWebDialogDelegateViewViews() {
 }
 
 // static
 ConstrainedWebDialogDelegate*
     ConstrainedWebDialogUI::CreateConstrainedWebDialog(
         Profile* profile,
         WebDialogDelegate* delegate,
         WebDialogWebContentsDelegate* tab_delegate,
         TabContentsWrapper* container) {
   ConstrainedWebDialogDelegateViewViews* constrained_delegate =
       new ConstrainedWebDialogDelegateViewViews(profile, delegate, tab_delegate);
   ConstrainedWindow* constrained_window =
       new ConstrainedWindowViews(container, constrained_delegate);
   constrained_delegate->set_window(constrained_window);
   return constrained_delegate;
 }