Add UpgradeToHttps() to reliably and forcibly upgrade all urls to https

gclient_utils.py

 # Copyright (c) 2011 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.
 
 """Generic utils."""
 
 import errno
 import logging
 import os
 import Queue
@@ skipping 716 matching lines @@
       # shell=True to allow the shell to handle all forms of quotes in
       # $EDITOR.
       subprocess2.check_call(cmd, shell=True)
     except subprocess2.CalledProcessError:
       return None
     return FileRead(filename)
   finally:
     os.remove(filename)
 
 
+def UpgradeToHttps(url):
+  """Upgrades random urls to https://.
+
+  Do not touch unknown urls like ssh:// or git://.
+  Fixes invalid GAE url.
+  """
+  if not url:
+    return url
+  if not re.match(r'[a-z\-]+\://.*', url):
+    # Make sure it is a valid uri.
+    url = 'https://%s' % url
+  url = re.sub(r'^http\:', r'https:', url)
+  url = re.sub(
+      '^https\:\/\/codereview.chromium.org',
+      'https://chromiumcodereview.appspot.com',
+      url)
+  return url
+
+
 def ParseCodereviewSettingsContent(content):
   """Process a codereview.settings file properly."""
   lines = (l for l in content.splitlines() if not l.strip().startswith("#"))
   try:
     keyvals = dict([x.strip() for x in l.split(':', 1)] for l in lines if l)
   except ValueError:
     raise Error(
         'Failed to process settings, please fix. Content:\n\n%s' % content)
-  # TODO(maruel): Post-process
+  def fix_url(key):
+    if keyvals.get(key):
+      keyvals[key] = UpgradeToHttps(keyvals[key])
+  fix_url('CODE_REVIEW_SERVER')
+  fix_url('VIEW_VC')
   return keyvals