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

Unified Diff: tests/selenium/main.py

Issue 151060: add ignore_edge option in test list. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/o3d/
Patch Set: '' Created 11 years, 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tests/selenium/javascript_unit_test_list.txt ('k') | tests/selenium/sample_list.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/selenium/main.py
===================================================================
--- tests/selenium/main.py (revision 19792)
+++ tests/selenium/main.py (working copy)
@@ -783,7 +783,9 @@
pixel_threshold = "10"
use_colorfactor = False
use_downsample = False
-
+ use_edge = True
+ edge_threshold = "5"
+
# Find out if the test specified any options relating to perceptual diff
# that will override the defaults.
for opt in test_options:
@@ -804,6 +806,10 @@
elif (opt.startswith("downsample")):
downsample_factor = selenium_utilities.GetArgument(opt)
use_downsample = True
+ elif (opt.startswith("pdiff_edge_ignore_off")):
+ use_edge = False
+ elif (opt.startswith("pdiff_edge_threshold")):
+ edge_threshold = selenium_utilities.GetArgument(opt)
# Check if file exists
if os.path.exists(generated_file):
@@ -828,7 +834,9 @@
arguments += ["-colorfactor", colorfactor]
if use_downsample:
arguments += ["-downsample", downsample_factor]
-
+ if use_edge:
+ arguments += ["-ignoreEdges", edge_threshold]
+
# Print the perceptual diff command line so we can debug easier.
if verbose:
print " ".join(arguments)
« no previous file with comments | « tests/selenium/javascript_unit_test_list.txt ('k') | tests/selenium/sample_list.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698