aboutsummaryrefslogtreecommitdiffhomepage
path: root/tools/skdiff_main.cpp
Commit message (Collapse)AuthorAge
* Move skdiff tool and add to gn build.Gravatar bungeman2016-10-14
| | | | | | | | | I really wanted this today, so I got it working again. Change-Id: I1a37d48d4806198b55c59d1df5ff15a03500195f Reviewed-on: https://skia-review.googlesource.com/3383 Commit-Queue: Ben Wagner <bungeman@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org>
* change SkStreams to work with sk_sp<SkData> instead of SkData*Gravatar reed2016-09-12
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2333713002 Review-Url: https://codereview.chromium.org/2333713002
* Move off SK_SUPPORT_LEGACY_DATA_FACTORIES.Gravatar bungeman2016-08-02
| | | | | | | This moves Skia code off of SK_SUPPORT_LEGACY_DATA_FACTORIES. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2206633004 Review-Url: https://codereview.chromium.org/2206633004
* Create output directory in skdiffGravatar brianosman2016-04-05
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1865563002 Review URL: https://codereview.chromium.org/1865563002
* Remove uses of SkImageDecoder from skdiffGravatar msarett2016-03-17
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1788643003 Review URL: https://codereview.chromium.org/1788643003
* Move SkTDArray to private.Gravatar bungeman2016-02-18
| | | | | | | TBR=reed Moving to private is good. Review URL: https://codereview.chromium.org/1707213002
* move SkTDict and SkTSearch into privateGravatar reed2016-01-03
| | | | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1558923002 TBR= Review URL: https://codereview.chromium.org/1558923002
* Style Change: NULL->nullptrGravatar halcanary2015-08-27
| | | | | | DOCS_PREVIEW= https://skia.org/?cl=1316233002 Review URL: https://codereview.chromium.org/1316233002
* Remove include of stdlib.h from SkTypes.h.Gravatar bungeman2015-08-26
| | | | | | | | | | | | | | | Unfortunately, immintrin.h (which is also included by SkTypes) includes xmmintrin.h which includes mm_malloc.h which includes stdlib.h for malloc even though, from the implementation, it is difficult to see why. Fortunately, arm_neon.h does not seem to be involved in such shenanigans, so building for Android will keep things sane. TBR=reed@google.com Doesn't change Skia API, just moves an include. Review URL: https://codereview.chromium.org/1313203003
* Move SkTemplates.h to private.Gravatar bungeman2015-08-19
| | | | | | | | SkTemplates.h contains a number of Skia specific utilities which are not designed for external use. In addition to reducing the external support burden, this will allow Skia to freely refactor this file. Review URL: https://codereview.chromium.org/1272293004
* add --listfailingbase optionGravatar reed2015-01-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/833223002
* "NULL !=" = NULLGravatar bsalomon2014-09-05
| | | | | | | | R=reed@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/544233002
* add a verbose flag to skdiff that shows the progress and status ofGravatar commit-bot@chromium.org2014-05-28
| | | | | | | | | | | | | each comparison BUG= R=bungeman@google.com Author: humper@google.com Review URL: https://codereview.chromium.org/302443012 git-svn-id: http://skia.googlecode.com/svn/trunk@14923 2bbb7eff-a529-9590-31e7-b0007b416f81
* force linking inside of skdiffGravatar zachr@google.com2013-06-18
| | | | | | Review URL: https://codereview.chromium.org/16879012 git-svn-id: http://skia.googlecode.com/svn/trunk@9669 2bbb7eff-a529-9590-31e7-b0007b416f81
* Sanitizing source files in Skia_Periodic_House_KeepingGravatar skia.committer@gmail.com2012-12-06
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@6687 2bbb7eff-a529-9590-31e7-b0007b416f81
* Update skdiff.Gravatar bungeman@google.com2012-12-05
| | | | | | | https://codereview.appspot.com/6850115/ git-svn-id: http://skia.googlecode.com/svn/trunk@6681 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fix a memory leak in skdiff that happens when theGravatar scroggo@google.com2012-11-19
| | | | | | | | | | directories do not contain the same files. Only create the bitmaps when we need to use them. Review URL: https://codereview.appspot.com/6850066 git-svn-id: http://skia.googlecode.com/svn/trunk@6481 2bbb7eff-a529-9590-31e7-b0007b416f81
* build iOS with 'make all'Gravatar caryclark@google.com2012-10-02
| | | | | | | | | | | This builds all skia tests by treating iOS tools as executable applications. A few warnings were fixed as well. Removed old trace draw code and remnants. Review URL: https://codereview.appspot.com/6597063 git-svn-id: http://skia.googlecode.com/svn/trunk@5776 2bbb7eff-a529-9590-31e7-b0007b416f81
* Shut up some Mac (xcode 3) warningsGravatar bsalomon@google.com2012-09-05
| | | | | | | | Review URL: http://codereview.appspot.com/6503053/ git-svn-id: http://skia.googlecode.com/svn/trunk@5402 2bbb7eff-a529-9590-31e7-b0007b416f81
* Sanitizing source files in Skia_Periodic_House_KeepingGravatar skia.committer@gmail.com2012-08-31
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@5373 2bbb7eff-a529-9590-31e7-b0007b416f81
* Adding checkboxes to skdiff output so a user can select a subset of images ↵Gravatar jkleinert@google.com2012-08-30
| | | | | | | | | to rebaseline. BUG=745 Review URL: https://codereview.appspot.com/6497054 git-svn-id: http://skia.googlecode.com/svn/trunk@5360 2bbb7eff-a529-9590-31e7-b0007b416f81
* skdiff: recurse over subdirectories, unless --norecurse option is givenGravatar epoger@google.com2012-08-16
| | | | | | | in anticipation of https://code.google.com/p/skia/issues/detail?id=743 ('move gm baselines outside of trunk, and modify naming convention') Review URL: https://codereview.appspot.com/6465053 git-svn-id: http://skia.googlecode.com/svn/trunk@5121 2bbb7eff-a529-9590-31e7-b0007b416f81
* Clean up most clang warnings outside animator/Gravatar tomhudson@google.com2012-08-14
| | | | | | | | http://codereview.appspot.com/6464058/ git-svn-id: http://skia.googlecode.com/svn/trunk@5079 2bbb7eff-a529-9590-31e7-b0007b416f81
* skdiff: clean up isPathAbsolute check for WindowsGravatar epoger@google.com2012-08-08
| | | | | | | This is a followup to https://codereview.appspot.com/6458046/ ('Fix skdiff when using windows path that begins with a drive letter') Review URL: https://codereview.appspot.com/6450106 git-svn-id: http://skia.googlecode.com/svn/trunk@5004 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fix skdiff when using windows path that begins with a drive letterGravatar bsalomon@google.com2012-07-30
| | | | | | | Review URL: https://mail.google.com/mail/u/0/?shva=1#inbox/138d92244ca33549 git-svn-id: http://skia.googlecode.com/svn/trunk@4838 2bbb7eff-a529-9590-31e7-b0007b416f81
* skdiff output: indicate which result types would cause failureGravatar epoger@google.com2012-07-19
| | | | | | Review URL: https://codereview.appspot.com/6419053 git-svn-id: http://skia.googlecode.com/svn/trunk@4676 2bbb7eff-a529-9590-31e7-b0007b416f81
* skdiff: cap return value at 255 to avoid automatic 'wrapping' (%256) of ↵Gravatar epoger@google.com2012-07-16
| | | | | | | | return value Review URL: https://codereview.appspot.com/6409045 git-svn-id: http://skia.googlecode.com/svn/trunk@4629 2bbb7eff-a529-9590-31e7-b0007b416f81
* skdiff: replace --failonmismatches with repeatable --failonresult optionGravatar epoger@google.com2012-07-13
| | | | | | Review URL: https://codereview.appspot.com/6392064 git-svn-id: http://skia.googlecode.com/svn/trunk@4611 2bbb7eff-a529-9590-31e7-b0007b416f81
* rename NO_ERROR to kNoError, because Windows defines a NO_ERROR macro elsewhereGravatar epoger@google.com2012-07-12
| | | | | | | TBR=rmistry Review URL: https://codereview.appspot.com/6355100 git-svn-id: http://skia.googlecode.com/svn/trunk@4581 2bbb7eff-a529-9590-31e7-b0007b416f81
* skdiff: add --failonmismatches and --listfilename options, plus cleanupGravatar epoger@google.com2012-07-12
| | | | | | | These changes are needed in order to switch the buildbots from using "gm -r" to "skdiff" to compare gm results, and should be generally good for humans too. Review URL: https://codereview.appspot.com/6392054 git-svn-id: http://skia.googlecode.com/svn/trunk@4579 2bbb7eff-a529-9590-31e7-b0007b416f81
* skdiff now has the option to not print out the directories it is using.Gravatar keyar@chromium.org2012-07-09
| | | | | | | | Further, test_pictures.py will call skdiff with this option. Review URL: https://codereview.appspot.com/6347066 git-svn-id: http://skia.googlecode.com/svn/trunk@4497 2bbb7eff-a529-9590-31e7-b0007b416f81
* Improve sorting of skdiff output, and make it consistent across platformsGravatar epoger@google.com2012-06-28
| | | | | | | BUG=https://code.google.com/p/skia/issues/detail?id=677 Review URL: https://codereview.appspot.com/6351045 git-svn-id: http://skia.googlecode.com/svn/trunk@4388 2bbb7eff-a529-9590-31e7-b0007b416f81
* skdiff: no longer skip .pdf files by defaultGravatar epoger@google.com2012-06-25
| | | | | | Review URL: https://codereview.appspot.com/6257080 git-svn-id: http://skia.googlecode.com/svn/trunk@4330 2bbb7eff-a529-9590-31e7-b0007b416f81
* fix warnings on Mac in miscellaneous filesGravatar caryclark@google.com2012-06-06
| | | | | | | | | | | | | | | | | | Fix these class of warnings: - unused functions - unused locals - sign mismatch - missing function prototypes - missing newline at end of file - 64 to 32 bit truncation The changes prefer to link in dead code in the debug build with 'if (false)' than to comment it out, but trivial cases are commented out or sometimes deleted if it appears to be a copy/paste error. Review URL: https://codereview.appspot.com/6305047 git-svn-id: http://skia.googlecode.com/svn/trunk@4186 2bbb7eff-a529-9590-31e7-b0007b416f81
* skdiff: return nonzero exit value if there are any mismatchesGravatar epoger@google.com2012-05-31
| | | | | | | This will allow us to replace "gm -r" in our buildbots with "skdiff". This will speed things up, and work around http://code.google.com/p/skia/issues/detail?id=473 ('PDF gradtext gm image results are nondeterministic') Review URL: https://codereview.appspot.com/6242071 git-svn-id: http://skia.googlecode.com/svn/trunk@4097 2bbb7eff-a529-9590-31e7-b0007b416f81
* skdiff: internally track lists of ALL result types, not just baseMissing and ↵Gravatar epoger@google.com2012-05-31
| | | | | | | | | comparisonMissing This does not change skdiff output at all; verified using tools/tests/run.sh Review URL: https://codereview.appspot.com/6260057 git-svn-id: http://skia.googlecode.com/svn/trunk@4096 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add to skdiff: test images for bitwise equality before testing for ↵Gravatar epoger@google.com2012-05-22
| | | | | | | | | | | | | | | | | | | pixel-wise equality - we now check for bitwise equality of files before bothering with pixel comparisons (this will help with formats for which we don't have good decoders, like PDF) - unparsable files are now reported as differing, unless they are bitwise equal (before, they were always reported as matching!) - "largest area mismatch" is now based on same-size image pairs only; previously, if any image pairs had mismatching size, it was 100% - removed repetitive "image size mismatch, so no diff to display" messages - changed format of leftmost table cells to be more readable BUG=http://code.google.com/p/skia/issues/detail?id=473 Review URL: https://codereview.appspot.com/6208089 git-svn-id: http://skia.googlecode.com/svn/trunk@4027 2bbb7eff-a529-9590-31e7-b0007b416f81
* Remove chromium mode from skdiff, because it is no longer actively used.Gravatar epoger@google.com2012-05-18
| | | | | | | | | Ran tools/tests/run.sh to validate this change. TBR=caryclark1 Review URL: https://codereview.appspot.com/6222045 git-svn-id: http://skia.googlecode.com/svn/trunk@4005 2bbb7eff-a529-9590-31e7-b0007b416f81
* skdiff: only write out diff images that are included in index.htmlGravatar epoger@google.com2012-05-16
| | | | | | | Not only are those files unnecessary, in some different-image-size cases, their contents are nondeterministic! Review URL: https://codereview.appspot.com/6208073 git-svn-id: http://skia.googlecode.com/svn/trunk@3975 2bbb7eff-a529-9590-31e7-b0007b416f81
* Make skdiff use enumeration of result types instead of separate booleans for ↵Gravatar epoger@google.com2012-05-16
| | | | | | | | | | | each result type. This is one step on the way to a more robust version of skdiff that we can use to address http://code.google.com/p/skia/issues/detail?id=473 ('PDF gradtext gm image results are nondeterministic') I have confirmed that skdiff results will not change, using tools/tests/run.sh. Review URL: https://codereview.appspot.com/6208063 git-svn-id: http://skia.googlecode.com/svn/trunk@3972 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add self-test for skdiff, and remove date/time from skdiff output so it passes.Gravatar epoger@google.com2012-05-15
| | | | | | | BUG=http://code.google.com/p/skia/issues/detail?id=612 Review URL: https://codereview.appspot.com/6198074 git-svn-id: http://skia.googlecode.com/svn/trunk@3938 2bbb7eff-a529-9590-31e7-b0007b416f81
* Just use std qsortGravatar reed@google.com2012-05-07
| | | | | | Review URL: https://codereview.appspot.com/6195052 git-svn-id: http://skia.googlecode.com/svn/trunk@3846 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add new flags to skdiffGravatar epoger@google.com2012-05-01
| | | | | | | | | | | | | | New flags are: --nodiffs: don't write out image diffs or index.html, just generate report on stdout --match: compare files whose filenames contain this substring; if unspecified, compare ALL files this flag may be repeated to add more matching substrings --nomatch: regardless of --match, DO NOT compare files whose filenames contain this substring this flag may be repeated to add more forbidden substrings Also implemented the --threshold flag, which was already documented but not implemented. Review URL: https://codereview.appspot.com/6135045 git-svn-id: http://skia.googlecode.com/svn/trunk@3806 2bbb7eff-a529-9590-31e7-b0007b416f81
* If skdiff encounters image size mismatch, do not display diff imagesGravatar epoger@google.com2012-04-12
| | | | | | | Currently, if skdiff encounters images whose expected and actual dimensions do not match, it will attempt (incorrectly) to display diffs. Better to not display the diffs at all than to display misleading diffs... Review URL: https://codereview.appspot.com/6015049 git-svn-id: http://skia.googlecode.com/svn/trunk@3662 2bbb7eff-a529-9590-31e7-b0007b416f81
* landing patch from http://codereview.appspot.com/5874051/Gravatar epoger@google.com2012-03-22
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@3468 2bbb7eff-a529-9590-31e7-b0007b416f81
* Make skdiff always generate white diffs alongside actual pixel diffsGravatar epoger@google.com2012-02-02
| | | | | | | (and some associated improvement to the output) Review URL: https://codereview.appspot.com/5616059 git-svn-id: http://skia.googlecode.com/svn/trunk@3138 2bbb7eff-a529-9590-31e7-b0007b416f81
* Make skdiff robust to image size differences: if the two image sizes differ,Gravatar tomhudson@google.com2011-11-30
| | | | | | | | | report that fact explicitly, record 100% pixel difference, and don't try to generate a valid delta image. git-svn-id: http://skia.googlecode.com/svn/trunk@2767 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fix more VS2010 warnings.Gravatar bsalomon@google.com2011-10-07
| | | | | | | | Review URL: http://codereview.appspot.com/5169053/ git-svn-id: http://skia.googlecode.com/svn/trunk@2432 2bbb7eff-a529-9590-31e7-b0007b416f81
* Make skdiff paths work on windowsGravatar bsalomon@google.com2011-09-23
| | | | | | | | http://codereview.appspot.com/5111041/ git-svn-id: http://skia.googlecode.com/svn/trunk@2321 2bbb7eff-a529-9590-31e7-b0007b416f81
* Move SkBitmap members of the skdiff DiffRecord to pointers so that theyGravatar tomhudson@google.com2011-08-02
| | | | | | | | | can be deleted as soon as they're written. When we're running in --chromium mode we now have much larger datasets. git-svn-id: http://skia.googlecode.com/svn/trunk@2025 2bbb7eff-a529-9590-31e7-b0007b416f81