Add a presubmit script and pylintrc for content/test/gpu

content/test/gpu/gpu_tests/test_expectations.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 fnmatch
 import urlparse
 
 # Valid expectation conditions are:
 #
 # Operating systems:
 #     win, xp, vista, win7, win8, win10, mac, leopard, snowleopard,
 #     lion, mountainlion, mavericks, yosemite, linux, chromeos,
 #     android
 #
 # Browser types:
 #     android-webview-shell, android-content-shell, debug, release
 #
 # Sample usage in SetExpectations in subclasses:
 #   self.Fail('gl-enable-vertex-attrib.html',
 #       ['mac', 'release'], bug=123)
 
 WIN_CONDITIONS = ['xp', 'vista', 'win7', 'win8', 'win10']
 MAC_CONDITIONS = ['leopard', 'snowleopard', 'lion', 'mountainlion',
                  'mavericks', 'yosemite']
 
 OS_CONDITIONS = ['win', 'mac', 'linux', 'chromeos', 'android'] + \
                 WIN_CONDITIONS + MAC_CONDITIONS
 
 BROWSER_TYPE_CONDITIONS = [
-    'android-webview-shell', 'android-content-shell', 'debug', 'release' ]
+    'android-webview-shell', 'android-content-shell', 'debug', 'release']
 
 class Expectation(object):
   """Represents a single test expectation for a page.
 
   Supports conditions based on operating system (e.g., win, mac) and
   browser type (e.g. 'debug', 'release').
 
   Subclass this class and call super.__init__ last in your constructor
   in order to add new user-defined conditions. The conditions are
   parsed at the end of this class's constructor, so be careful not to
@@ skipping 94 matching lines @@
 
   def _HasWildcardCharacters(self, input_string):
     # Could make this more precise.
     return '*' in input_string or '+' in input_string
 
   def _BuildExpectationsCache(self, browser, page):
     # Turn off name matching while building the cache.
     self._skip_matching_names = True
     for e in self._expectations:
       if self.ExpectationAppliesToPage(e, browser, page):
-        if (self._HasWildcardCharacters(e.pattern)):
+        if self._HasWildcardCharacters(e.pattern):
           self._expectations_with_wildcards.append(e)
         else:
           if e.pattern in self._expectations_by_pattern:
-            print "WARNING: Non-wildcard pattern collision for", pattern
+            print "WARNING: Non-wildcard pattern collision for", e.pattern
           self._expectations_by_pattern[e.pattern] = e
     self._built_expectation_cache = True
     self._skip_matching_names = False
 
   def _GetNormalizedURL(self, url, browser):
     # Telemetry uses backslashes in its file:// URLs on Windows,
     # breaking matching of test expectations.
     if not browser.platform.GetOSName() == 'win':
       return url
     return url.replace('\\', '/')
 
   def _GetURLPath(self, url):
     components = urlparse.urlsplit(url)
     # For compatibility, the file:// scheme must be treated specially.
     # The top-level directory shows up in the netloc portion of the URL.
     if components[0] == 'file':
       url_path = components[1] + components[2]
     else:
       url_path = components[2]
     # Chop any leading slash since the expectations used by this class
     # assume that.
-    if (url_path and url_path[0] == '/'):
+    if url_path and url_path[0] == '/':
       url_path = url_path[1:]
     # Python's urlsplit doesn't seem to handle query arguments for
     # file:// URLs properly. Split them off manually.
     query_index = url_path.find('?')
     if query_index > 0:
       url_path = url_path[0:query_index]
     return url_path
 
   def _GetExpectationObjectForPage(self, browser, page):
     if not self._built_expectation_cache:
@@ skipping 71 matching lines @@
     platform = browser.platform
     os_matches = (not expectation.os_conditions or
         platform.GetOSName() in expectation.os_conditions or
         platform.GetOSVersionName() in expectation.os_conditions)
 
     browser_matches = (
       (not expectation.browser_conditions) or
       browser.browser_type in expectation.browser_conditions)
 
     return os_matches and browser_matches