Revert 124660 - Allow comments in extension config files.

Revert 124660 - Allow comments in extension config files.

Added a script to remove comments from the extension api JSON files before
processing for the extension docs. We will now be able to comment in the JSON
files, and they should be much easier to read. Also added the license header
to all the JSON files.

BUG=114233
TEST=Put comments in one of the JSON files and remake the docs. They will make
with no problems.


Review URL: http://codereview.chromium.org/9447090

TBR=cduvall@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=124675

[yzshen, 2012-03-02 17:30:04]

Here is the output of the bot:

python src/tools/checklicenses/checklicenses.py --root src
 in dir /b/build/slave/Linux/build (timeout 600 secs)
 watching logfiles {}
 argv: ['python', 'src/tools/checklicenses/checklicenses.py', '--root', 'src']
 environment:
  CHROME_HEADLESS=1
  HOME=/home/chrome-bot
  LANG=en_US.UTF-8
  LOGNAME=chrome-bot
  PAGER=cat
  PATH=/b/depot_tools:/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin
  PWD=/b/build/slave/Linux/build
 
PYTHONPATH=/b/build/site_config:/b/build/scripts:/b/build/scripts/release:/b/build/third_party:/b/build_internal/site_config:/b/build_internal/symsrc:/b/build/slave:/b/build/third_party/buildbot_7_12:/b/build/third_party/twisted_8_1:
  SHELL=/bin/bash
  SSH_CLIENT=192.168.110.65 37137 22
  SSH_CONNECTION=192.168.110.65 37137 192.168.110.15 22
  SSH_TTY=/dev/pts/1
  USER=chrome-bot
 closing stdin
 using PTY: TrueUsing base directory: src
Checking: src

'third_party/json_minify/__init__.py' has non-whitelisted license 'UNKNOWN'
'third_party/json_minify/minify_json.py' has non-whitelisted license 'UNKNOWN'

FAILED

Please read http://www.chromium.org/developers/adding-3rd-party-libraries
for more info how to handle the failure.

Please respect OWNERS of checklicenses.py. Changes violating
this requirement may be reverted.program finished with exit code 1
elapsedTime=48.039917



On Fri, Mar 2, 2012 at 9:26 AM, <yzshen@chromium.org> wrote:

