Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(266)

Issue 493363002: create sk_tools::Expectation class, similar to skiagm::Expectations class (Closed)

Created:
6 years, 4 months ago by epoger
Modified:
6 years, 4 months ago
Reviewers:
stephana
CC:
reviews_skia.org, mtklein, tfarina
Base URL:
https://skia.googlesource.com/skia.git@master
Project:
skia
Visibility:
Public.

Description

create sk_tools::Expectation class, similar to skiagm::Expectations class The overarching goal here is for our "gm" and "render_pictures" tools to handle image expectations/actuals in the same way, sharing the same code, so their results can be processed through a single pipeline. By adding an Expectation class within tools/image_expectations.h, similar to the Expectations class in gm/gm_expectations.h, we get one step closer to that goal. TBR=stephana Committed: https://skia.googlesource.com/skia/+/85b438dfab7f6ec1fd46dbc19215ccf2171ad6ae

Patch Set 1 : add sk_tools::Expectation class, to hold image digest and ignoreFailure boolean #

Patch Set 2 : make ImageDigest compute the bitmap hash lazily #

Patch Set 3 : properly handle ignoreFailure using sk_tools::Expectation class, with unittest to exercise #

Unified diffs Side-by-side diffs Delta from patch set Stats (+203 lines, -78 lines) Patch
M tools/PictureRenderer.cpp View 1 3 chunks +4 lines, -4 lines 0 comments Download
M tools/image_expectations.h View 1 2 5 chunks +85 lines, -19 lines 0 comments Download
M tools/image_expectations.cpp View 1 2 5 chunks +67 lines, -52 lines 0 comments Download
M tools/render_pictures_main.cpp View 1 chunk +1 line, -1 line 0 comments Download
M tools/tests/render_pictures_test.py View 1 2 3 chunks +46 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
epoger
Apologies for the TBR commit on something of this size, but I want to make ...
6 years, 4 months ago (2014-08-22 06:13:16 UTC) #1
epoger
The CQ bit was checked by epoger@google.com
6 years, 4 months ago (2014-08-22 06:13:26 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/epoger@google.com/493363002/40001
6 years, 4 months ago (2014-08-22 06:13:41 UTC) #3
commit-bot: I haz the power
Committed patchset #3 (40001) as 85b438dfab7f6ec1fd46dbc19215ccf2171ad6ae
6 years, 4 months ago (2014-08-22 06:21:39 UTC) #4
tfarina
6 years, 4 months ago (2014-08-22 12:58:14 UTC) #5
Message was sent while issue was closed.
Is there a skbug associated with this?

Powered by Google App Engine
This is Rietveld 408576698