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

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

Issue 85863003: Change DetailOutputMap to FieldValueMap (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 7 years, 1 month 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 24ea4616ccfb50b4beab99c8b6edeb94864c04f4..16a9c33adb076293c0c705df7ba21d4e43cb7e4a 100644
--- a/chrome/browser/ui/cocoa/autofill/autofill_section_container.mm
+++ b/chrome/browser/ui/cocoa/autofill/autofill_section_container.mm
@@ -90,10 +90,10 @@ bool CompareInputRows(const autofill::DetailInput* input1,
- (const autofill::DetailInput*)detailInputForType:
(autofill::ServerFieldType)type;
-// Takes an NSArray of controls and builds a DetailOutputMap from them.
+// Takes an NSArray of controls and builds a FieldValueMap from them.
// Translates between Cocoa code and delegate, essentially.
// All controls must inherit from NSControl and conform to AutofillInputView.
-- (void)fillDetailOutputs:(autofill::DetailOutputMap*)outputs
+- (void)fillDetailOutputs:(autofill::FieldValueMap*)outputs
fromControls:(NSArray*)controls;
// Updates input fields based on delegate status. If |shouldClobber| is YES,
@@ -132,7 +132,7 @@ bool CompareInputRows(const autofill::DetailInput* input1,
return self;
}
-- (void)getInputs:(autofill::DetailOutputMap*)output {
+- (void)getInputs:(autofill::FieldValueMap*)output {
[self fillDetailOutputs:output fromControls:[inputs_ subviews]];
}
@@ -341,15 +341,14 @@ bool CompareInputRows(const autofill::DetailInput* input1,
return [field isEnabled];
}]];
- autofill::DetailOutputMap detailOutputs;
+ autofill::FieldValueMap detailOutputs;
[self fillDetailOutputs:&detailOutputs fromControls:fields];
autofill::ValidityMessages messages = delegate_->InputsAreValid(
section_, detailOutputs);
for (NSControl<AutofillInputField>* input in fields) {
- const autofill::ServerFieldType type = [self fieldTypeForControl:input];
const autofill::ValidityMessage& message =
- messages.GetMessageOrDefault(type);
+ messages.GetMessageOrDefault([self fieldTypeForControl:input]);
if (validationType != autofill::VALIDATE_FINAL && !message.sure)
continue;
[input setValidityMessage:base::SysUTF16ToNSString(message.text)];
@@ -439,16 +438,14 @@ bool CompareInputRows(const autofill::DetailInput* input1,
return NULL;
}
-- (void)fillDetailOutputs:(autofill::DetailOutputMap*)outputs
+- (void)fillDetailOutputs:(autofill::FieldValueMap*)outputs
fromControls:(NSArray*)controls {
for (NSControl<AutofillInputField>* input in controls) {
DCHECK([input isKindOfClass:[NSControl class]]);
DCHECK([input conformsToProtocol:@protocol(AutofillInputField)]);
- autofill::ServerFieldType fieldType = [self fieldTypeForControl:input];
- DCHECK([self detailInputForType:fieldType]);
- NSString* value = [input fieldValue];
- outputs->insert(std::make_pair([self detailInputForType:fieldType],
- base::SysNSStringToUTF16(value)));
+ outputs->insert(std::make_pair(
+ [self fieldTypeForControl:input],
+ base::SysNSStringToUTF16([input fieldValue])));
}
}

Powered by Google App Engine
This is Rietveld 408576698