Bug 972099: Set the value for preference gfx.color_management.force_srgb in the profile, rather than in the JS scripts afterwards, as it's too late by then. r=ahalberstadt
authorMilan Sreckovic <milan@mozilla.com>
Mon, 17 Feb 2014 16:19:09 -0500
changeset 169500 aa631f68a4feeeacc4345f2c631119a3f861de3f
parent 169499 9a268a4c902bdd2b2092a85ac4ae9142aeb36b2d
child 169501 1bf41db3ca502ea9a3c9eadd5e7a31f7f55483bc
push id270
push userpvanderbeken@mozilla.com
push dateThu, 06 Mar 2014 09:24:21 +0000
reviewersahalberstadt
bugs972099
milestone30.0a1
Bug 972099: Set the value for preference gfx.color_management.force_srgb in the profile, rather than in the JS scripts afterwards, as it's too late by then. r=ahalberstadt
layout/tools/reftest/runreftest.py
--- a/layout/tools/reftest/runreftest.py
+++ b/layout/tools/reftest/runreftest.py
@@ -165,16 +165,19 @@ class RefTest(object):
 
     for v in options.extraPrefs:
       thispref = v.split('=')
       if len(thispref) < 2:
         print "Error: syntax error in --setpref=" + v
         sys.exit(1)
       prefs[thispref[0]] = mozprofile.Preferences.cast(thispref[1].strip())
 
+    # We need to set this here, see bug 972099
+    prefs['gfx.color_management.force_srgb'] = True
+
     # install the reftest extension bits into the profile
     addons = []
     addons.append(os.path.join(SCRIPT_DIRECTORY, "reftest"))
 
     # I would prefer to use "--install-extension reftest/specialpowers", but that requires tight coordination with
     # release engineering and landing on multiple branches at once.
     if special_powers and (manifest.endswith('crashtests.list') or manifest.endswith('jstests.list')):
       addons.append(os.path.join(SCRIPT_DIRECTORY, 'specialpowers'))