Add reviewers to roll_deps.py, remove linux_heapcheck bot

tools/roll_deps.py

Index: tools/roll_deps.py
diff --git a/tools/roll_deps.py b/tools/roll_deps.py
index 10822f4d4c754ecb922799df420baa120ca11242..b34baac41837f0da8f61b3e656e80a773a2bd2f8 100755
--- a/tools/roll_deps.py
+++ b/tools/roll_deps.py
@@ -64,6 +64,15 @@ class DepsRollConfig(object):
         self.cl_bot_list = [bot for bot in options.bots.split(',') if bot]
         self.skia_git_checkout_path = options.skia_git_path
         self.default_branch_name = 'autogenerated_deps_roll_branch'
+        self.reviewers_list = ','.join([
+            # 'rmistry@google.com',
+            # 'reed@google.com',
+            # 'bsalomon@google.com',
+            # 'robertphillips@google.com',
+            ])

[robertphillips, 2014/01/21 20:18:55]

Is this going to mess up the chromium-reviews@chromium.org cc?

+        self.cc_list =  ','.join([
+            # 'skia-team@google.com',
+            ])
 
     @staticmethod
     def GetOptionParser():
@@ -85,7 +94,6 @@ class DepsRollConfig(object):
             'linux_chromeos',
             'linux_chromeos_asan',
             'linux_gpu',
-            'linux_heapcheck',
             'linux_layout',
             'linux_layout_rel',
             'mac',
@@ -604,8 +612,13 @@ class GitBranchCLUpload(object):
             vsp.check_call([git, 'add', filename])
         vsp.check_call([git, 'commit', '-q', '-m', self._message])
 
-        git_cl = [git, 'cl', 'upload', '-f', '--cc=skia-team@google.com',
+        git_cl = [git, 'cl', 'upload', '-f',
                   '--bypass-hooks', '--bypass-watchlists']
+        if self._config.cc_list:
+            git_cl.append('--cc=%s' % self._config.cc_list)
+        if self._config.reviewers_list:
+            git_cl.append('--reviewers=%s' % self._config.reviewers_list)
+
         git_try = [git, 'cl', 'try', '--revision', svn_info]
         git_try.extend([arg for bot in self._config.cl_bot_list
                         for arg in ('-b', bot)])