[Android] Lint the rest of pylib.

build/android/pylib/host_driven/test_case.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.
 
 """Base class for host-driven test cases.
 
 This test case is intended to serve as the base class for any host-driven
 test cases. It is similar to the Python unitttest module in that test cases
 inherit from this class and add methods which will be run as tests.
 
 When a HostDrivenTestCase object is instantiated, its purpose is to run only one
 test method in the derived class. The test runner gives it the name of the test
 method the instance will run. The test runner calls SetUp with the device ID
 which the test method will run against. The test runner runs the test method
 itself, collecting the result, and calls TearDown.
 
 Tests can perform arbitrary Python commands and asserts in test methods. Tests
 that run instrumentation tests can make use of the _RunJavaTestFilters helper
 function to trigger Java tests and convert results into a single host-driven
 test result.
 """
 
 import logging
 import os
 import time
 
 from pylib import android_commands
+from pylib import constants
 from pylib import forwarder
 from pylib import valgrind_tools
 from pylib.base import base_test_result
 from pylib.instrumentation import test_package
 from pylib.instrumentation import test_result
 from pylib.instrumentation import test_runner
 
 # aka the parent of com.google.android
 BASE_ROOT = 'src' + os.sep
 
@@ skipping 38 matching lines @@
     self.push_deps = push_deps
     self.cleanup_test_files = cleanup_test_files
     if ports_to_forward:
       self.ports_to_forward = ports_to_forward
 
   def TearDown(self):
     pass
 
   # TODO(craigdh): Remove GetOutDir once references have been removed
   # downstream.
-  def GetOutDir(self):
+  @staticmethod
+  def GetOutDir():
     return constants.GetOutDirectory()
 
   def Run(self):
     logging.info('Running host-driven test: %s', self.tagged_name)
     # Get the test method on the derived class and execute it
     return getattr(self, self.test_name)()
 
   @staticmethod
   def __GetHostForwarderLog():
     return ('-- Begin Full HostForwarder log\n'
@@ skipping 86 matching lines @@
     overall_result.AddResult(
         test_result.InstrumentationTestResult(
             self.tagged_name, test_type, start_ms, duration_ms, log=log))
     return overall_result
 
   def __str__(self):
     return self.tagged_name
 
   def __repr__(self):
     return self.tagged_name