Removed visually non-distinct characters from password generator

This CL removes the following characters from generated passwords as they are
visually not easy to distinguish and users may still need to type a generated
password from time to time:

- l (lowercase L)
- I (capital i)
- 1 (one)
- O (capital o)
- 0 (zero)
- o (lowercase O)

Bug: 846694
Change-Id: Iea1a62abf6565bcad74a2ff762c0ad69bb36eba0
Reviewed-on: https://chromium-review.googlesource.com/1096756Reviewed-by: Vaclav Brozek <vabr@chromium.org>
Commit-Queue: Dominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#566361}

components/autofill/core/browser/password_generator.cc

@@ -21,11 +21,18 @@ const uint32_t kDefaultPasswordLength = 15;
 namespace {
 
 // Default character sets used if the spec does not override the character set.
-constexpr char kLowerCaseChars[] = "abcdefghijklmnopqrstuvwxyz";
-constexpr char kUpperCaseChars[] = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";
+// Removed characters due to visual similarity:
+// - l (lowercase L)
+// - I (capital i)
+// - 1 (one)
+// - O (capital o)
+// - 0 (zero)
+// - o (lowercase O)
+constexpr char kLowerCaseChars[] = "abcdefghijkmnpqrstuvwxyz";
+constexpr char kUpperCaseChars[] = "ABCDEFGHJKLMNPQRSTUVWXYZ";
 constexpr char kAlphabeticChars[] =
-    "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ";
-constexpr char kDigits[] = "0123456789";
+    "abcdefghijkmnpqrstuvwxyzABCDEFGHJKLMNPQRSTUVWXYZ";
+constexpr char kDigits[] = "23456789";
 constexpr char kSymbols[] = "-_.";
 
 // Returns a default password requirements specification that requires: