Changeset 156640 in webkit
- Timestamp:
- Sep 30, 2013 1:26:47 AM (11 years ago)
- Location:
- trunk/Tools
- Files:
-
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Tools/ChangeLog
r156631 r156640 1 2013-09-30 Zan Dobersek <zdobersek@igalia.com> 2 3 [webkitpy] Make the Xvfb screen depth overridable 4 https://bugs.webkit.org/show_bug.cgi?id=122089 5 6 Reviewed by Gyuyoung Kim. 7 8 Make the Xvfb screen depth overridable by checking if the XVFB_SCREEN_DEPTH environment variable 9 was set. The default value that's used if the variable wasn't set is 24. 10 11 * Scripts/webkitpy/port/xvfbdriver.py: 12 (XvfbDriver._xvfb_screen_depth): 13 (XvfbDriver._start): 14 * Scripts/webkitpy/port/xvfbdriver_unittest.py: 15 (XvfbDriverTest.make_driver): 16 (XvfbDriverTest.test_start_no_pixel_tests): 17 (XvfbDriverTest.test_start_pixel_tests): 18 (XvfbDriverTest.test_start_arbitrary_worker_number): 19 (XvfbDriverTest.test_next_free_display): 20 (XvfbDriverTest.test_start_next_worker): 21 1 22 2013-09-29 Mark Rowe <mrowe@apple.com> 2 23 -
trunk/Tools/Scripts/webkitpy/port/xvfbdriver.py
r156475 r156640 66 66 return i 67 67 68 def _xvfb_screen_depth(self): 69 return os.environ.get('XVFB_SCREEN_DEPTH', '24') 70 68 71 def _start(self, pixel_tests, per_test_args): 69 72 self.stop() … … 75 78 self._lock_file = "/tmp/.X%d-lock" % display_id 76 79 77 run_xvfb = ["Xvfb", ":%d" % display_id, "-screen", "0", "800x600x 8", "-nolisten", "tcp"]80 run_xvfb = ["Xvfb", ":%d" % display_id, "-screen", "0", "800x600x%s" % self._xvfb_screen_depth(), "-nolisten", "tcp"] 78 81 with open(os.devnull, 'w') as devnull: 79 82 self._xvfb_process = self._port.host.executive.popen(run_xvfb, stderr=devnull) -
trunk/Tools/Scripts/webkitpy/port/xvfbdriver_unittest.py
r156475 r156640 52 52 driver = XvfbDriver(port, worker_number=worker_number, pixel_tests=True) 53 53 driver._startup_delay_secs = 0 54 driver._xvfb_screen_depth = lambda: '24' 54 55 return driver 55 56 … … 67 68 def test_start_no_pixel_tests(self): 68 69 driver = self.make_driver() 69 expected_logs = "MOCK run_command: ['ps', '-eo', 'comm,command'], cwd=None\nMOCK popen: ['Xvfb', ':0', '-screen', '0', '800x600x 8', '-nolisten', 'tcp']\n"70 expected_logs = "MOCK run_command: ['ps', '-eo', 'comm,command'], cwd=None\nMOCK popen: ['Xvfb', ':0', '-screen', '0', '800x600x24', '-nolisten', 'tcp']\n" 70 71 self.assertDriverStartSuccessful(driver, expected_logs=expected_logs, expected_display=":0") 71 72 self.cleanup_driver(driver) … … 73 74 def test_start_pixel_tests(self): 74 75 driver = self.make_driver() 75 expected_logs = "MOCK run_command: ['ps', '-eo', 'comm,command'], cwd=None\nMOCK popen: ['Xvfb', ':0', '-screen', '0', '800x600x 8', '-nolisten', 'tcp']\n"76 expected_logs = "MOCK run_command: ['ps', '-eo', 'comm,command'], cwd=None\nMOCK popen: ['Xvfb', ':0', '-screen', '0', '800x600x24', '-nolisten', 'tcp']\n" 76 77 self.assertDriverStartSuccessful(driver, expected_logs=expected_logs, expected_display=":0", pixel_tests=True) 77 78 self.cleanup_driver(driver) … … 79 80 def test_start_arbitrary_worker_number(self): 80 81 driver = self.make_driver(worker_number=17) 81 expected_logs = "MOCK run_command: ['ps', '-eo', 'comm,command'], cwd=None\nMOCK popen: ['Xvfb', ':0', '-screen', '0', '800x600x 8', '-nolisten', 'tcp']\n"82 expected_logs = "MOCK run_command: ['ps', '-eo', 'comm,command'], cwd=None\nMOCK popen: ['Xvfb', ':0', '-screen', '0', '800x600x24', '-nolisten', 'tcp']\n" 82 83 self.assertDriverStartSuccessful(driver, expected_logs=expected_logs, expected_display=":0", pixel_tests=True) 83 84 self.cleanup_driver(driver) 84 85 85 86 def test_next_free_display(self): 86 output = "Xorg /usr/bin/X :0 -auth /var/run/lightdm/root/:0 -nolisten tcp vt7 -novtswitch -background none\nXvfb Xvfb :1 -screen 0 800x600x 8-nolisten tcp"87 output = "Xorg /usr/bin/X :0 -auth /var/run/lightdm/root/:0 -nolisten tcp vt7 -novtswitch -background none\nXvfb Xvfb :1 -screen 0 800x600x24 -nolisten tcp" 87 88 executive = MockExecutive2(output) 88 89 driver = self.make_driver(executive=executive) … … 94 95 self.assertEqual(driver._next_free_display(), 1) 95 96 self.cleanup_driver(driver) 96 output = "Xvfb Xvfb :0 -screen 0 800x600x 8-nolisten tcp"97 output = "Xvfb Xvfb :0 -screen 0 800x600x24 -nolisten tcp" 97 98 executive = MockExecutive2(output) 98 99 driver = self.make_driver(executive=executive) 99 100 self.assertEqual(driver._next_free_display(), 1) 100 101 self.cleanup_driver(driver) 101 output = "Xvfb Xvfb :1 -screen 0 800x600x 8 -nolisten tcp\nXvfb Xvfb :0 -screen 0 800x600x8 -nolisten tcp\nXvfb Xvfb :3 -screen 0 800x600x8-nolisten tcp"102 output = "Xvfb Xvfb :1 -screen 0 800x600x24 -nolisten tcp\nXvfb Xvfb :0 -screen 0 800x600x24 -nolisten tcp\nXvfb Xvfb :3 -screen 0 800x600x24 -nolisten tcp" 102 103 executive = MockExecutive2(output) 103 104 driver = self.make_driver(executive=executive) … … 108 109 driver = self.make_driver() 109 110 driver._next_free_display = lambda: 0 110 expected_logs = "MOCK popen: ['Xvfb', ':0', '-screen', '0', '800x600x 8', '-nolisten', 'tcp']\n"111 expected_logs = "MOCK popen: ['Xvfb', ':0', '-screen', '0', '800x600x24', '-nolisten', 'tcp']\n" 111 112 self.assertDriverStartSuccessful(driver, expected_logs=expected_logs, expected_display=":0", pixel_tests=True) 112 113 self.cleanup_driver(driver) 113 114 driver = self.make_driver() 114 115 driver._next_free_display = lambda: 3 115 expected_logs = "MOCK popen: ['Xvfb', ':3', '-screen', '0', '800x600x 8', '-nolisten', 'tcp']\n"116 expected_logs = "MOCK popen: ['Xvfb', ':3', '-screen', '0', '800x600x24', '-nolisten', 'tcp']\n" 116 117 self.assertDriverStartSuccessful(driver, expected_logs=expected_logs, expected_display=":3", pixel_tests=True) 117 118 self.cleanup_driver(driver)
Note: See TracChangeset
for help on using the changeset viewer.