[Web Payments] Implement proper focusing in Payment Request

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

 // Copyright 2016 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_sheet_view_controller.h"
 
 #include <algorithm>
 #include <memory>
 #include <string>
 #include <utility>
@@ skipping 423 matching lines @@
 
 void PaymentSheetViewController::FillContentView(views::View* content_view) {
   views::GridLayout* layout = new views::GridLayout(content_view);
   content_view->SetLayoutManager(layout);
   views::ColumnSet* columns = layout->AddColumnSet(0);
   columns->AddColumn(views::GridLayout::FILL, views::GridLayout::CENTER, 1,
                      views::GridLayout::USE_PREF, 0, 0);
 
   // The shipping address and contact info rows are optional.
   layout->StartRow(0, 0);
-  layout->AddView(CreatePaymentSheetSummaryRow().release());
+  std::unique_ptr<views::View> summary_row = CreatePaymentSheetSummaryRow();

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

need this change?

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

Oops, not anymore.

+  layout->AddView(summary_row.release());
 
   if (spec()->request_shipping()) {
     layout->StartRow(0, 0);
     layout->AddView(CreateShippingRow().release());
     // It's possible for requestShipping to be true and for there to be no
     // shipping options yet (they will come in updateWith).
     // TODO(crbug.com/707353): Put a better placeholder row, instead of no row.
     std::unique_ptr<views::Button> shipping_row = CreateShippingOptionRow();
     if (shipping_row) {
       layout->StartRow(0, 0);
@@ skipping 401 matching lines @@
   } else {
     return builder
         .Id(DialogViewID::PAYMENT_SHEET_SHIPPING_OPTION_SECTION_BUTTON)
         .CreateWithButton(
             l10n_util::GetStringUTF16(IDS_PAYMENT_REQUEST_CHOOSE_AN_ADDRESS),
             l10n_util::GetStringUTF16(IDS_CHOOSE), /*button_enabled=*/false);
   }
 }
 
 }  // namespace payments