Remove temporary directory cleanup on windows.

tools/bots/compiler.py

 #!/usr/bin/python
 
 # Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 # for details. All rights reserved. Use of this source code is governed by a
 # BSD-style license that can be found in the LICENSE file.
 
 """
 Dart2js buildbot steps
 
 Runs tests for the  dart2js compiler.
@@ skipping 239 matching lines @@
       extras_flags = flags
       if (system == 'linux'
           and runtime == 'd8'
           and not '--host-checked' in extras_flags):
         # Run the extra tests in checked mode, but only on linux/d8.
         # Other systems have less resources and tend to time out.
         extras_flags = extras_flags + ['--host-checked']
       TestStep("dart2js_extra", mode, system, 'dart2js', runtime, extras,
                extras_flags, arch)
 
-
-def _DeleteTempWebdriverProfiles(directory):
-  """Find all the firefox profiles in a particular directory and delete them."""
-  for f in os.listdir(directory):
-    item = os.path.join(directory, f)
-    if os.path.isdir(item) and (f.startswith('tmp') or f.startswith('opera')):
-      subprocess.Popen('rm -rf %s' % item, shell=True)
-
-
-def CleanUpTemporaryFiles(system, browser):
-  """For some browser (selenium) tests, the browser creates a temporary profile
-  on each browser session start. On Windows, generally these files are
-  automatically deleted when all python processes complete. However, since our
-  buildbot slave script also runs on python, we never get the opportunity to
-  clear out the temp files, so we do so explicitly here. Our batch browser
-  testing will make this problem occur much less frequently, but will still
-  happen eventually unless we do this.
-
-  This problem also occurs with batch tests in Firefox. For some reason selenium
-  automatically deletes the temporary profiles for Firefox for one browser,
-  but not multiple ones when we have many open batch tasks running. This
-  behavior has not been reproduced outside of the buildbots.
-
-  Args:
-     - system: either 'linux', 'mac', 'windows'
-     - browser: one of the browsers, see GetBuildInfo
-  """
-  if system == 'windows':
-    temp_dir = 'C:\\Users\\chrome-bot\\AppData\\Local\\Temp'
-    for name in os.listdir(temp_dir):
-      fullname = os.path.join(temp_dir, name)
-      if os.path.isdir(fullname):
-        shutil.rmtree(fullname, ignore_errors=True)
-  elif browser == 'ff' or 'opera':
-    # Note: the buildbots run as root, so we can do this without requiring a
-    # password. The command won't actually work on regular machines without
-    # root permissions.
-    _DeleteTempWebdriverProfiles('/tmp')
-    _DeleteTempWebdriverProfiles('/var/tmp')
-
-
 def GetHasHardCodedCheckedMode(build_info):
   # TODO(ricow): We currently run checked mode tests on chrome on linux and
   # on the slow (all) IE windows bots. This is a hack and we should use the
   # normal sharding and checked splitting functionality when we get more
   # vms for testing this.
   if (build_info.system == 'linux' and build_info.runtime == 'drt'):
     return True
   if build_info.runtime.startswith('ie') and build_info.test_set == 'all':
     return True
   return False
@@ skipping 46 matching lines @@
                list(test_flags), build_info.is_buildbot, build_info.test_set,
                build_info.arch, compiler=build_info.compiler)
 
   # See comment in GetHasHardCodedCheckedMode, this is a hack.
   if (GetHasHardCodedCheckedMode(build_info)):
     TestCompiler(build_info.runtime, build_info.mode, build_info.system,
                  test_flags + ['--checked'], build_info.is_buildbot,
                  build_info.test_set, build_info.arch,
                  compiler=build_info.compiler)
 
-  if build_info.runtime != 'd8':
-    CleanUpTemporaryFiles(build_info.system, build_info.runtime)
-
 
 def BuildCompiler(build_info):
   """
   Builds the SDK.
 
   - build_info: the buildInfo object, containing information about what sort of
       build and test to be run.
   """
   with bot.BuildStep('Build SDK and d8'):
     args = [sys.executable, './tools/build.py', '--mode=' + build_info.mode,
             '--arch=' + build_info.arch, 'dart2js_bot']
     print 'Build SDK and d8: %s' % (' '.join(args))
     bot.RunProcess(args)
 
 
 if __name__ == '__main__':
   bot.RunBot(GetBuildInfo, RunCompilerTests, build_step=BuildCompiler)