bug 815617 - speed up reftest-analyzer by avoiding use of a large dilation radius. r=bz
authorJonathan Kew <jkew@mozilla.com>
Wed, 28 Nov 2012 09:04:13 +0000
changeset 114345 9d18ded603301309aaebc9261db21dd941e16762
parent 114344 7d678d658159f4ebe557072e1510ffc097949f80
child 114346 72e123c2602da3920a17c3983f572f4c5524dd2d
push id18709
push userjkew@mozilla.com
push dateWed, 28 Nov 2012 09:04:28 +0000
treeherdermozilla-inbound@f1df34df5f3b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbz
bugs815617
milestone20.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
bug 815617 - speed up reftest-analyzer by avoiding use of a large dilation radius. r=bz
layout/tools/reftest/reftest-analyzer.xhtml
--- a/layout/tools/reftest/reftest-analyzer.xhtml
+++ b/layout/tools/reftest/reftest-analyzer.xhtml
@@ -543,24 +543,24 @@ function show_pixelinfo(x, y, pix1rgb, p
             <feFuncR type="linear" slope="-1" intercept="1" />
             <feFuncG type="linear" slope="-1" intercept="1" />
             <feFuncB type="linear" slope="-1" intercept="1" />
           </feComponentTransfer>
           <!-- c will be nonblack (and fully on) for every pixel+component where there are differences -->
           <feComposite result="c" in="c1" in2="c2" operator="arithmetic" k2="255" k3="255" />
           <!-- a will be opaque for every pixel with differences and transparent for all others -->
           <feColorMatrix result="a" type="matrix" values="0 0 0 0 0  0 0 0 0 0  0 0 0 0 0  1 1 1 0 0" />
-          
-          <!-- a, dilated by 4 pixels -->
-          <feMorphology result="dila4" in="a" operator="dilate" radius="4" />
+
           <!-- a, dilated by 1 pixel -->
           <feMorphology result="dila1" in="a" operator="dilate" radius="1" />
-          
-          <!-- all the pixels in the 3-pixel dilation of a but not in the 1-pixel dilation of a, to highlight the diffs -->
-          <feComposite result="highlight" in="dila4" in2="dila1" operator="out" />
+          <!-- a, dilated by 2 pixels -->
+          <feMorphology result="dila2" in="dila1" operator="dilate" radius="1" />
+
+          <!-- all the pixels in the 2-pixel dilation of a but not in the 1-pixel dilation, to highlight the diffs -->
+          <feComposite result="highlight" in="dila2" in2="dila1" operator="out" />
 
           <feFlood result="red" flood-color="red" />
           <feComposite result="redhighlight" in="red" in2="highlight" operator="in" />
           <feFlood result="black" flood-color="black" flood-opacity="0.5" />
           <feMerge>
             <feMergeNode in="black" />
             <feMergeNode in="redhighlight" />
           </feMerge>