> Reviewers: cduvall,
>
> Description:
> Revert 124660 - Allow comments in extension config files.
>
> Added a script to remove comments from the extension api JSON files before
> processing for the extension docs. We will now be able to comment in the
> JSON
> files, and they should be much easier to read. Also added the license
> header
> to all the JSON files.
>
> BUG=114233
> TEST=Put comments in one of the JSON files and remake the docs. They will
> make
> with no problems.
>
>
> Review URL:
http://codereview.chromium.**org/9447090<http://codereview.chromium.org/9447090>
>
> TBR=cduvall@chromium.org
>
> Please review this at
https://chromiumcodereview.**appspot.com/9582013/<https://chromiumcodereview....
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src/<http://svn.chromium.org/chrome/trunk/src/>
>
> Affected files:
>  MM    chrome/common/extensions/api/**bookmarks.json
>  M     chrome/common/extensions/api/**browserAction.json
>  M     chrome/common/extensions/api/**browsingData.json
>  MM    chrome/common/extensions/api/**chromeAuthPrivate.json
>  MM    chrome/common/extensions/api/**chromePrivate.json
>  M     chrome/common/extensions/api/**chromeosInfoPrivate.json
>  M     chrome/common/extensions/api/**contentSettings.json
>  M     chrome/common/extensions/api/**contextMenus.json
>  MM    chrome/common/extensions/api/**cookies.json
>  M     chrome/common/extensions/api/**debugger.json
>  M     chrome/common/extensions/api/**devtools.json
>  M     chrome/common/extensions/api/**devtools_api.json
>  M     chrome/common/extensions/api/**experimental.accessibility.**json
>  M     chrome/common/extensions/api/**experimental.app.json
>  M     chrome/common/extensions/api/**experimental.bookmarkManager.**json
>  M     chrome/common/extensions/api/**experimental.declarative.json
>  M     chrome/common/extensions/api/**experimental.dns.json
>  M     chrome/common/extensions/api/**experimental.downloads.json
>  MM    chrome/common/extensions/api/**experimental.extension.json
>  M     chrome/common/extensions/api/**experimental.fonts.json
>  M     chrome/common/extensions/api/**experimental.infobars.json
>  M     chrome/common/extensions/api/**experimental.input.ui.json
>  MM    chrome/common/extensions/api/**experimental.input.**
> virtualKeyboard.json
>  M     chrome/common/extensions/api/**experimental.keybinding.json
>  M     chrome/common/extensions/api/**experimental.processes.json
>  MM    chrome/common/extensions/api/**experimental.rlz.json
>  M     chrome/common/extensions/api/**experimental.serial.json
>  M     chrome/common/extensions/api/**experimental.socket.json
>  MM    chrome/common/extensions/api/**experimental.speechInput.json
>  MM    chrome/common/extensions/api/**experimental.topSites.json
>  M     chrome/common/extensions/api/**extension.json
>  MM    chrome/common/extensions/api/**fileBrowserHandler.json
>  M     chrome/common/extensions/api/**fileBrowserPrivate.json
>  MM    chrome/common/extensions/api/**history.json
>  M     chrome/common/extensions/api/**i18n.json
>  MM    chrome/common/extensions/api/**idle.json
>  M     chrome/common/extensions/api/**input.ime.json
>  MM    chrome/common/extensions/api/**inputMethodPrivate.json
>  MM    chrome/common/extensions/api/**management.json
>  M     chrome/common/extensions/api/**mediaPlayerPrivate.json
>  MM    chrome/common/extensions/api/**metricsPrivate.json
>  M     chrome/common/extensions/api/**omnibox.json
>  M     chrome/common/extensions/api/**pageAction.json
>  M     chrome/common/extensions/api/**pageActions.json
>  MM    chrome/common/extensions/api/**pageCapture.json
>  M     chrome/common/extensions/api/**permissions.json
>  M     chrome/common/extensions/api/**privacy.json
>  M     chrome/common/extensions/api/**proxy.json
>  M     chrome/common/extensions/api/**storage.json
>  M     chrome/common/extensions/api/**systemPrivate.json
>  M     chrome/common/extensions/api/**tabs.json
>  MM    chrome/common/extensions/api/**terminalPrivate.json
>  MM    chrome/common/extensions/api/**test.json
>  MM    chrome/common/extensions/api/**tts.json
>  M     chrome/common/extensions/api/**ttsEngine.json
>  MM    chrome/common/extensions/api/**types.json
>  M     chrome/common/extensions/api/**webNavigation.json
>  M     chrome/common/extensions/api/**webRequest.json
>  MM    chrome/common/extensions/api/**webSocketProxyPrivate.json
>  M     chrome/common/extensions/api/**webstorePrivate.json
>  M     chrome/common/extensions/api/**windows.json
>  M     chrome/common/extensions/docs/**build/directory.py
>  M     chrome/common/extensions/docs/**js/api_page_generator.js
>  MM    chrome/common/extensions/docs/**template/page_shell.html
>  D     third_party/json_minify/README
>  D     third_party/json_minify/**README.chromium
>  D     third_party/json_minify/__**init__.py
>  D     third_party/json_minify/**minify-sans-regexp.js
>  D     third_party/json_minify/**minify_json.py
>  M     tools/json_schema_compiler/**compiler.py
>  M     tools/json_schema_compiler/**cpp_type_generator_test.py
>  D     tools/json_schema_compiler/**json_schema.py
>  M     tools/json_schema_compiler/**model_test.py
>  M     tools/json_schema_compiler/**previewserver.py
>
>
>


-- 
Best regards,
Yuzhu Shen.

[cduvall, 2012-03-03 21:22:47]

Fixed the license headers. Sorry about that.