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

Issue 1569823006: DM: more invariants (Closed)

Created:
4 years, 11 months ago by mtklein_C
Modified:
4 years, 11 months ago
Reviewers:
mtklein, reed1
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@master
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

DM: more invariants - add ViaPicture - run ViaPicture on most bots - run ViaSecondPicture (tests SkPictureRecorder reuse) and ViaTwice (tests caching) on some bots - extend ViaSerialization reference checking to ViaPicture, ViaSecondPicture, ViaSingletonPictures, and ViaTwice - suppress current reference check failures with --blacklist I'm open to following up on changing how those suppressions work. Passing --nocheck turns off these invariant checks, letting us debug failures with normal image diff tools. CQ_EXTRA_TRYBOTS=client.skia:Test-Win8-MSVC-ShuttleB-CPU-AVX2-x86_64-Release-Trybot BUG=skia:4769 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1569823006 Committed: https://skia.googlesource.com/skia/+/4a34ecbe352114bd8403b134eb72f6e63bec0b11

Patch Set 1 #

Patch Set 2 : --nocheck, bug #

Patch Set 3 : run twice-8888 and 2ndpic-8888 on some bots #

Unified diffs Side-by-side diffs Delta from patch set Stats (+736 lines, -60 lines) Patch
M dm/DM.cpp View 1 chunk +1 line, -0 lines 0 comments Download
M dm/DMSrcSink.h View 1 chunk +6 lines, -0 lines 0 comments Download
M dm/DMSrcSink.cpp View 1 7 chunks +52 lines, -22 lines 0 comments Download
M tools/dm_flags.json View 1 2 25 chunks +665 lines, -37 lines 0 comments Download
M tools/dm_flags.py View 1 2 3 chunks +12 lines, -1 line 0 comments Download

Messages

Total messages: 18 (11 generated)
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1569823006/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1569823006/20001
4 years, 11 months ago (2016-01-08 17:10:56 UTC) #5
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1569823006/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1569823006/40001
4 years, 11 months ago (2016-01-08 17:17:35 UTC) #8
mtklein_C
4 years, 11 months ago (2016-01-08 17:26:17 UTC) #11
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 11 months ago (2016-01-08 17:55:50 UTC) #13
reed1
lgtm
4 years, 11 months ago (2016-01-08 18:17:10 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1569823006/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1569823006/40001
4 years, 11 months ago (2016-01-08 18:18:57 UTC) #16
commit-bot: I haz the power
4 years, 11 months ago (2016-01-08 18:19:38 UTC) #18
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/4a34ecbe352114bd8403b134eb72f6e63bec0b11

Powered by Google App Engine
This is Rietveld 408576698