commit 04455ee52fca1151cd245967ce3d1d45eb5bf595
author Mike Frysinger <vapier@chromium.org> Tue May 20 23:29:15 2014 -0400
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Mon May 26 18:34:15 2014 +0000
tree 7925b75534155c0c7e4b0dee69223e8cdfb62e9e
parent 88b8d78c5da4a385ee42ca699ba400a5306ef111

chrome: update pdf handling

There is no internal_pdf gyp flag anymore.  Instead, it's always built,
so drop all the special handling of the internal repo.

We still apply the USE=chrome_pdf flag to binary builds though because
when we turn that off, it'll force a lot of people to needlessly rebuild
their Chrome.  Once the flag is dropped from the ebuild, we can drop it
from our config too.

BUG=None
TEST=`cbuildbot lumpy-tot-chrome-pfq-informational` passes
TEST=`./buildbot/run_tests` passes

Change-Id: I2750e253257637122e9c1a74cdc8f7c218774127
Reviewed-on: https://chromium-review.googlesource.com/200552
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

scripts/sync_chrome.py

diff --git a/scripts/sync_chrome.py b/scripts/sync_chrome.py
index 7198e6d..43ff629 100644
--- a/scripts/sync_chrome.py
+++ b/scripts/sync_chrome.py
@@ -24,6 +24,7 @@
 
   parser.add_argument('--internal', help='Sync internal version of Chrome',
                       action='store_true', default=False)
+  # TODO(build): Delete this once all the calls disappear.
   parser.add_argument('--pdf', help='Sync PDF source code',
                       action='store_true', default=False)
   parser.add_argument('--reset', help='Revert local changes',
@@ -49,7 +50,7 @@
 
   # Sync new Chrome.
   gclient.WriteConfigFile(options.gclient, options.chrome_root,
-                          options.internal, options.pdf, options.version)
+                          options.internal, options.version)
   sync_fn = functools.partial(
       gclient.Sync, options.gclient, options.chrome_root, reset=options.reset)