cros: Bind hangup red button on remote controller to close web dialog

chrome/browser/chromeos/login/ui/login_web_dialog.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 "chrome/browser/chromeos/login/ui/login_web_dialog.h"
 
 #include <deque>
 
 #include "base/lazy_instance.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 18 matching lines @@
 const double kDefaultWidthRatio = 0.6;
 const double kDefaultHeightRatio = 0.6;
 
 // Default width/height ratio of minimal dialog size.
 const double kMinimumWidthRatio = 0.25;
 const double kMinimumHeightRatio = 0.25;
 
 base::LazyInstance<std::deque<WebContents*>>::DestructorAtExit
     g_web_contents_stack = LAZY_INSTANCE_INITIALIZER;
 
+// Returns the accelerator which is mapped as hangup button on Chrome OS CFM
+// remote controller to close the dialog.
+ui::Accelerator GetCloseAccelerator() {
+  return ui::Accelerator(ui::VKEY_BROWSER_BACK, ui::EF_SHIFT_DOWN);
+}
+
 }  // namespace
 
 ///////////////////////////////////////////////////////////////////////////////
 // LoginWebDialog, public:
 
 void LoginWebDialog::Delegate::OnDialogClosed() {
 }
 
 LoginWebDialog::LoginWebDialog(content::BrowserContext* browser_context,
                                Delegate* delegate,
                                gfx::NativeWindow parent_window,
                                const base::string16& title,
                                const GURL& url)
     : browser_context_(browser_context),
       parent_window_(parent_window),
       delegate_(delegate),
       title_(title),
-      url_(url),
-      is_open_(false) {
+      url_(url) {
   gfx::Rect screen_bounds(CalculateScreenBounds(gfx::Size()));
   width_ = static_cast<int>(kDefaultWidthRatio * screen_bounds.width());
   height_ = static_cast<int>(kDefaultHeightRatio * screen_bounds.height());
 }
 
 LoginWebDialog::~LoginWebDialog() {}
 
 void LoginWebDialog::Show() {
+  dialog_window_ = nullptr;
   if (parent_window_) {
-    chrome::ShowWebDialog(parent_window_, browser_context_, this);
+    dialog_window_ =
+        chrome::ShowWebDialog(parent_window_, browser_context_, this);
   } else {
-    chrome::ShowWebDialogInContainer(
+    dialog_window_ = chrome::ShowWebDialogInContainer(
         SystemTrayClient::GetDialogParentContainerId(), browser_context_, this);
   }
-  is_open_ = true;
 }
 
 void LoginWebDialog::SetDialogSize(int width, int height) {
   DCHECK_GE(width, 0);
   DCHECK_GE(height, 0);
   width_ = width;
   height_ = height;
 }
 
 void LoginWebDialog::SetDialogTitle(const base::string16& title) {
@@ skipping 38 matching lines @@
   auto& stack = g_web_contents_stack.Get();
   return stack.empty() ? nullptr : stack.front();
 }
 
 void LoginWebDialog::OnDialogShown(content::WebUI* webui,
                                    content::RenderViewHost* render_view_host) {
   g_web_contents_stack.Pointer()->push_front(webui->GetWebContents());
 }
 
 void LoginWebDialog::OnDialogClosed(const std::string& json_retval) {
-  is_open_ = false;
+  dialog_window_ = nullptr;
   if (delegate_)
     delegate_->OnDialogClosed();
   delete this;
 }
 
 void LoginWebDialog::OnCloseContents(WebContents* source,
                                      bool* out_close_dialog) {
   *out_close_dialog = true;
 
   if (GetCurrentWebContents() == source)
@@ skipping 21 matching lines @@
   // fire an auto-reload, which in turn leads to opening a new browser window,
   // so we must suppress it.
   // http://crbug.com/443096
   return (source && !chrome::FindBrowserWithWebContents(source));
 }
 
 bool LoginWebDialog::HandleShouldCreateWebContents() {
   return false;
 }
 
+std::vector<ui::Accelerator> LoginWebDialog::GetAccelerators() {
+  std::vector<ui::Accelerator> accelerators;
+  accelerators.push_back(GetCloseAccelerator());
+  return accelerators;

[xiyuan, 2017/05/12 22:29:57]

nit: return {GetCloseAccelerator()};

[Qiang(Joe) Xu, 2017/05/12 22:39:32]

Done.

+}
+
+bool LoginWebDialog::AcceleratorPressed(const ui::Accelerator& accelerator) {
+  if (!dialog_window_)
+    return false;
+
+  if (GetCloseAccelerator() == accelerator) {
+    views::Widget::GetWidgetForNativeWindow(dialog_window_)->Close();
+    return true;
+  }
+
+  return false;
+}
+
 }  // namespace chromeos