Fix lazy coding in ViaSingletonPictures.
authormtklein <mtklein@chromium.org>
Wed, 6 Apr 2016 22:01:57 +0000 (15:01 -0700)
committerCommit bot <commit-bot@chromium.org>
Wed, 6 Apr 2016 22:01:57 +0000 (15:01 -0700)
commit11064dfeb49daa64bd4e539284b9a6d1b78d6e5e
treeee5f9ae275a3e59a2ff853a25e3b846db0e5e61e
parentdeacc97bc63513b5eacaf21f858727f6e8b98ce5
Fix lazy coding in ViaSingletonPictures.

I was using SkRect::MakeLargest() as bounds, which is sort of nutso,
as that clearly is way out of bounds for how big a picture can feasibly
be, i.e. something closer to SkIRect::MakeLargest().

This was causing spurious quick rejects in drawPatch().  I didn't really
look much deeper to figure out why.  It's easy enough to just feed it the
proper bounds of the entire content.

This means patch_primitive draws correctly in sp-8888 mode.
I also noticed the GM was too small... it clipped off most of its content.
So I've made it larger.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1865143002

Review URL: https://codereview.chromium.org/1865143002
dm/DMSrcSink.cpp
gm/patch.cpp
tools/dm_flags.json
tools/dm_flags.py