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

Unified Diff: chrome/browser/ui/cocoa/autofill/autofill_section_container.mm

Issue 21724002: rAc: try really hard not to ellipsize addresses (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sync'd Created 7 years, 4 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/cocoa/autofill/autofill_section_container.mm
diff --git a/chrome/browser/ui/cocoa/autofill/autofill_section_container.mm b/chrome/browser/ui/cocoa/autofill/autofill_section_container.mm
index 427f4ac92c47576191458898444d529b184beccd..dc6a968015054388454e1bc05b89bd8635ecf056 100644
--- a/chrome/browser/ui/cocoa/autofill/autofill_section_container.mm
+++ b/chrome/browser/ui/cocoa/autofill/autofill_section_container.mm
@@ -283,20 +283,18 @@ bool CompareInputRows(const autofill::DetailInput* input1,
- (void)updateSuggestionState {
const autofill::SuggestionState& suggestionState =
delegate_->SuggestionStateForSection(section_);
- bool showSuggestions = !suggestionState.text.empty();
+ // TODO(estade): use |vertically_compact_text| when it fits.
+ const base::string16& text = suggestionState.horizontally_compact_text;
+ bool showSuggestions = suggestionState.visible;
[[suggestContainer_ view] setHidden:!showSuggestions];
[inputs_ setHidden:showSuggestions];
- string16 line1;
- string16 line2;
- BreakSuggestionText(suggestionState.text, &line1, &line2);
- ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
- gfx::Font font = rb.GetFont(ui::ResourceBundle::BaseFont).DeriveFont(
- 0, suggestionState.text_style);
+ base::string16 line1;
+ base::string16 line2;
+ BreakSuggestionText(text, &line1, &line2);
[suggestContainer_ setSuggestionText:base::SysUTF16ToNSString(line1)
- line2:base::SysUTF16ToNSString(line2)
- withFont:font.GetNativeFont()];
+ line2:base::SysUTF16ToNSString(line2)];
[suggestContainer_ setIcon:suggestionState.icon.AsNSImage()];
if (!suggestionState.extra_text.empty()) {
NSString* extraText =

Powered by Google App Engine
This is Rietveld 408576698