Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(794)

Unified Diff: chrome/browser/ui/autofill/autofill_popup_layout_model_unittest.cc

Issue 2971783002: Skeleton for showing "Show all saved passwords row" for Linux/CrOs/Windows platforms (Closed)
Patch Set: . Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/autofill/autofill_popup_layout_model_unittest.cc
diff --git a/chrome/browser/ui/autofill/autofill_popup_layout_model_unittest.cc b/chrome/browser/ui/autofill/autofill_popup_layout_model_unittest.cc
index 4242fc2b8706f9a6f456de4bb588ea5b572d35f7..2e19991b68c6a3076d27d93b31eeb473281bba97 100644
--- a/chrome/browser/ui/autofill/autofill_popup_layout_model_unittest.cc
+++ b/chrome/browser/ui/autofill/autofill_popup_layout_model_unittest.cc
@@ -95,24 +95,24 @@ TEST_F(AutofillPopupLayoutModelTest, RowWidthWithoutText) {
// Refer to GetSuggestions() in TestAutofillPopupViewDelegate.
EXPECT_EQ(base_size,
- layout_model()->RowWidthWithoutText(0, /* with_label= */ false));
+ layout_model()->RowWidthWithoutText(0, /* has_substext= */ false));
EXPECT_EQ(base_size + subtext_increase,
- layout_model()->RowWidthWithoutText(1, /* with_label= */ true));
+ layout_model()->RowWidthWithoutText(1, /* has_substext= */ true));
EXPECT_EQ(base_size + AutofillPopupLayoutModel::kIconPadding +
ui::ResourceBundle::GetSharedInstance()
.GetImageNamed(IDR_AUTOFILL_CC_AMEX)
.Width(),
- layout_model()->RowWidthWithoutText(2, /* with_label= */ false));
+ layout_model()->RowWidthWithoutText(2, /* has_substext= */ false));
EXPECT_EQ(base_size + subtext_increase +
AutofillPopupLayoutModel::kIconPadding +
ui::ResourceBundle::GetSharedInstance()
.GetImageNamed(IDR_AUTOFILL_CC_GENERIC)
.Width(),
- layout_model()->RowWidthWithoutText(3, /* with_label= */ true));
+ layout_model()->RowWidthWithoutText(3, /* has_substext= */ true));
EXPECT_EQ(base_size + AutofillPopupLayoutModel::kHttpWarningNamePadding +
- AutofillPopupLayoutModel::kHttpWarningIconPadding +
+ AutofillPopupLayoutModel::kPaddingAfterLeadingIcon +
layout_model()->GetIconImage(4).width(),
- layout_model()->RowWidthWithoutText(4, /* with_label= */ true));
+ layout_model()->RowWidthWithoutText(4, /* has_substext= */ true));
}
#endif
« no previous file with comments | « chrome/browser/ui/autofill/autofill_popup_layout_model.cc ('k') | chrome/browser/ui/autofill/chrome_autofill_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698