Revert of Add simple PRESUBMIT check to ensure that all files ending with .json can

Revert of Add simple PRESUBMIT check to ensure that all files ending with .json can (https://codereview.chromium.org/258473002/)

Reason for revert:
This presubmit rejects existing .json files in the tree that contain comments.
(It also doesn't provide enough information about where the json is invalid.)

Original issue's description:
> Add simple PRESUBMIT check to ensure that all files ending with .json can
> be loaded as json.
> 
> R=jochen@chromium.org, maruel@chromium.org
> BUG=366395
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=265881

TBR=jochen@chromium.org,maruel@chromium.org,iannucci@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=366395

[Yoyo Zhou, 2014-04-24 20:19:20]

Created Revert of Add simple PRESUBMIT check to ensure that all files ending
with .json can

[Yoyo Zhou, 2014-04-24 20:20:06]

Please check the CQ box if you agree with the reasons here.

[Yoyo Zhou, 2014-04-25 04:51:55]

This invalid check is preventing my CL from passing. I'm going to check the CQ
box here to revert it.

[Yoyo Zhou, 2014-04-25 04:52:03]

The CQ bit was checked by yoz@chromium.org

[commit-bot: I haz the power, 2014-04-25 08:26:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoz@chromium.org/251443005/1

[commit-bot: I haz the power, 2014-04-25 08:26:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-25 08:26:10]

Failed to apply patch for PRESUBMIT.py:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file PRESUBMIT.py
  Hunk #1 FAILED at 1062.
  Hunk #2 succeeded at 1127 (offset 1 line).
  1 out of 2 hunks FAILED -- saving rejects to file PRESUBMIT.py.rej

Patch:       PRESUBMIT.py
Index: PRESUBMIT.py
diff --git a/PRESUBMIT.py b/PRESUBMIT.py
index
8a6454358882c90b10928d80b7300d602e7f01c6..67d55784370af84d31bed60125693639449c3d1f
100644
--- a/PRESUBMIT.py
+++ b/PRESUBMIT.py
@@ -1062,20 +1062,6 @@
   return []
 
 
-def _CheckJSONParsability(input_api, output_api):
-  results = []
-  file_filter = lambda f: f.LocalPath().endswith('.json')
-  for fpath in input_api.AffectedFiles(file_filter=file_filter):
-    with open(fpath.LocalPath(), 'r') as f:
-      try:
-        input_api.json.load(f)
-      except ValueError:
-        results.append(output_api.PresubmitError(
-          "File %r does not parse as valid JSON" % (fpath.LocalPath())
-        ))
-  return results
-
-
 def _CheckJavaStyle(input_api, output_api):
   """Runs checkstyle on changed java files and returns errors if any exist."""
   original_sys_path = sys.path
@@ -1126,7 +1112,6 @@
   results.extend(_CheckForAnonymousVariables(input_api, output_api))
   results.extend(_CheckCygwinShell(input_api, output_api))
   results.extend(_CheckUserActionUpdate(input_api, output_api))
-  results.extend(_CheckJSONParsability(input_api, output_api))
 
   if any('PRESUBMIT.py' == f.LocalPath() for f in input_api.AffectedFiles()):
     results.extend(input_api.canned_checks.RunUnitTestsInDirectory(

[Yoyo Zhou, 2014-04-25 15:06:58]

Closing this - need to revert manually, since another change landed on top of
this one.