From b86c00ff9579b45e1ba8fee15b892c78afcead51 Mon Sep 17 00:00:00 2001
From: Nikolaos Trogkanis <tronikos@gmail.com>
Date: Tue, 23 Apr 2013 07:36:38 +0000
Subject: [PATCH] JS: Fix some compiler issues. Add missing ReginoCode.CH in
 regioncodefortesting.js. Change visibility of isNumberGeographical to public
 so that it can be accessed from tests.

Review URL: https://codereview.appspot.com/8649050
---
 javascript/i18n/phonenumbers/phonenumberutil.js      | 5 ++---
 javascript/i18n/phonenumbers/phonenumberutil_test.js | 6 +++---
 javascript/i18n/phonenumbers/regioncodefortesting.js | 1 +
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/javascript/i18n/phonenumbers/phonenumberutil.js b/javascript/i18n/phonenumbers/phonenumberutil.js
index 1b7d76870..a62761f4c 100644
--- a/javascript/i18n/phonenumbers/phonenumberutil.js
+++ b/javascript/i18n/phonenumbers/phonenumberutil.js
@@ -1138,7 +1138,7 @@ i18n.phonenumbers.PhoneNumberUtil.prototype.getLengthOfGeographicalAreaCode =
     return 0;
   }
 
-  if (!this.isNumberGeographical_(number)) {
+  if (!this.isNumberGeographical(number)) {
     return 0;
   }
 
@@ -1296,9 +1296,8 @@ i18n.phonenumbers.PhoneNumberUtil.prototype.formattingRuleHasFirstGroupOnly =
  *
  * @param {i18n.phonenumbers.PhoneNumber} phoneNumber The phone number to test.
  * @return {boolean} true if the phone number has a geographical association.
- * @private
  */
-i18n.phonenumbers.PhoneNumberUtil.prototype.isNumberGeographical_ =
+i18n.phonenumbers.PhoneNumberUtil.prototype.isNumberGeographical =
     function(phoneNumber) {
   /** @type {i18n.phonenumbers.PhoneNumberType} */
   var numberType = this.getNumberType(phoneNumber);
diff --git a/javascript/i18n/phonenumbers/phonenumberutil_test.js b/javascript/i18n/phonenumbers/phonenumberutil_test.js
index 1ee08a5f1..5e29a9022 100644
--- a/javascript/i18n/phonenumbers/phonenumberutil_test.js
+++ b/javascript/i18n/phonenumbers/phonenumberutil_test.js
@@ -318,11 +318,11 @@ function testGetInstanceLoadInternationalTollFreeMetadata() {
 
 function testIsNumberGeographical() {
   // Bahamas, mobile phone number.
-  assertFalse(phoneUtil.isNumberGeographical_(BS_MOBILE));
+  assertFalse(phoneUtil.isNumberGeographical(BS_MOBILE));
   // Australian fixed line number.
-  assertTrue(phoneUtil.isNumberGeographical_(AU_NUMBER));
+  assertTrue(phoneUtil.isNumberGeographical(AU_NUMBER));
   // International toll free number.
-  assertFalse(phoneUtil.isNumberGeographical_(INTERNATIONAL_TOLL_FREE));
+  assertFalse(phoneUtil.isNumberGeographical(INTERNATIONAL_TOLL_FREE));
 }
 
 function testIsLeadingZeroPossible() {
diff --git a/javascript/i18n/phonenumbers/regioncodefortesting.js b/javascript/i18n/phonenumbers/regioncodefortesting.js
index 3142fad34..f17960553 100644
--- a/javascript/i18n/phonenumbers/regioncodefortesting.js
+++ b/javascript/i18n/phonenumbers/regioncodefortesting.js
@@ -41,6 +41,7 @@ i18n.phonenumbers.RegionCode = {
   BS: 'BS',
   BY: 'BY',
   CA: 'CA',
+  CH: 'CH',
   CN: 'CN',
   CS: 'CS',
   DE: 'DE',