Change the maximum account chooser height to 3.5 rows on Views.

chrome/browser/ui/views/passwords/account_chooser_dialog_view.cc

 // Copyright 2015 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/passwords/account_chooser_dialog_view.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/passwords/manage_passwords_view_utils.h"
 #include "chrome/browser/ui/passwords/password_dialog_controller.h"
 #include "chrome/browser/ui/views/passwords/credentials_item_view.h"
 #include "chrome/grit/generated_resources.h"
 #include "components/autofill/core/common/password_form.h"
 #include "components/constrained_window/constrained_window_views.h"
 #include "content/public/browser/web_contents.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/strings/grit/ui_strings.h"
 #include "ui/views/border.h"
 #include "ui/views/controls/scroll_view.h"
 #include "ui/views/controls/styled_label.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/grid_layout.h"
 #include "ui/views/layout/layout_constants.h"
 #include "ui/views/widget/widget.h"
 
 namespace {
 
-// Maximum number of accounts displayed before vertical scrolling appears.
-const size_t kMaxAccounts = 3;
+// Maximum height of the credential list. The unit is one row's height.
+constexpr double kMaxHeightAccounts = 3.5;
 
-const int kVerticalAvatarMargin = 8;
+constexpr int kVerticalAvatarMargin = 8;
 
 // An identifier for views::ColumnSet.
 enum ColumnSetType {
   SINGLE_VIEW_COLUMN_SET,
   SINGLE_VIEW_COLUMN_SET_NO_PADDING,
 };
 
 // Construct a |type| ColumnSet and add it to |layout|.
 void BuildColumnSet(ColumnSetType type, views::GridLayout* layout) {
   views::ColumnSet* column_set = layout->AddColumnSet(type);
@@ skipping 38 matching lines @@
         button_listener, titles.first, titles.second, kButtonHoverColor,
         form.get(), request_context);
     credential_view->SetLowerLabelColor(kAutoSigninTextColor);
     credential_view->SetBorder(views::Border::CreateEmptyBorder(
         kVerticalAvatarMargin, views::kButtonHEdgeMarginNew,
         kVerticalAvatarMargin, views::kButtonHEdgeMarginNew));
     item_height = std::max(item_height, credential_view->GetPreferredHeight());
     list_view->AddChildView(credential_view);
   }
   views::ScrollView* scroll_view = new views::ScrollView;
-  scroll_view->ClipHeightTo(0, kMaxAccounts * item_height);
+  scroll_view->ClipHeightTo(0, kMaxHeightAccounts * item_height);
   scroll_view->SetContents(list_view);
   return scroll_view;
 }
 
 }  // namespace
 
 AccountChooserDialogView::AccountChooserDialogView(
     PasswordDialogController* controller,
     content::WebContents* web_contents)
     : controller_(controller),
@@ skipping 92 matching lines @@
       GetProfileFromWebContents(web_contents_)->GetRequestContext()));
   // DialogClientView adds kRelatedControlVerticalSpacing padding once more for
   // the buttons.
   layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing);
 }
 
 AccountChooserPrompt* CreateAccountChooserPromptView(
     PasswordDialogController* controller, content::WebContents* web_contents) {
   return new AccountChooserDialogView(controller, web_contents);
 }