diff options
| author |  reed <reed@google.com> | 2014-12-11 07:07:37 -0800 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2014-12-11 07:07:38 -0800 |
| commit | 2ff1fcede1e9525285c5de1f35fb2dcb0fab32bd (patch) | |
| tree | d031604a9c76281aa645c56a94ada8422b5b291d /tests/PictureBBHTest.cpp | |
| parent | 8da8e9d7511c2f60855c2e3334ae0fe27365bc96 (diff) | |
Defer saves() until they're needed
patch from issue 759443006 at patchset 40001 (http://crrev.com/759443006#ps40001)
BUG=skia:
Review URL: https://codereview.chromium.org/767333002
Diffstat (limited to 'tests/PictureBBHTest.cpp')
| -rw-r--r-- | tests/PictureBBHTest.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/PictureBBHTest.cpp b/tests/PictureBBHTest.cpp index bc712f575e..562d9b5991 100644 --- a/tests/PictureBBHTest.cpp +++ b/tests/PictureBBHTest.cpp @@ -119,7 +119,8 @@ static void test_clear(skiatest::Reporter* r, SkBBHFactory* factory) { // Should be Clip - Save - Clear - Restore. // Buggy implementations might return 1 (just Clip) or 3 (Clip - Save - Restore). - REPORTER_ASSERT(r, dstPic->approximateOpCount() == 4); + // TODO: can we just search that it contains "clear"? <reed> + REPORTER_ASSERT(r, dstPic->approximateOpCount() == 4 || dstPic->approximateOpCount() == 2); } DEF_TEST(PictureBBH_Clear, r) { |