Move the "extension" namespace from extension_api.json into extension_api_extension.json.

chrome/common/extensions/docs/build/build.py

 #!/usr/bin/python
 # Copyright (c) 2011 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.
 
 """Docbuilder for extension docs."""
 
 import os
 import os.path
 import shutil
 import sys
 import time
 import urllib
 
 from subprocess import Popen, PIPE
 from optparse import OptionParser
 
 _script_path = os.path.realpath(__file__)
 _build_dir = os.path.dirname(_script_path)
 _base_dir = os.path.normpath(_build_dir + "/..")
 _static_dir = _base_dir + "/static"
 _js_dir = _base_dir + "/js"
 _template_dir = _base_dir + "/template"
 _samples_dir = _base_dir + "/examples"
 _extension_api_dir = os.path.normpath(_base_dir + "/../api")
 
 _extension_api_json = _extension_api_dir + "/extension_api.json"
+_extension_api_extension_json =

[Aaron Boodman, 2011/12/07 23:30:35]

If this is only used in the one place below, maybe not worth creating a variable
for it.

[koz (OOO until 15th September), 2011/12/09 19:24:18]

As per kalman's comment, I've rolled all of these into one variable.

+    _extension_api_dir + "/extension_api_extension.json"

[not at google - send to devlin, 2011/12/08 00:32:49]

would be nice to do an ls *.json on here so that when we add more files, and
other people add files once this refactor is finished, they don't have to worry
about touching python

[koz (OOO until 15th September), 2011/12/09 19:24:18]

Done.

 _devtools_api_json = _extension_api_dir + "/devtools_api.json"
 _api_template_html = _template_dir + "/api_template.html"
 _page_shell_html = _template_dir + "/page_shell.html"
 _generator_html = _build_dir + "/generator.html"
 _samples_json = _base_dir + "/samples.json"
 
 _expected_output_preamble = "#BEGIN"
 _expected_output_postamble = "#END"
 
 # HACK! This is required because we can only depend on python 2.4 and
@@ skipping 140 matching lines @@
                     default=True)
   options, args = parser.parse_args()
 
   if (options.dump_render_tree_path and
       os.path.isfile(options.dump_render_tree_path)):
     dump_render_tree = options.dump_render_tree_path
   else:
     dump_render_tree = FindDumpRenderTree()
 
   # Load the manifest of existing API Methods
-  api_manifest = ApiManifest([_extension_api_json, _devtools_api_json])
+  api_manifest = ApiManifest([_extension_api_json,
+                              _extension_api_extension_json,
+                              _devtools_api_json])

[Aaron Boodman, 2011/12/07 23:30:35]

That was lucky.

[koz (OOO until 15th September), 2011/12/09 19:24:18]

Yes :-)

 
   # Read static file names
   static_names = GetStaticFileNames()
 
   # Read module names
   module_names = api_manifest.getModuleNames()
 
   # All pages to generate
   page_names = static_names | module_names
 
@@ skipping 36 matching lines @@
   if (os.path.isfile(debug_log)):
     os.remove(debug_log)
 
   if 'EX_OK' in dir(os):
     return os.EX_OK
   else:
     return 0
 
 if __name__ == '__main__':
   sys.exit(main())