Fix Deps for android/perf recipe.

scripts/slave/recipe_modules/gclient/config.py

 # Copyright 2013 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.
 
 import types
 
 from slave.recipe_config import config_item_context, ConfigGroup, BadConf
 from slave.recipe_config import ConfigList, Dict, Single, Static, Set, List
 
 def BaseConfig(USE_MIRROR=True, GIT_MODE=False, CACHE_DIR=None, **_kwargs):
@@ skipping 299 matching lines @@
 
 @config_ctx(config_vars={'GIT_MODE': True})
 def tools_build(c):
   if not c.GIT_MODE:
     raise BadConf('tools_build only supports git')
   s = c.solutions.add()
   s.name = 'build'
   s.url = ChromiumGitURL(c, 'chromium', 'tools', 'build.git')
   m = c.got_revision_mapping
   m['build'] = 'got_revision'
+
+@config_ctx(includes=['chromium', 'chrome_internal'])
+def perf(c):
+  s = c.solutions[0]
+  s.custom_vars['llvm_url'] = 'svn://svn-mirror.golo.chromium.org/llvm-project'
+  s.custom_deps.update({

[tonyg, 2014/07/10 19:47:31]

Let's add a comment noting that these are removed because they're large and the
perf bots don't need them.

[zty, 2014/07/10 20:29:08]

Done.

+    'src/chrome/test/data/pdf_private': None,
+    'src/third_party/WebKit/LayoutTests': None,
+    'src/tools/valgrind': None,
+  })
+  s.managed = False

[tonyg, 2014/07/10 19:47:31]

I'm curious what this does.

[zty, 2014/07/10 20:29:08]

"managed" is a property on gclient solutions. If it is set to True, gclient sync
would attempt to do git pull --rebase. Otherwise, it blows up all the local
changes. managed=True is considered evil and is going away. If bot_update is
enabled, managed is always False. Hence the noop here. But I would like it to be
explicit.
On 2014/07/10 19:47:31, tonyg wrote:

+  needed_components_internal = [
+    "src/chrome/test/data/perf/frame_rate/private",
+    "src/chrome/test/data/plugin",
+    "src/data/memory_test",
+    "src/data/page_cycler",

[tonyg, 2014/07/10 19:47:31]

I *think* this is the only one we actually need. Eventually I'd like to get rid
of all these src-internal deps, so maybe eliminate the others unless we find
that we need them?

+    "src/data/tab_switching",
+  ]
+  for key in needed_components_internal:
+    del c.solutions[1].custom_deps[key]
+  c.solutions[1].managed = False