Commit 6eff7a08 authored by jamesr@google.com's avatar jamesr@google.com

[chromium] Chromium GPU layout tests should use the normal process number...

[chromium] Chromium GPU layout tests should use the normal process number selection, not force 1 worker
https://bugs.webkit.org/show_bug.cgi?id=66603

Reviewed by Dirk Pranke.

The various ChromiumGpu ports should use the same child worker
selection logic as other ports so they can run with full
parallelization on multicore systems.

* Scripts/webkitpy/layout_tests/port/chromium_gpu.py:
* Scripts/webkitpy/layout_tests/port/chromium_gpu_unittest.py:

git-svn-id: svn://svn.chromium.org/blink/trunk@93444 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent 07c3d803
2011-08-19 James Robinson <jamesr@chromium.org>
[chromium] Chromium GPU layout tests should use the normal process number selection, not force 1 worker
https://bugs.webkit.org/show_bug.cgi?id=66603
Reviewed by Dirk Pranke.
The various ChromiumGpu ports should use the same child worker
selection logic as other ports so they can run with full
parallelization on multicore systems.
* Scripts/webkitpy/layout_tests/port/chromium_gpu.py:
* Scripts/webkitpy/layout_tests/port/chromium_gpu_unittest.py:
2011-08-19 Alok Priyadarshi <alokp@chromium.org> 2011-08-19 Alok Priyadarshi <alokp@chromium.org>
[chromium] Remove dependency of WebThemeControlDRTWin on skia::PlatformCanvas [chromium] Remove dependency of WebThemeControlDRTWin on skia::PlatformCanvas
...@@ -95,9 +95,6 @@ class ChromiumGpuLinuxPort(chromium_linux.ChromiumLinuxPort): ...@@ -95,9 +95,6 @@ class ChromiumGpuLinuxPort(chromium_linux.ChromiumLinuxPort):
return (map(self._webkit_baseline_path, ['chromium-gpu-linux', 'chromium-gpu-win', 'chromium-gpu']) + return (map(self._webkit_baseline_path, ['chromium-gpu-linux', 'chromium-gpu-win', 'chromium-gpu']) +
chromium_linux.ChromiumLinuxPort.baseline_search_path(self)) chromium_linux.ChromiumLinuxPort.baseline_search_path(self))
def default_child_processes(self):
return 1
def tests(self, paths): def tests(self, paths):
return _tests(self, paths) return _tests(self, paths)
...@@ -111,9 +108,6 @@ class ChromiumGpuMacPort(chromium_mac.ChromiumMacPort): ...@@ -111,9 +108,6 @@ class ChromiumGpuMacPort(chromium_mac.ChromiumMacPort):
return (map(self._webkit_baseline_path, ['chromium-gpu-mac', 'chromium-gpu']) + return (map(self._webkit_baseline_path, ['chromium-gpu-mac', 'chromium-gpu']) +
chromium_mac.ChromiumMacPort.baseline_search_path(self)) chromium_mac.ChromiumMacPort.baseline_search_path(self))
def default_child_processes(self):
return 1
def tests(self, paths): def tests(self, paths):
return _tests(self, paths) return _tests(self, paths)
...@@ -127,8 +121,5 @@ class ChromiumGpuWinPort(chromium_win.ChromiumWinPort): ...@@ -127,8 +121,5 @@ class ChromiumGpuWinPort(chromium_win.ChromiumWinPort):
return (map(self._webkit_baseline_path, ['chromium-gpu-win', 'chromium-gpu']) + return (map(self._webkit_baseline_path, ['chromium-gpu-win', 'chromium-gpu']) +
chromium_win.ChromiumWinPort.baseline_search_path(self)) chromium_win.ChromiumWinPort.baseline_search_path(self))
def default_child_processes(self):
return 1
def tests(self, paths): def tests(self, paths):
return _tests(self, paths) return _tests(self, paths)
...@@ -67,7 +67,6 @@ class ChromiumGpuTest(unittest.TestCase): ...@@ -67,7 +67,6 @@ class ChromiumGpuTest(unittest.TestCase):
port = chromium_gpu.get(port_name=port_name, options=mock_options) port = chromium_gpu.get(port_name=port_name, options=mock_options)
self.assertTrue(port._options.accelerated_compositing) self.assertTrue(port._options.accelerated_compositing)
self.assertTrue(port._options.accelerated_2d_canvas) self.assertTrue(port._options.accelerated_2d_canvas)
self.assertEqual(port.default_child_processes(), 1)
self.assertEqual(port._options.builder_name, 'foo - GPU') self.assertEqual(port._options.builder_name, 'foo - GPU')
self.assertTrue(port.name().startswith(port_name)) self.assertTrue(port.name().startswith(port_name))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment