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 35 matching lines @@
 
 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());
+
+  // Registers 'Shift + Browser Back' accelerator to web dialog, which allows
+  // Chrome OS CFM remote controller using hangup button to close dialog.
+  accelerators_.push_back(
+      ui::Accelerator(ui::VKEY_BROWSER_BACK, ui::EF_SHIFT_DOWN));

[xiyuan, 2017/05/11 04:48:27]

Move this into GetAccelerators and get rid of |accelerators_| member.

[Qiang(Joe) Xu, 2017/05/12 21:50:45]

Done.

 }
 
 LoginWebDialog::~LoginWebDialog() {}
 
 void LoginWebDialog::Show() {
   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(

[sky, 2017/05/11 15:04:11]

Initialize dialog_window_ to null.

[Qiang(Joe) Xu, 2017/05/12 21:50:45]

Done.

         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() {
+  return accelerators_;
+}
+
+bool LoginWebDialog::AcceleratorPressed(const ui::Accelerator& accelerator) {
+  if (!dialog_window_)
+    return false;
+
+  if (accelerator.key_code() == ui::VKEY_BROWSER_BACK &&

[sky, 2017/05/11 15:04:11]

Have a function that you use to get the accelerator so that you don't need to
duplicate it here and line 64. I'm thinking of something like:

ui::Accelerator GetCloseAccelerator() {
  return ui::Accelerator...;
}

In an anonmyous namespace.

[Qiang(Joe) Xu, 2017/05/12 21:50:45]

Done.

+      accelerator.IsShiftDown()) {
+    views::Widget::GetWidgetForNativeWindow(dialog_window_)->Close();
+    return true;
+  }
+
+  return false;
+}
+
 }  // namespace chromeos