SkMultiPictureDocument: don't rely on SkPicture::cullRect

SkMultiPictureDocument: don't rely on SkPicture::cullRect

SkPicture::cullRect() is not guaranteed to be the same as the
bounds passed to SkPictureRecorder::beginRecording().

Committed: https://skia.googlesource.com/skia/+/59d6402ba7dae4ed1e26c155b7c446d6f7ec3527

[hal.canary, 2016-06-13 18:01:11]

Description was changed from

==========
SkMultiPictureDocument: don't rely on SkPicture::cullRect

.
==========

to

==========
SkMultiPictureDocument: don't rely on SkPicture::cullRect

.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2067473003
==========

[hal.canary, 2016-06-13 18:01:45]

halcanary@google.com changed reviewers:
+ tomhudson@google.com

[hal.canary, 2016-06-13 18:01:45]

PTAL

[hal.canary, 2016-06-13 18:39:21]

The CQ bit was checked by halcanary@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-06-13 18:39:32]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2067473003/1

[commit-bot: I haz the power, 2016-06-13 19:03:02]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-13 19:03:03]

Dry run: This issue passed the CQ dry run.

[tomhudson, 2016-06-14 15:56:17]

WHY NOT?

(rely on SkPicture::cullRect)

[hal.canary, 2016-06-14 15:58:16]

On 2016/06/14 15:56:17, tomhudson wrote:
> WHY NOT?
> 
> (rely on SkPicture::cullRect)

According to mtklein@, it is not guaranteed to be the same as the input bounds.

[tomhudson, 2016-06-14 17:23:29]

LGTM

https://codereview.chromium.org/2067473003/diff/1/src/utils/SkMultiPictureDoc...
File src/utils/SkMultiPictureDocument.cpp (right):

https://codereview.chromium.org/2067473003/diff/1/src/utils/SkMultiPictureDoc...
src/utils/SkMultiPictureDocument.cpp:77: SkSize fPageSize;
bikeshed: fCurrentPageSize?

[hal.canary, 2016-06-14 17:53:02]

Description was changed from

==========
SkMultiPictureDocument: don't rely on SkPicture::cullRect

.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2067473003
==========

to

==========
SkMultiPictureDocument: don't rely on SkPicture::cullRect

SkPicture::cullRect() is not guaranteed to be the same as the
bounds passed to SkPictureRecorder::beginRecording().
==========

[hal.canary, 2016-06-14 17:53:28]

https://codereview.chromium.org/2067473003/diff/1/src/utils/SkMultiPictureDoc...
File src/utils/SkMultiPictureDocument.cpp (right):

https://codereview.chromium.org/2067473003/diff/1/src/utils/SkMultiPictureDoc...
src/utils/SkMultiPictureDocument.cpp:77: SkSize fPageSize;
On 2016/06/14 17:23:29, tomhudson wrote:
> bikeshed: fCurrentPageSize?

Done.

[hal.canary, 2016-06-14 17:53:32]

The CQ bit was checked by halcanary@google.com

[hal.canary, 2016-06-14 17:53:33]

The patchset sent to the CQ was uploaded after l-g-t-m from tomhudson@google.com
Link to the patchset: https://codereview.chromium.org/2067473003/#ps20001
(title: "2016-06-14 (Tuesday) 13:51:32 EDT")

[commit-bot: I haz the power, 2016-06-14 17:53:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2067473003/20001

[commit-bot: I haz the power, 2016-06-14 18:06:40]

Description was changed from

==========
SkMultiPictureDocument: don't rely on SkPicture::cullRect

SkPicture::cullRect() is not guaranteed to be the same as the
bounds passed to SkPictureRecorder::beginRecording().
==========

to

==========
SkMultiPictureDocument: don't rely on SkPicture::cullRect

SkPicture::cullRect() is not guaranteed to be the same as the
bounds passed to SkPictureRecorder::beginRecording().

Committed:
https://skia.googlesource.com/skia/+/59d6402ba7dae4ed1e26c155b7c446d6f7ec3527
==========

[commit-bot: I haz the power, 2016-06-14 18:06:41]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/59d6402ba7dae4ed1e26c155b7c446d6f7ec3527