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

Unified Diff: chrome/browser/ui/autofill/autofill_popup_controller_impl.h

Issue 14053015: Don't add subtext padding if there is no subpadding. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 7 years, 8 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_impl.h
diff --git a/chrome/browser/ui/autofill/autofill_popup_controller_impl.h b/chrome/browser/ui/autofill/autofill_popup_controller_impl.h
index fc90f185daf44721381a4e53816d0b008b778b09..fc585058c95362a30e560aa782ad6350fb5c8a30 100644
--- a/chrome/browser/ui/autofill/autofill_popup_controller_impl.h
+++ b/chrome/browser/ui/autofill/autofill_popup_controller_impl.h
@@ -132,6 +132,11 @@ class AutofillPopupControllerImpl : public AutofillPopupController,
// Calculates the desired height of the popup based on its contents.
int GetDesiredPopupHeight() const;
+
+ // Calculate the width of the row, excluding all the text. This provides
+ // the size of the row that won't be reducible (since all the text can be
+ // elided if there isn't enough space).
+ int RowWidthWithoutText(int row) const;
#endif
base::WeakPtr<AutofillPopupControllerImpl> GetWeakPtr();
@@ -139,11 +144,6 @@ class AutofillPopupControllerImpl : public AutofillPopupController,
private:
const gfx::Rect RoundedElementBounds() const;
#if !defined(OS_ANDROID)
- // Calculate the width of the row, excluding all the text. This provides
- // the size of the row that won't be reducible (since all the text can be
- // elided if there isn't enough space).
- int RowWidthWithoutText(int row) const;
-
// Calculates and sets the bounds of the popup, including placing it properly
// to prevent it from going off the screen.
void UpdatePopupBounds();

Powered by Google App Engine
This is Rietveld 408576698