[WPT Export] Add ability to pass GitHub creds as JSON file

third_party/WebKit/Tools/Scripts/wpt-export

 #!/usr/bin/env python
 # Copyright 2017 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.
 
 """Exports local changes to web-platform-tests in Chromium to upstream repo.
 
 This script checks LayoutTests/external/wpt for changes that can be exported,
 then creates a patch, and creates and lands a pull request for the upstream
 repository.
 """
 
 import argparse
 import logging
+import json

[qyearsley, 2017/02/07 19:32:16]

Nit: sort import order

 
 from webkitpy.common.host import Host
 from webkitpy.w3c.test_exporter import TestExporter
 from webkitpy.w3c.wpt_github import WPTGitHub
 
 
 _log = logging.getLogger(__name__)
 
 
 def main():
     logging.basicConfig(level=logging.INFO, format='%(message)s')
 
     parser = argparse.ArgumentParser(description=__doc__)
     parser.add_argument(
         '--dry-run', action='store_true',
         help='See what would be done without actually creating or merging '
              'any pull requests.')
     parser.add_argument(
-        '--user',
+        '--gh-user',
         help='GitHub user name. Can also be provided using the GH_USER '
              'environment variable.')
     parser.add_argument(
-        '--token',
+        '--gh-token',
         help='GitHub token or password. Can also be provided using the GH_TOKEN '
              'environment variable.')
+    parser.add_argument(
+        '--github-credentials-json',
+        help='A JSON file with schema {"GH_USER": "", "GH_TOKEN": ""}.')

[qyearsley, 2017/02/07 19:32:17]

Optional: Note that this will override any other username/token provided via
environment variables or command-line flags.

     args = parser.parse_args()
     host = Host()
-    if not args.user:
+
+    if not args.gh_user:
         args.user = host.environ.get('GH_USER')
-    if not args.token:
-        args.token = host.environ.get('GH_TOKEN')
-    if not (args.user and args.token):
-        parser.error('Must provide both user and token for GitHub.')
+    if not args.gh_token:
+        args.gh_token = host.environ.get('GH_TOKEN')
+    if args.github_credentials_json:
+        with open(args.github_credentials_json, 'r') as f:
+            contents = json.load(f)
+            args.gh_user = contents['GH_USER']
+            args.gh_token = contents['GH_TOKEN']
 
-    wpt_github = WPTGitHub(host, args.user, args.token)
+    if not (args.gh_user and args.gh_token):
+        parser.error('Must provide both gh_user and gh_token for GitHub.')
+
+    wpt_github = WPTGitHub(host, args.gh_user, args.gh_token)
     test_exporter = TestExporter(host, wpt_github, dry_run=args.dry_run)
 
     test_exporter.run()
 
 
 if __name__ == '__main__':
     main()