autofill names - dont parse when calling SetRawInfo(FULL_NAME)

components/autofill/core/browser/address.cc

Index: components/autofill/core/browser/address.cc
diff --git a/components/autofill/core/browser/address.cc b/components/autofill/core/browser/address.cc
index fc2e5a3c6ffc3429eb05d4d6e3e04d0465c3c7ff..4b977bfbe45a63a3379971e0d0cce13470d8256d 100644
--- a/components/autofill/core/browser/address.cc
+++ b/components/autofill/core/browser/address.cc
@@ -172,19 +172,9 @@ bool Address::SetInfo(const AutofillType& type,
     return !country_code_.empty();
   }
 
-  // If the address doesn't have any newlines, don't attempt to parse it into
-  // lines, since this is potentially a user-entered address in the user's own
-  // format, so the code would have to rely on iffy heuristics at best.
-  // Instead, just give up when importing addresses like this.
-  if (storable_type == ADDRESS_HOME_STREET_ADDRESS && !value.empty() &&

[Evan Stade, 2014/06/28 01:01:35]

this behavior was causing problems when I was trying to call SetInfo for data
entered via requestAutocomplete. I decided to delete this code because I don't
really agree with the logic behind it. "123 Main St, Apt 7" is a perfectly way
to type an address and doesn't need to be broken up into multiple address lines.

-      value.find(base::char16('\n')) == base::string16::npos) {
-    street_address_.clear();
-    return false;
-  }
-
   SetRawInfo(storable_type, value);
 
-  // Likewise, give up when importing addresses with any entirely blank lines.
+  // Give up when importing addresses with any entirely blank lines.
   // There's a good chance that this formatting is not intentional, but it's
   // also not obviously safe to just strip the newlines.
   if (storable_type == ADDRESS_HOME_STREET_ADDRESS &&