[telemetry] Remove all pyc files before doing presumibit work

tools/telemetry/build/update_docs.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 logging
 import optparse
 import os
 import pkgutil
 import pydoc
 import re
 import sys
 
 from telemetry.core import util
 
 telemetry_dir = util.GetTelemetryDir()
 docs_dir = os.path.join(telemetry_dir, 'docs')
 
 def EnsureTelemetryIsInPath():
   if telemetry_dir not in sys.path:
     sys.path.append(telemetry_dir)
 
 def RemoveAllDocs():
   for dirname, _, filenames in os.walk(docs_dir):
     for filename in filenames:
       os.remove(os.path.join(dirname, filename))
 
+def RemoveAllStalePycFiles():
+  for dirname, _, filenames in os.walk(telemetry_dir):
+    for filename in filenames:
+      if not filename.endswith('.pyc'):
+        continue
+      pycpath = os.path.join(dirname, filename)

[tonyg, 2013/06/21 19:02:38]

I'd find these more readable as pyc_path and py_path.

+      pypath = os.path.splitext(pycpath)[0] + '.py'
+      if not os.path.exists(pypath):
+        os.remove(pypath)
+
 def GenerateHTMLForModule(module):
   html = pydoc.html.page(pydoc.describe(module),
                          pydoc.html.document(module, module.__name__))
 
   # pydoc writes out html with links in a variety of funky ways. We need
   # to fix them up.
   assert not telemetry_dir.endswith(os.sep)
   links = re.findall('(<a href="(.+?)">(.+?)</a>)', html)
   for link_match in links:
     link, href, link_text = link_match
@@ skipping 16 matching lines @@
   return html
 
 def WriteHTMLForModule(module):
   page = GenerateHTMLForModule(module)
   path = os.path.join(docs_dir, '%s.html' % module.__name__)
   with open(path, 'w') as f:
     sys.stderr.write('Wrote %s\n' % os.path.relpath(path))
     f.write(page)
 
 def GetAllModulesToDocument(module):
+  RemoveAllStalePycFiles()
   modules = [module]
   for _, modname, _ in pkgutil.walk_packages(
       module.__path__, module.__name__ + '.'):
     if modname.endswith('_unittest'):
       logging.debug("skipping %s due to being a unittest", modname)
       continue
 
     module = __import__(modname, fromlist=[""])
     name, _ = os.path.splitext(module.__file__)
     if not os.path.exists(name + '.py'):
@@ skipping 76 matching lines @@
   EnsureTelemetryIsInPath()
   import telemetry
 
   old_cwd = os.getcwd()
   try:
     os.chdir(telemetry_dir)
     for module in GetAllModulesToDocument(telemetry):
       WriteHTMLForModule(module)
   finally:
     os.chdir(old_cwd)