Move browser_test_runner.LoadAllTestsInModule to serially_executed_browser_test_case

telemetry/examples/browser_tests/failed_tests.py

Index: telemetry/examples/browser_tests/failed_tests.py
diff --git a/telemetry/examples/browser_tests/failed_tests.py b/telemetry/examples/browser_tests/failed_tests.py
index 3c9a1b9a98970bb83b606c0030e6c0ef18da6798..42c00dcb63e948fc92640237487166c142cafe60 100644
--- a/telemetry/examples/browser_tests/failed_tests.py
+++ b/telemetry/examples/browser_tests/failed_tests.py
@@ -2,6 +2,7 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
+import sys
 
 from telemetry.testing import serially_executed_browser_test_case
 
@@ -38,3 +39,9 @@ class TearDownClassFailedTest(
 
   def DummyTest(self):
     pass
+
+
+def load_tests(loader, tests, pattern):
+  del loader, tests, pattern  # Unused.

[Dirk Pranke, 2016/12/11 03:23:01]

Nit: maybe use a pylint annotation here instead?

[nednguyen, 2016/12/11 13:01:44]

This is our preferred pattern: go/del-for-unused (Googler only, sorry)

+  return serially_executed_browser_test_case.LoadAllTestsInModule(
+      sys.modules[__name__])