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

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

Issue 63053003: Ask libaddressinput for address components to use in requestAutocomplete(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge Created 6 years, 11 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 8557450ef6c25d0426130f226a837cc2f07d28c8..31d0bb8178683dcc85fee5c9924045ea032e7c38 100644
--- a/chrome/browser/ui/cocoa/autofill/autofill_section_container.mm
+++ b/chrome/browser/ui/cocoa/autofill/autofill_section_container.mm
@@ -621,7 +621,7 @@ bool ShouldOverwriteComboboxes(autofill::DialogSection section,
base::scoped_nsobject<AutofillTextField> field(
[[AutofillTextField alloc] init]);
[[field cell] setPlaceholderString:
- l10n_util::GetNSStringWithFixup(input.placeholder_text_rid)];
+ l10n_util::FixUpWindowsStyleLabel(input.placeholder_text)];
NSString* tooltipText =
base::SysUTF16ToNSString(delegate_->TooltipForField(input.type));
if ([tooltipText length] > 0) {
@@ -652,8 +652,10 @@ bool ShouldOverwriteComboboxes(autofill::DialogSection section,
}
layout->AddView(control);
- if (input.length == autofill::DetailInput::LONG)
+ if (input.length == autofill::DetailInput::LONG ||
+ input.length == autofill::DetailInput::SHORT_EOL) {
++column_set_id;
+ }
}
[self updateFieldIcons];

Powered by Google App Engine
This is Rietveld 408576698