fix: Replace with respectful terms (#801)

Replace blacklist and whitelist with disallowlist and allowlist when possible.

Author: bojeil-google

javascript/widgets/config.js

@@ -459,19 +459,19 @@ class Config {
         (!Array.isArray(blacklistedCountries))) {
       throw new Error('BlacklistedCountries must be an array.');
     }
-    // If both whitelist and blacklist are provided, throw error.
+    // If both allowlist and disallowlist are provided, throw error.
     if (whitelistedCountries && blacklistedCountries) {
       throw new Error(
           'Both whitelistedCountries and blacklistedCountries are provided.');
     }
-    // If no whitelist or blacklist provided, return all available countries.
+    // If no allowlist or disallowlist provided, return all available countries.
     if (!whitelistedCountries && !blacklistedCountries) {
       return country.COUNTRY_LIST;
     }
     let countries = [];
     const availableCountries = [];
     if (whitelistedCountries) {
-      // Whitelist is provided.
+      // Allowlist is provided.
       const whitelistedCountryMap = {};
       for (let i = 0; i < whitelistedCountries.length; i++) {
         countries = country

javascript/widgets/config_test.js

@@ -442,16 +442,20 @@ testSuite({
     assertArrayEquals([], warningLogMessages);
 
     // Phone config with blacklisted reCAPTCHA parameters.
-    const blacklist = {
+    const disallowlist = {
       'sitekey': 'SITEKEY',
       'tabindex': 0,
       'callback': function(token) {},
       'expired-callback': function() {},
     };
     config.update(
-        'signInOptions',
-        ['github.com', {'provider': 'google.com'},
-         {'provider': 'phone', 'recaptchaParameters': blacklist}, 'password']);
+      'signInOptions',
+      ['github.com', {'provider': 'google.com'},
+       {
+         'provider': 'phone',
+         'recaptchaParameters': disallowlist
+       },
+       'password']);
     assertObjectEquals({}, config.getRecaptchaParameters());
     // Expected warning should be logged.
     assertArrayEquals(
@@ -958,7 +962,7 @@ testSuite({
   },
 
   testUpdateConfig_phoneSignInOption_error() {
-    // Tests when both whitelist and blacklist are provided.
+    // Tests when both allowlist and disallowlist are provided.
     let error = assertThrows(() => {
       config.update('signInOptions', [{
         'provider': 'phone',
@@ -969,7 +973,7 @@ testSuite({
     assertEquals(
         'Both whitelistedCountries and blacklistedCountries are provided.',
         error.message);
-    // Tests when empty whitelist is provided.
+    // Tests when empty allowlist is provided.
     error = assertThrows(() => {
       config.update('signInOptions', [{
         'provider': 'phone',
@@ -1022,7 +1026,7 @@ testSuite({
   },
 
   testSetConfig_phoneSignInOption_error() {
-    // Tests when both whitelist and blacklist are provided.
+    // Tests when both allowlist and disallowlist are provided.
     let error = assertThrows(() => {
       config.setConfig({
         'signInOptions': [{
@@ -1035,7 +1039,7 @@ testSuite({
     assertEquals(
         'Both whitelistedCountries and blacklistedCountries are provided.',
         error.message);
-    // Tests when empty whitelist is provided.
+    // Tests when empty allowlist is provided.
     error = assertThrows(() => {
       config.setConfig({
         'signInOptions': [{