Quick fix for crash during phone parsing.

This CL fixes out of range access, when by some reasons during the
phone parsing, libphonenumber returns that national code is longer that
the parsed string. Probably the reasons are some incorrect treating of
non-ASCII characters.

Bug: 895578
Change-Id: I2f0a7622453191778907a8f54e74e35a1484765b
Reviewed-on: https://chromium-review.googlesource.com/c/1286136
Commit-Queue: Vadym Doroshenko <dvadym@chromium.org>
Reviewed-by: Vaclav Brozek <vabr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#600366}

components/autofill/core/browser/phone_number_i18n.cc

@@ -166,6 +166,13 @@ bool ParsePhoneNumber(const base::string16& value,
   if (destination_length > area_length)
     area_length = destination_length;
 
+  if (area_length >= static_cast<int>(national_significant_number.size())) {
+    // For some non-ASCII strings |destination_length| is bigger than phone
+    // string size. It might be because of incorrect treating of non-ASCII
+    // characters.
+    return false;
+  }
+
   std::string area_code;
   std::string subscriber_number;
   if (area_length > 0) {