History log of /external/skia/tools/skpdiff/skpdiff_main.cpp
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
4808757d7a8bbe8c773a312a894738e9ff701b5f 02-Apr-2015 borenet <borenet@google.com> Remove all code related to NaCl

BUG=skia:3600
DOCS_PREVIEW= https://skia.org/?cl=1036283002

Review URL: https://codereview.chromium.org/1036283002
/external/skia/tools/skpdiff/skpdiff_main.cpp
6b87df2997980218308954ade47df7fdd6680bc7 06-Oct-2014 tfarina <tfarina@chromium.org> Fix usage of SK_BUILD_* defines.

Since we just 'define' them, but not attribute anything to them, like
'1' for example, cpp expands it to nothing and that breaks the "#if"
clauses.

To fix that, uses "#if defined(...)" which will correctly check if your
macro name was defined or not.

BUG=skia:2850
TEST=make most
R=robertphillips@google.com

Review URL: https://codereview.chromium.org/628763005
/external/skia/tools/skpdiff/skpdiff_main.cpp
49f085dddff10473b6ebf832a974288300224e60 05-Sep-2014 bsalomon <bsalomon@google.com> "NULL !=" = NULL

R=reed@google.com

Author: bsalomon@google.com

Review URL: https://codereview.chromium.org/544233002
/external/skia/tools/skpdiff/skpdiff_main.cpp
406654be7a930b484159f5bca107d3b11d8a9ede 04-Sep-2014 mtklein <mtklein@chromium.org> SkThreadPool ~~> SkTaskGroup

SkTaskGroup is like SkThreadPool except the threads stay in
one global pool. Each SkTaskGroup itself is tiny (4 bytes)
and its wait() method applies only to tasks add()ed to that
instance, not the whole thread pool.

This means we don't need to bring up new thread pools when
tests themselves want to use multithreading (e.g. pathops,
quilt). We just create a new SkTaskGroup and wait for that
to complete. This should be more efficient, and allow us
to expand where we use threads to really latency sensitive
places. E.g. we can probably now use these in nanobench
for CPU .skp rendering.

Now that all threads are sharing the same pool, I think we
can remove most of the custom mechanism pathops tests use
to control threading. They'll just ride on the global pool
with all other tests now.

This (temporarily?) removes the GPU multithreading feature
from DM, which we don't use.

On my desktop, DM runs a little faster (57s -> 55s) in
Debug, and a lot faster in Release (36s -> 24s). The bots
show speedups of similar proportions, cutting more than a
minute off the N4/Release and Win7/Debug runtimes.

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/9c7207b5dc71dc5a96a2eb107d401133333d5b6f

R=caryclark@google.com, bsalomon@google.com, bungeman@google.com, mtklein@google.com, reed@google.com

Author: mtklein@chromium.org

Review URL: https://codereview.chromium.org/531653002
/external/skia/tools/skpdiff/skpdiff_main.cpp
21b342d19c71fa5abe7e4aa5cfb518fd04cb9d67 13-Aug-2014 stephana <stephana@google.com> This eliminates the need to copy the generated images from a temporary directory to the directory that is served by the rebaseline_server.

BUG=skia:2815, skia:2818
R=epoger@google.com

Author: stephana@google.com

Review URL: https://codereview.chromium.org/457203003
/external/skia/tools/skpdiff/skpdiff_main.cpp
54f1ad8bb5bdd2ac2ea7981427abeb193383d449 02-Jul-2014 epoger <epoger@google.com> rebaseline_server: use just skpdiff, not Python Image Library

BUG=skia:2414
R=djsollen@google.com, borenet@google.com

Author: epoger@google.com

Review URL: https://codereview.chromium.org/325413003
/external/skia/tools/skpdiff/skpdiff_main.cpp
513a7bffd344a2bba6e014ec08838ea0bbb8aa68 07-Nov-2013 djsollen@google.com <djsollen@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> update skpdiff visualization (image magnification with alpha mask)

R=epoger@google.com

Review URL: https://codereview.chromium.org/29103005

git-svn-id: http://skia.googlecode.com/svn/trunk@12174 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/tools/skpdiff/skpdiff_main.cpp
cbbf1ca304d35e3acd944609cf7a1c5127d0ca56 16-Oct-2013 djsollen@google.com <djsollen@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> add initial support ofr multi-threaded execution

Review URL: https://codereview.chromium.org/27541003

git-svn-id: http://skia.googlecode.com/svn/trunk@11823 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/tools/skpdiff/skpdiff_main.cpp
d2d9f563be80a6169380b6ac0d2b0306688817e4 23-Jul-2013 edisonn@google.com <edisonn@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> skpdiff: remove default value for output. Diifs can be very big, and fill a hard drive, no sense to generate them if not needed.

R=zachr@google.com

Review URL: https://codereview.chromium.org/20054003

git-svn-id: http://skia.googlecode.com/svn/trunk@10291 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/tools/skpdiff/skpdiff_main.cpp
5b3e2b96227f21d9643ca1678d777fd23767756d 22-Jul-2013 zachr@google.com <zachr@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> move skpdiff into tools

R=djsollen@google.com

Review URL: https://codereview.chromium.org/19608005

git-svn-id: http://skia.googlecode.com/svn/trunk@10249 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/tools/skpdiff/skpdiff_main.cpp
904f86e6aae156df06c2d1a02cf7e91ecfda5d86 22-Jul-2013 zachr@google.com <zachr@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> Revert "migrate skpdiff to tools"

This reverts commit 58a7ec70ce6b4548793a2ddc37c63d3d44edbc36.

git-svn-id: http://skia.googlecode.com/svn/trunk@10226 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/tools/skpdiff/skpdiff_main.cpp
a54aaf7fba0dae99305715b8f9d46a875eb6b5a6 22-Jul-2013 zachr@google.com <zachr@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> migrate skpdiff to tools

R=djsollen@google.com

Review URL: https://codereview.chromium.org/19671002

git-svn-id: http://skia.googlecode.com/svn/trunk@10225 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/tools/skpdiff/skpdiff_main.cpp