SPv2: Paint SVG images and self-contained recorded pictures with their own trees.

SPv2: Paint SVG images and self-contained recorded pictures with their own trees.

A followup patch will implement flattening the trees into a single display list.

BUG=665259
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2632823002
Cr-Commit-Position: refs/heads/master@{#444199}
Committed: https://chromium.googlesource.com/chromium/src/+/d1f880de459aec1ee63f170addd7fd3e18cf85fd

[chrishtr, 2017-01-14 00:51:40]

Description was changed from

==========
none


none


none


none

BUG=
==========

to

==========
none


none


none


none

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[chrishtr, 2017-01-14 00:57:23]

Description was changed from

==========
none


none


none


none

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
SPv2: Paint SVG images and self-contained recorded pictures with their own
trees.

A followup patch will implement flattening the trees into a single display list.

BUG=665259
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[chrishtr, 2017-01-14 00:57:27]

The CQ bit was checked by chrishtr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-14 00:57:39]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[chrishtr, 2017-01-14 00:57:39]

chrishtr@chromium.org changed reviewers:
+ pdr@chromium.org

[chrishtr, 2017-01-14 16:22:57]

The CQ bit was checked by chrishtr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-14 16:23:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-14 17:49:50]

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

[commit-bot: I haz the power, 2017-01-14 17:49:51]

Dry run: This issue passed the CQ dry run.

[pdr., 2017-01-17 21:48:00]

LGTM

https://codereview.chromium.org/2632823002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/svg/graphics/SVGImage.cpp (right):

https://codereview.chromium.org/2632823002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/svg/graphics/SVGImage.cpp:392: // SVG images
paint into their own property tree set that is distinct
Could you move this into SkPictureBuilder?

https://codereview.chromium.org/2632823002/diff/20001/third_party/WebKit/Sour...
File
third_party/WebKit/Source/platform/graphics/paint/TransformPaintPropertyNode.h
(right):

https://codereview.chromium.org/2632823002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/paint/TransformPaintPropertyNode.h:30:
// This node is really a sentinel, and does not represent a real transform
I'll be removing these soon.

[chrishtr, 2017-01-17 22:20:26]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2017-01-17 22:20:27]

https://codereview.chromium.org/2632823002/diff/20001/third_party/WebKit/Sour...
File
third_party/WebKit/Source/platform/graphics/paint/TransformPaintPropertyNode.h
(right):

https://codereview.chromium.org/2632823002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/paint/TransformPaintPropertyNode.h:30:
// This node is really a sentinel, and does not represent a real transform
On 2017/01/17 at 21:48:00, pdr. wrote:
> I'll be removing these soon.

ACK!!!!!!!!!!!

[commit-bot: I haz the power, 2017-01-17 22:20:59]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-18 00:18:26]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484691626539680,
"parent_rev": "d41aee024940ce81f8f63f245a97da89e6771668", "commit_rev":
"d1f880de459aec1ee63f170addd7fd3e18cf85fd"}

[commit-bot: I haz the power, 2017-01-18 00:19:11]

Description was changed from

==========
SPv2: Paint SVG images and self-contained recorded pictures with their own
trees.

A followup patch will implement flattening the trees into a single display list.

BUG=665259
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
SPv2: Paint SVG images and self-contained recorded pictures with their own
trees.

A followup patch will implement flattening the trees into a single display list.

BUG=665259
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2632823002
Cr-Commit-Position: refs/heads/master@{#444199}
Committed:
https://chromium.googlesource.com/chromium/src/+/d1f880de459aec1ee63f170addd7...
==========

[commit-bot: I haz the power, 2017-01-18 00:19:12]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/d1f880de459aec1ee63f170addd7...