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

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

Issue 12188020: Adding the page and DPI scale adjustment for Autofill Popups. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixing Autofill unittests Created 7 years, 10 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_controller_unittest.cc
diff --git a/chrome/browser/ui/autofill/autofill_popup_controller_unittest.cc b/chrome/browser/ui/autofill/autofill_popup_controller_unittest.cc
index 3a43b3e0e684d512bccf366a9165e0d02c32418f..dcf9e6fa5573cb11f176b8ce63c5ef047e07d3e9 100644
--- a/chrome/browser/ui/autofill/autofill_popup_controller_unittest.cc
+++ b/chrome/browser/ui/autofill/autofill_popup_controller_unittest.cc
@@ -76,8 +76,8 @@ class TestAutofillPopupController : public AutofillPopupControllerImpl {
const gfx::Rect& popup_bounds() const {
return AutofillPopupControllerImpl::popup_bounds();
}
- const gfx::Rect& element_bounds() const {
- return AutofillPopupControllerImpl::element_bounds();
+ const gfx::Rect rounded_element_bounds() const {
+ return AutofillPopupControllerImpl::rounded_element_bounds();
}
#if !defined(OS_ANDROID)
const gfx::Font& name_font() const {
@@ -289,7 +289,7 @@ TEST_F(AutofillPopupControllerUnitTest, GetOrCreate) {
bounds);
EXPECT_EQ(
bounds,
- static_cast<AutofillPopupController*>(controller3)->element_bounds());
+ static_cast<AutofillPopupController*>(controller3)->rounded_element_bounds());
controller3->Hide();
EXPECT_CALL(delegate, ControllerDestroyed());

Powered by Google App Engine
This is Rietveld 408576698