Remove Win Aura bots from the buildbot and tryservers since they're now redundant as trunk Windows …

scripts/tools/lkgr_finder.py

 #!/usr/bin/env python
 # Copyright (c) 2012 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.
 
 """Fetch the latest results for a pre-selected set of builders we care about.
 If we find a 'good' revision -- based on criteria explained below -- we
 mark the revision as LKGR, and POST it to the LKGR server:
 
 http://chromium-status.appspot.com/lkgr
@@ skipping 78 matching lines @@
 # *_LKGR_STEPS controls which steps must pass for a revision to be marked
 # as LKGR.
 #-------------------------------------------------------------------------------
 
 CHROMIUM_LKGR_STEPS = {
   'chromium.win': {
     'Win Builder (dbg)': [
       'compile',
     ],
     'Win7 Tests (dbg)(1)': [
+      'ash_unittests',

[ghost stip (do not use), 2013/11/07 22:14:48]

I'm not as familiar with lkgr_finder, so I'd appreciate Paweł's review as well

[Paweł Hajdan Jr., 2013/11/07 22:22:16]

Provided the additions are correct, fine.

One landing advice though: land removals together with the change, but move
additions to a separate CL. The additions should be landed after several cycles
of the new bots to avoid holding up LKGR. It can also be tested locally (happy
to explain more) before committing live.

[jam, 2013/11/08 01:05:27]

Good idea, I'll split off the removals into another cl that I can land later
(https://codereview.chromium.org/66063002). I think that'll also help the tricky
landing wrt to CQ/try server.

+      'aura_unittests',
       'base_unittests',
       'cacheinvalidation_unittests',
       'cc_unittests',
       'check_deps',
       'chromedriver2_unittests',
       'components_unittests',
+      'compositor_unittests',
       'content_unittests',
       'courgette_unittests',
       'crypto_unittests',
       'installer_util_unittests',
       'ipc_tests',
       'jingle_unittests',
       'media_unittests',
       'ppapi_unittests',
       'printing_unittests',
       'remoting_unittests',
       'sql_unittests',
       'sync_unit_tests',
       'ui_unittests',
       'unit_tests',
       'url_unittests',
+      'views_unittests',
       'webkit_compositor_bindings_unittests',
     ],
     'Win7 Tests (dbg)(2)': [
       'net_unittests', 'browser_tests',
     ],
     'Win7 Tests (dbg)(3)': [
       'browser_tests',
     ],
     'Win7 Tests (dbg)(4)': [
       'browser_tests',
     ],
     'Win7 Tests (dbg)(5)': [
       'browser_tests',
     ],
     'Win7 Tests (dbg)(6)': [
       'browser_tests',
     ],
     'Chrome Frame Tests (ie8)': [
       'chrome_frame_tests',
       'chrome_frame_unittests',
     ],
     # 'Interactive Tests (dbg)': [

[Paweł Hajdan Jr., 2013/11/07 22:22:16]

While you're here, let's also remove this.

[jam, 2013/11/08 01:05:27]

Done.

     #   'interactive_ui_tests',
     # ],
-    'Win Aura Builder': [
-      'compile',
-    ],
-    'Win Aura Tests (1)': [
-      'ash_unittests',
-      'aura_unittests',
-      'browser_tests',
-      'content_browsertests',
-    ],
-    'Win Aura Tests (2)': [
-      'browser_tests',
-      'compositor_unittests',
-      'content_unittests',
-      'unit_tests',
-      'views_unittests',
-    ],
-    'Win Aura Tests (3)': [
-      'browser_tests',
-      'interactive_ui_tests',
-    ],
   },  # chromium.win
   'chromium.mac': {
     'Mac Builder (dbg)': [
       'compile',
     ],
     'Mac 10.6 Tests (dbg)(1)': [
       'browser_tests',
       'cc_unittests',
       'chromedriver2_unittests',
       'jingle_unittests',
@@ skipping 136 matching lines @@
     'Linux Builder (dbg)(32)': ['compile'],
     'Linux Aura': ['compile'],
     'Android Builder (dbg)': ['slave_steps'],
     'Android Builder': ['slave_steps'],
   },
   'chromium.mac': {
     'Mac Builder (dbg)': ['compile'],
     },
   'chromium.win': {
     'Win Builder (dbg)': ['compile'],
-    'Win Aura Builder': ['compile'],
   },
   'chromium.webkit': {
     'WebKit Win Builder (deps)': ['compile'],
     'WebKit Mac Builder (deps)': ['compile'],
     'WebKit Linux (deps)': ['compile'],
   },
 }
 
 V8_NORMAL_STEPS = ['compile', 'Check', 'Test262', 'Mozilla']
 
@@ skipping 817 matching lines @@
                '%s%s LKGR (%s) exceeds lag threshold' %
                (subject_base, lkgr_type, lkgr), '\n'.join(RUN_LOG))
       return 1
 
   VerbosePrint('-' * 52)
 
   return 0
 
 if __name__ == '__main__':
   sys.exit(main())