Remove serial test support from test-webkitpy.

Tools/Scripts/webkitpy/layout_tests/port/android_unittest.py

Index: Tools/Scripts/webkitpy/layout_tests/port/android_unittest.py
diff --git a/Tools/Scripts/webkitpy/layout_tests/port/android_unittest.py b/Tools/Scripts/webkitpy/layout_tests/port/android_unittest.py
index 01761c78cfb9bfa0590a9f71c220051cc55a65ab..c4052dbd9029d19bab79111533f6ad07a1b79f9f 100644
--- a/Tools/Scripts/webkitpy/layout_tests/port/android_unittest.py
+++ b/Tools/Scripts/webkitpy/layout_tests/port/android_unittest.py
@@ -123,13 +123,6 @@ class AndroidCommandsTest(unittest.TestCase):
     def make_android_commands(self, device_count, serial):
         return android.AndroidCommands(self.make_executive(device_count), serial, debug_logging=False)
 
-    # The "adb" binary with the latest version should be used.
-    def serial_test_adb_command_path(self):

[Dirk Pranke, 2014/08/09 00:00:11]

This test has unpleasant side effects and is testing a code path that

a) is already covered any time we create an AndroidCommands() object

b) shouldn't exist at all (using a static mutable value to cache the path to adb
is just wrong)

c) was originally intended to look for adb in multiple places, which we don't
actually use now.

At some point I'll clean up the other code, but this test does more harm than
good.

-        executive = self.make_executive(0)
-
-        android.AndroidCommands.set_adb_command_path_options(['path1', 'path2', 'path3'])
-        self.assertEqual('path2', android.AndroidCommands.adb_command_path(executive, debug_logging=False))
-
     # The used adb command should include the device's serial number, and get_serial() should reflect this.
     def test_adb_command_and_get_serial(self):
         android_commands = self.make_android_commands(1, '123456789ABCDEF0')