<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Feb 11, 2016 at 9:58 PM, Mike Hommey <span dir="ltr"><<a href="mailto:mh@glandium.org" target="_blank">mh@glandium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Fri, Feb 12, 2016 at 02:55:17PM +0900, Mike Hommey wrote:<br>
> On Thu, Feb 11, 2016 at 09:10:31PM -0800, Matthew N. wrote:<br>
> > Comparing screenshots<br>
> > The simplest way to compare images is via the web UI at<br>
> > <a href="http://screenshots.mattn.ca/compare/" rel="noreferrer" target="_blank">http://screenshots.mattn.ca/compare/</a> (Example: <a href="http://bit.ly/1Qv4uWD" rel="noreferrer" target="_blank">http://bit.ly/1Qv4uWD</a><br>
> > ). Simply provide the project and revision of a base push with images,<br>
> > like the Nightly rev. from about:buildconfig, and your new revision,<br>
> > like a try push with some patches to review.<br>
><br>
> In case you're not aware, reftests have a difference analyzer, that has<br>
> additional features to yours. Check out<br>
> <a href="http://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=http://archive.mozilla.org/pub/firefox/try-builds/catlee@mozilla.com-ef2ce6ac4dc847d46a09500171fed421da02a6d9/try-linux/try_ubuntu32_vm_test-reftest-1-bm01-tests1-linux32-build1450.txt.gz&only_show_unexpected=1" rel="noreferrer" target="_blank">http://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=http://archive.mozilla.org/pub/firefox/try-builds/catlee@mozilla.com-ef2ce6ac4dc847d46a09500171fed421da02a6d9/try-linux/try_ubuntu32_vm_test-reftest-1-bm01-tests1-linux32-build1450.txt.gz&only_show_unexpected=1</a><br>
><br>
> It is based on reading the logs and looking for the data: uris of<br>
> screenshots in them, but the difference circling feature and the zoom<br>
> are most useful.<br>
><br>
> BTW, I took a look at a random comparison on the url you gave, and the<br>
> only differences I could see were on the Windows background...<br>
><br>
> <a href="http://screenshots.mattn.ca/comparisons/try/dc5a8fa40cf5/try/288bdb0dd9b4/windowsxp/1_bottomToolbox.png" rel="noreferrer" target="_blank">http://screenshots.mattn.ca/comparisons/try/dc5a8fa40cf5/try/288bdb0dd9b4/windowsxp/1_bottomToolbox.png</a><br>
<br>
</span>BTW, an image diffing tool could be a nice addition to reviewboard,<br>
although I'm not sure how often we change images.<br></blockquote><div><br></div><div>Apparently it is already there but not enabled in Review Board. Bug 1239879. <br></div></div></div></div>