Revert "IDN Display: Add whole-script-confusable characters for Greek"

This reverts commit de165b66.

Reason for revert: Potentially implicated for browser_tests ErrorPageForIDNTest.IDN failure

First failing run
https://ci.chromium.org/p/chromium/builders/ci/Win10%20Tests%20x64/43723

[ RUN      ] ErrorPageForIDNTest.IDN
../../chrome/browser/net/errorpage_browsertest.cc(1443): error: Value of: IsDisplayingText(browser(), kHostnameJSUnicode)
  Actual: false
Expected: true

Original change's description:
> IDN Display: Add whole-script-confusable characters for Greek
> 
> This CL adds a list of Greek characters that look like Latin
> characters. A domain consisting only of these characters and not
> having a .gr ccTLD or other Greek TLDs such as .ελ will be
> converted to punycode, as is done for other
> whole-script-confusables such as Cyrillic and Hebrew.
> 
> Bug: 722167
> Change-Id: Id0157503c605a6a73eb49147c26b48d5995d1ff2
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986209
> Commit-Queue: Mustafa Emre Acer <meacer@chromium.org>
> Reviewed-by: Joe DeBlasio <jdeblasio@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#728697}

TBR=meacer@chromium.org,jdeblasio@chromium.org

Change-Id: I01408fe8791d50ce10feeb7656bfd72b9cea105d
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 722167
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1989297Reviewed-by: Robert Liao <robliao@chromium.org>
Commit-Queue: Robert Liao <robliao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#728827}

components/url_formatter/spoof_checks/idn_spoof_checker.cc

@@ -9,7 +9,6 @@
 #include "base/strings/string_piece.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
-#include "base/strings/utf_string_conversions.h"
 #include "base/threading/thread_local_storage.h"
 #include "build/build_config.h"
 #include "net/base/lookup_string_in_fixed_set.h"
@@ -210,20 +209,12 @@ IDNSpoofChecker::IDNSpoofChecker() {
        // (Potential set: [ሀሁሃሠሡሰሱሲስበቡቢተቱቲታነከኩኪካኬክዐዑዕዖዘዙዚዛዝዞጠጡጢጣጦፐፒꬁꬂꬅ])
        "[[:Ethi:]]",
        "[ሀሠሰስበነተከዐዕዘጠፐꬅ]",
-       {"er", "et"}},
-      {// Greek
-       "[[:Grek:]]",
-       // This ignores variants such as ά, έ, ή, ί.
-       "[αικνρυωηοτ]",
-       {"gr"}},
-  };
+       {"er", "et"}}};
   for (const WholeScriptConfusableData& data : kWholeScriptConfusables) {
     auto all_letters = std::make_unique<icu::UnicodeSet>(
         icu::UnicodeString::fromUTF8(data.script_regex), status);
-    DCHECK(U_SUCCESS(status));
     auto latin_lookalikes = std::make_unique<icu::UnicodeSet>(
         icu::UnicodeString::fromUTF8(data.latin_lookalike_letters), status);
-    DCHECK(U_SUCCESS(status));
     auto script = std::make_unique<WholeScriptConfusable>(
         std::move(all_letters), std::move(latin_lookalikes), data.allowed_tlds);
     wholescriptconfusables_.push_back(std::move(script));

components/url_formatter/spoof_checks/idn_spoof_checker.h

@@ -100,9 +100,7 @@ class IDNSpoofChecker {
         const std::vector<std::string>& allowed_tlds);
     ~WholeScriptConfusable();
 
-    // Captures all letters belonging to this script. See kScriptNameCodeList in
-    // blink/renderer/platform/text/locale_to_script_mapping.cc for script
-    // codes.
+    // Captures all letters belonging to this script.
     std::unique_ptr<icu::UnicodeSet> all_letters;
     // The subset of all_letters that look like Latin ASCII letters. A domain
     // label entirely made of them is blocked as a simplified

components/url_formatter/spoof_checks/idn_spoof_checker_unittest.cc

@@ -1207,11 +1207,6 @@ const IDNTestCase kIdnCases[] = {
     // Whole-script-confusable in Ethiopic.
     {"xn--6xd66aa62c.com", L"ሠዐዐፐ.com", kUnsafe},
     {"xn--6xd66aa62c.et", L"ሠዐዐፐ.et", kSafe},
-
-    // Whole-script-confusable in Greek.
-    {"xn--mxapd.com", L"ικα.com", kUnsafe},
-    {"xn--mxapd.gr", L"ικα.gr", kSafe},
-    {"xn--mxapd.xn--qxam", L"ικα.ελ", kSafe},
 };
 
 namespace test {