[content/test/gpu] Run pylint check of gpu tests in unittest instead of PRESUBMIT

content/test/gpu/PRESUBMIT.py

Index: content/test/gpu/PRESUBMIT.py
diff --git a/content/test/gpu/PRESUBMIT.py b/content/test/gpu/PRESUBMIT.py
index da0d24fe639a5aa9c5805fbccee6b2c38d36f2b0..2764d1a8cf81f9fcde439744c5355fe2b70ab826 100644
--- a/content/test/gpu/PRESUBMIT.py
+++ b/content/test/gpu/PRESUBMIT.py
@@ -10,11 +10,6 @@ for more details about the presubmit API built into depot_tools.
 
 import re
 
-def _PyLintChecks(input_api, output_api):
-  pylint_checks = input_api.canned_checks.GetPylint(input_api, output_api,
-          extra_paths_list=_GetPathsToPrepend(input_api), pylintrc='pylintrc')
-  return input_api.RunTests(pylint_checks)
-
 def _GetPathsToPrepend(input_api):
   current_dir = input_api.PresubmitLocalPath()
   chromium_src_dir = input_api.os_path.join(current_dir, '..', '..', '..')
@@ -57,13 +52,11 @@ def _WebGLTextExpectationsTests(input_api, output_api):
 
 def CheckChangeOnUpload(input_api, output_api):
   results = []
-  results.extend(_PyLintChecks(input_api, output_api))

[Ken Russell (switch to Gerrit), 2016/04/27 09:54:45]

Can we run all of the GPU unit tests upon upload? It's important that the lint
checks be done at this point rather than the CQ failing the CL.

[nednguyen, 2016/04/27 17:30:02]

We already did at line 61 & 67. I remember you was the one who set it this way
:-)

[Ken Russell (switch to Gerrit), 2016/04/27 17:33:28]

It wasn't obvious to me any more that "_WebGLTextExpectationsTests" ran all of
the unittests. Could you rename that function to make it more obvious what it
does? Thanks.

[nednguyen, 2016/04/27 17:42:31]

Done in https://codereview.chromium.org/1923183003

   results.extend(_WebGLTextExpectationsTests(input_api, output_api))
   return results
 
 def CheckChangeOnCommit(input_api, output_api):
   results = []
-  results.extend(_PyLintChecks(input_api, output_api))
   results.extend(_WebGLTextExpectationsTests(input_api, output_api))
   return results