Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(69)

Side by Side Diff: content/test/gpu/gpu_tests/cloud_storage_integration_test_base.py

Issue 2794713002: Use separate pixel reference images for windows 10 and 7 (Closed)
Patch Set: replace os-type string Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | content/test/gpu/gpu_tests/pixel_expectations.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 """Base classes for a test which uploads results (reference images, 5 """Base classes for a test which uploads results (reference images,
6 error images) to cloud storage.""" 6 error images) to cloud storage."""
7 7
8 import logging 8 import logging
9 import os 9 import os
10 import re 10 import re
(...skipping 229 matching lines...) Expand 10 before | Expand all | Expand 10 after
240 ('disable_multisample_render_to_texture' in 240 ('disable_multisample_render_to_texture' in
241 system_info.gpu.driver_bug_workarounds)) 241 system_info.gpu.driver_bug_workarounds))
242 params.model_name = system_info.model_name 242 params.model_name = system_info.model_name
243 243
244 @classmethod 244 @classmethod
245 def _FormatGpuInfo(cls, tab): 245 def _FormatGpuInfo(cls, tab):
246 cls._ComputeGpuInfo(tab) 246 cls._ComputeGpuInfo(tab)
247 params = cls._reference_image_parameters 247 params = cls._reference_image_parameters
248 msaa_string = '_msaa' if params.msaa else '_non_msaa' 248 msaa_string = '_msaa' if params.msaa else '_non_msaa'
249 if params.vendor_id: 249 if params.vendor_id:
250 os_type = cls.GetParsedCommandLineOptions().os_type
251 if str(cls.browser.platform.GetOSVersionName()).lower() == "win10":
252 # Allow separate baselines for Windows 10 and Windows 7.
253 os_type = "win10"
250 return '%s_%04x_%04x%s' % ( 254 return '%s_%04x_%04x%s' % (
251 cls.GetParsedCommandLineOptions().os_type, params.vendor_id, 255 os_type, params.vendor_id, params.device_id, msaa_string)
252 params.device_id, msaa_string)
253 else: 256 else:
254 # This is the code path for Android devices. Include the model 257 # This is the code path for Android devices. Include the model
255 # name (e.g. "Nexus 9") in the GPU string to disambiguate 258 # name (e.g. "Nexus 9") in the GPU string to disambiguate
256 # multiple devices on the waterfall which might have the same 259 # multiple devices on the waterfall which might have the same
257 # device string ("NVIDIA Tegra") but different screen 260 # device string ("NVIDIA Tegra") but different screen
258 # resolutions and device pixel ratios. 261 # resolutions and device pixel ratios.
259 return '%s_%s_%s_%s%s' % ( 262 return '%s_%s_%s_%s%s' % (
260 cls.GetParsedCommandLineOptions().os_type, 263 cls.GetParsedCommandLineOptions().os_type,
261 params.vendor_string, params.device_string, 264 params.vendor_string, params.device_string,
262 params.model_name, msaa_string) 265 params.model_name, msaa_string)
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
353 except Exception: 356 except Exception:
354 # An exception raised from self.fail() indicates a failure. 357 # An exception raised from self.fail() indicates a failure.
355 image_name = self._UrlToImageName(url) 358 image_name = self._UrlToImageName(url)
356 if self.GetParsedCommandLineOptions().test_machine_name: 359 if self.GetParsedCommandLineOptions().test_machine_name:
357 self._UploadErrorImagesToCloudStorage(image_name, screenshot, None) 360 self._UploadErrorImagesToCloudStorage(image_name, screenshot, None)
358 else: 361 else:
359 self._WriteErrorImages( 362 self._WriteErrorImages(
360 self.GetParsedCommandLineOptions().generated_dir, image_name, 363 self.GetParsedCommandLineOptions().generated_dir, image_name,
361 screenshot, None) 364 screenshot, None)
362 raise 365 raise
OLDNEW
« no previous file with comments | « no previous file | content/test/gpu/gpu_tests/pixel_expectations.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698