Use SVGResources in SVGLayoutTreeAsText writeResources

Use SVGResources in SVGLayoutTreeAsText writeResources

As the FIXME suggests, this should better reflect the actual state of
the resources resolution by including the cycle-solving step.

Some tests (recursive-clip/mask, clip-path-recursive-call-by-child)
needs to be updated to reflect the change to actually show cycles
having been broken. The rests of the tests are updated to reflect
that LayoutSVGInlineText can't have resources.

BUG=454767
Review-Url: https://codereview.chromium.org/2763363002
Cr-Commit-Position: refs/heads/master@{#459119}
Committed: https://chromium.googlesource.com/chromium/src/+/0e50a38b387adde5829f18e549917fea3a06a494

[fs, 2017-03-22 15:44:09]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-22 15:44:26]

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-03-22 17:25:02]

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

[commit-bot: I haz the power, 2017-03-22 17:25:03]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[fs, 2017-03-22 18:52:16]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-22 18:52:55]

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

[fs, 2017-03-22 19:01:23]

Description was changed from

==========
Use SVGResources in SVGLayoutTreeAsText writeResources

As the FIXME suggests, this should better reflect the actual state of
the resources resolution by including the cycle-solving step.

BUG=454767
==========

to

==========
Use SVGResources in SVGLayoutTreeAsText writeResources

As the FIXME suggests, this should better reflect the actual state of
the resources resolution by including the cycle-solving step.

Some tests (recursive-clip/mask, clip-path-recursive-call-by-child)
needs to be updated to reflect the change to actually show cycles
having been broken. The rests of the tests are updated to reflect
that LayoutSVGInlineText can't have resources.

BUG=454767
==========

[fs, 2017-03-22 20:02:37]

fs@opera.com changed reviewers:
+ pdr@chromium.org, schenney@chromium.org

[commit-bot: I haz the power, 2017-03-22 20:51:54]

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

[commit-bot: I haz the power, 2017-03-22 20:51:55]

Dry run: This issue passed the CQ dry run.

[Stephen Chennney, 2017-03-23 16:24:40]

Always good to see the true state of the world. LGTM.

[fs, 2017-03-23 16:36:23]

The CQ bit was checked by fs@opera.com

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

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-03-23 16:46:32]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490286983406970,
"parent_rev": "6f58d02c2c69db09f474ac879795e6a97b5fa01f", "commit_rev":
"0e50a38b387adde5829f18e549917fea3a06a494"}

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

Description was changed from

==========
Use SVGResources in SVGLayoutTreeAsText writeResources

As the FIXME suggests, this should better reflect the actual state of
the resources resolution by including the cycle-solving step.

Some tests (recursive-clip/mask, clip-path-recursive-call-by-child)
needs to be updated to reflect the change to actually show cycles
having been broken. The rests of the tests are updated to reflect
that LayoutSVGInlineText can't have resources.

BUG=454767
==========

to

==========
Use SVGResources in SVGLayoutTreeAsText writeResources

As the FIXME suggests, this should better reflect the actual state of
the resources resolution by including the cycle-solving step.

Some tests (recursive-clip/mask, clip-path-recursive-call-by-child)
needs to be updated to reflect the change to actually show cycles
having been broken. The rests of the tests are updated to reflect
that LayoutSVGInlineText can't have resources.

BUG=454767

Review-Url: https://codereview.chromium.org/2763363002
Cr-Commit-Position: refs/heads/master@{#459119}
Committed:
https://chromium.googlesource.com/chromium/src/+/0e50a38b387adde5829f18e54991...
==========

[commit-bot: I haz the power, 2017-03-23 16:47:15]

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