[Web Payments] Implement proper focusing in Payment Request

chrome/browser/ui/views/payments/payment_request_sheet_controller.cc

 // Copyright 2017 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/views/payments/payment_request_sheet_controller.h"
 
 #include "chrome/browser/ui/views/payments/payment_request_dialog_view.h"
 #include "chrome/browser/ui/views/payments/payment_request_dialog_view_ids.h"
 #include "chrome/browser/ui/views/payments/payment_request_views_util.h"
 #include "components/payments/content/payment_request.h"
 #include "components/strings/grit/components_strings.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/views/background.h"
 #include "ui/views/controls/button/md_text_button.h"
 #include "ui/views/controls/scroll_view.h"
+#include "ui/views/focus/focus_search.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/fill_layout.h"
 #include "ui/views/layout/grid_layout.h"
 
 namespace payments {
 
+namespace {
+
+class SheetView : public views::View, public views::FocusTraversable {

[Mathieu, 2017/05/02 13:16:10]

If you die, we have no idea what this does (please don't die!). Let's add a
class comment to describe why this is needed at all, capturing all your research
into this.

I just re-read the code a couple of times, and the comment from
SetFirstFocusableView could be brought up to the class level because this is the
whole point of the class. 

[anthonyvd, 2017/05/02 13:43:32]

Done.

+ public:
+  SheetView()
+      : first_focusable_(nullptr),
+        focus_search_(base::MakeUnique<views::FocusSearch>(this, true, false)) {
+  }
+
+  // Sets |view| as the first focusable view in this pane. If it's nullptr, the
+  // fallback is to use focus_search_ to find the first focusable view.
+  void SetFirstFocusableView(views::View* view) { first_focusable_ = view; }
+
+ private:
+  // views::FocusTraversable:
+  views::FocusSearch* GetFocusSearch() override { return focus_search_.get(); }
+
+  views::FocusTraversable* GetFocusTraversableParent() override {
+    return parent()->GetFocusTraversable();
+  }
+
+  views::View* GetFocusTraversableParentView() override { return this; }
+
+  // views::View:
+  views::FocusTraversable* GetPaneFocusTraversable() override { return this; }
+
+  void RequestFocus() override {
+    views::View* first_focusable = first_focusable_;
+
+    if (!first_focusable) {
+      views::FocusTraversable* dummy_focus_traversable;
+      views::View* dummy_focus_traversable_view;
+      first_focusable = focus_search_->FindNextFocusableView(
+          nullptr, false, views::FocusSearch::DOWN, false,
+          &dummy_focus_traversable, &dummy_focus_traversable_view);
+    }
+
+    if (first_focusable)
+      first_focusable->RequestFocus();
+  }
+
+  views::View* first_focusable_;
+  std::unique_ptr<views::FocusSearch> focus_search_;
+
+  DISALLOW_COPY_AND_ASSIGN(SheetView);
+};
+
+}  // namespace
+
 PaymentRequestSheetController::PaymentRequestSheetController(
     PaymentRequestSpec* spec,
     PaymentRequestState* state,
     PaymentRequestDialogView* dialog)
     : spec_(spec), state_(state), dialog_(dialog) {}
 
 PaymentRequestSheetController::~PaymentRequestSheetController() {}
 
 std::unique_ptr<views::View> PaymentRequestSheetController::CreateView() {
   std::unique_ptr<views::View> view = CreatePaymentView();
@@ skipping 42 matching lines @@
       dialog()->Pay();
       break;
     case PaymentRequestCommonTags::PAYMENT_REQUEST_COMMON_TAG_MAX:
       NOTREACHED();
       break;
   }
 }
 
 std::unique_ptr<views::View>
 PaymentRequestSheetController::CreatePaymentView() {
-  std::unique_ptr<views::View> view = base::MakeUnique<views::View>();
+  std::unique_ptr<SheetView> view = base::MakeUnique<SheetView>();
   view->set_background(views::Background::CreateSolidBackground(SK_ColorWHITE));
 
   // Paint the sheets to layers, otherwise the MD buttons (which do paint to a
   // layer) won't do proper clipping.
   view->SetPaintToLayer();
 
   views::GridLayout* layout = new views::GridLayout(view.get());
   view->SetLayoutManager(layout);
 
   // Note: each view is responsible for its own padding (insets).
@@ skipping 28 matching lines @@
   scroll_ = base::MakeUnique<views::ScrollView>();
   scroll_->set_owned_by_client();
   scroll_->EnableViewPortLayer();
   scroll_->set_hide_horizontal_scrollbar(true);
   scroll_->SetContents(pane_);
   layout->AddView(scroll_.get());
 
   layout->StartRow(0, 0);
   layout->AddView(CreateFooterView().release());
 
+  view->SetFirstFocusableView(GetFirstFocusedView());
+
   return view;
 }
 
 std::unique_ptr<views::View> PaymentRequestSheetController::CreateFooterView() {
   std::unique_ptr<views::View> container = base::MakeUnique<views::View>();
 
   views::GridLayout* layout = new views::GridLayout(container.get());
   container->SetLayoutManager(layout);
 
   views::ColumnSet* columns = layout->AddColumnSet(0);
@@ skipping 17 matching lines @@
     layout->AddView(extra_view.release());
   else
     layout->SkipColumns(1);
 
   std::unique_ptr<views::View> trailing_buttons_container =
       base::MakeUnique<views::View>();
 
   trailing_buttons_container->SetLayoutManager(new views::BoxLayout(
       views::BoxLayout::kHorizontal, 0, 0, kPaymentRequestButtonSpacing));
 
-  std::unique_ptr<views::Button> primary_button = CreatePrimaryButton();
-  if (primary_button)
-    trailing_buttons_container->AddChildView(primary_button.release());
+  primary_button_ = CreatePrimaryButton();
+  if (primary_button_) {
+    primary_button_->set_owned_by_client();
+    trailing_buttons_container->AddChildView(primary_button_.get());
+  }
 
-  views::LabelButton* button = views::MdTextButton::CreateSecondaryUiButton(
-      this, GetSecondaryButtonLabel());
-  button->set_tag(static_cast<int>(PaymentRequestCommonTags::CLOSE_BUTTON_TAG));
-  button->set_id(static_cast<int>(DialogViewID::CANCEL_BUTTON));
-  trailing_buttons_container->AddChildView(button);
+  secondary_button_ = std::unique_ptr<views::Button>(
+      views::MdTextButton::CreateSecondaryUiButton(this,
+                                                   GetSecondaryButtonLabel()));
+  secondary_button_->set_owned_by_client();
+  secondary_button_->set_tag(
+      static_cast<int>(PaymentRequestCommonTags::CLOSE_BUTTON_TAG));
+  secondary_button_->set_id(static_cast<int>(DialogViewID::CANCEL_BUTTON));
+  trailing_buttons_container->AddChildView(secondary_button_.get());
 
   layout->AddView(trailing_buttons_container.release());
 
   return container;
 }
 
+views::View* PaymentRequestSheetController::GetFirstFocusedView() {
+  if (primary_button_ && primary_button_->enabled())
+    return primary_button_.get();
+
+  DCHECK(secondary_button_);
+
+  return secondary_button_.get();
+}
+
 }  // namespace payments