Removing deprecated environments from App Remoting build files.

remoting/webapp/build-webapp.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Creates a directory with with the unpacked contents of the remoting webapp.
 
 The directory will contain a copy-of or a link-to to all remoting webapp
 resources.  This includes HTML/JS and any plugin binaries. The script also
 massages resulting files appropriately with host plugin data. Finally,
@@ skipping 127 matching lines @@
     files_listfile: The name of a file containing a list of files, one per
                     line, identifying the resources to include in this webapp.
                     This is an alternate to specifying the files directly via
                     the 'files' option. The files listed in this file are
                     appended to the files passed via the 'files' option, if any.
     locales_listfile: The name of a file containing a list of locales, one per
                       line, which are copied, along with their directory
                       structure, from the _locales directory down.
     jinja_paths: An array of paths to search for {%include} directives in
                  addition to the directory containing the manifest template.
-    service_environment: Used to point the webapp to one of the
-                         dev/test/staging/vendor/prod/prod-testing environments
+    service_environment: Used to point the webapp to the dev/prod environments.
     use_gcd: True if GCD support should be enabled.
   """
 
   # Load the locales files from the locales_listfile.
   if not locales_listfile:
     raise Exception('You must specify a locales_listfile')
   locales = []
   with open(locales_listfile) as input:
     for s in input:
       locales.append(s.rstrip())
@@ skipping 51 matching lines @@
     elif extension == '.pak':
       destination_file = os.path.join(remoting_locales,
                                       os.path.split(current_locale)[1])
       shutil.copy2(current_locale, destination_file)
     else:
       raise Exception('Unknown extension: ' + current_locale)
 
   is_app_remoting_webapp = webapp_type == 'app_remoting'
   is_app_remoting_shared_module = webapp_type == 'shared_module'
   is_app_remoting = is_app_remoting_webapp or is_app_remoting_shared_module
-  is_prod_service_environment = service_environment == 'vendor' or \
-                                service_environment == 'prod' or \
-                                service_environment == 'prod-testing'
+  is_prod_service_environment = service_environment == 'prod'
   is_desktop_remoting = not is_app_remoting
 
   # Allow host names for google services/apis to be overriden via env vars.
   oauth2AccountsHost = os.environ.get(
       'OAUTH2_ACCOUNTS_HOST', 'https://accounts.google.com')
   oauth2ApiHost = os.environ.get(
       'OAUTH2_API_HOST', 'https://www.googleapis.com')
   directoryApiHost = os.environ.get(
       'DIRECTORY_API_HOST', 'https://www.googleapis.com')
   remotingApiHost = os.environ.get(
       'REMOTING_API_HOST', 'https://remoting-pa.googleapis.com')
 
   if is_app_remoting:
     appRemotingApiHost = os.environ.get(
         'APP_REMOTING_API_HOST', None)
 
   if is_app_remoting_webapp:
     appRemotingApplicationId = os.environ.get(
         'APP_REMOTING_APPLICATION_ID', None)
 
     # Release/Official builds are special because they are what we will upload
     # to the web store.  The checks below will validate that prod builds are
     # being generated correctly (no overrides) and with the correct buildtype.
-    # They also verify that folks are not accidentally building dev/test/staging
-    # apps for release (no impersonation) instead of dev.
+    # They also verify that folks are not accidentally building dev apps for
+    # Release (no impersonation) instead of Dev.
     if is_prod_service_environment and buildtype == 'Dev':
       raise Exception("Prod environment cannot be built for 'dev' builds")
 
     if buildtype != 'Dev':
       if not is_prod_service_environment:
         raise Exception('Invalid service_environment targeted for '
                         + buildtype + ': ' + service_environment)
       if appid != None:
         raise Exception('Cannot pass in an appid for '
                         + buildtype + ' builds: ' + service_environment)
@@ skipping 13 matching lines @@
       appRemotingApplicationId = "chrome.i18n.getMessage('@@extension_id')"
     findAndReplace(os.path.join(destination, 'arv_main.js'),
                    "'APP_REMOTING_APPLICATION_ID'", appRemotingApplicationId)
 
   oauth2BaseUrl = oauth2AccountsHost + '/o/oauth2'
   oauth2ApiBaseUrl = oauth2ApiHost + '/oauth2'
   directoryApiBaseUrl = directoryApiHost + '/chromoting/v1'
   telemetryApiBaseUrl = remotingApiHost + '/v1/events'
 
   if is_app_remoting:
-    # Set the apiary endpoint and then set the endpoint version
+    # Set the base endpoint url first and then set the endpoint version.
     if not appRemotingApiHost:
       if is_prod_service_environment:
         appRemotingApiHost = 'https://www.googleapis.com'
       else:
         appRemotingApiHost = 'https://www-googleapis-test.sandbox.google.com'
 
     # TODO(garykac) Currently, the shared module is always set up for the
     # dev service_environment. Update build so that the dev environment can
     # be controlled by the app stub rather than hard-coded into the shared
     # module.
     if service_environment == 'dev' or is_app_remoting_shared_module:
       appRemotingServicePath = '/appremoting/v1beta1_dev'
-    elif service_environment == 'test':
-      appRemotingServicePath = '/appremoting/v1beta1'
-    elif service_environment == 'staging':
-      appRemotingServicePath = '/appremoting/v1beta1_staging'
-    elif service_environment == 'vendor':
-      appRemotingServicePath = '/appremoting/v1beta1_vendor'
     elif service_environment == 'prod':
       appRemotingServicePath = '/appremoting/v1beta1'
-    elif service_environment == 'prod-testing':
-      appRemotingServicePath = '/appremoting/v1beta1_prod_testing'
     else:
       raise Exception('Unknown service environment: ' + service_environment)
     appRemotingApiBaseUrl = appRemotingApiHost + appRemotingServicePath
   else:
     appRemotingApiBaseUrl = ''
 
   # TODO(garykac) replaceString (et al.) implictly update plugin_settings.js,
   # which doesn't exist for the app stub. We need to move app-specific
   # AppRemoting options into arv_main.js.
   if not is_app_remoting_webapp:
@@ skipping 169 matching lines @@
   parser.add_argument('--use_gcd', choices=['0', '1'], default='0')
 
   args = parser.parse_args()
   args.use_gcd = (args.use_gcd != '0')
   args.app_capabilities = set(args.app_capabilities)
   return buildWebApp(**vars(args))
 
 
 if __name__ == '__main__':
   sys.exit(main())