Clean up the getLayoutSVGResource*ById helpers

Clean up the getLayoutSVGResource*ById helpers

Drop getLayoutSVGResourceContainerById helper, since we can just use
SVGTreeScopeResources::resourceById directly.

For getLayoutSVGResourceById, rather than calling through
TreeScope::ensureSVGTreeScopedResources, just pass the
SVGTreeScopeResources and do the lookup from there. Said object is
already available in the relevant places (only the layout tree dumping
needs to be updated wrt that.)

BUG=454767
Review-Url: https://codereview.chromium.org/2708543003
Cr-Commit-Position: refs/heads/master@{#451839}
Committed: https://chromium.googlesource.com/chromium/src/+/7161f0e4a74110c45b7ca11fd3292a8c3ea7b965

[fs, 2017-02-20 10:38:02]

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

[commit-bot: I haz the power, 2017-02-20 10:38:29]

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

[fs, 2017-02-20 12:05:23]

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

[commit-bot: I haz the power, 2017-02-20 12:10:02]

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

[commit-bot: I haz the power, 2017-02-20 12:10:02]

Dry run: This issue passed the CQ dry run.

[Stephen Chennney, 2017-02-21 18:24:01]

lgtm

[fs, 2017-02-21 18:26:47]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2017-02-21 18:27:30]

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-02-21 21:44:04]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487701607499000, "parent_rev":
"946a9f3c1606d0e90b11303580e9b56d5ecf7432", "commit_rev":
"7161f0e4a74110c45b7ca11fd3292a8c3ea7b965"}

[commit-bot: I haz the power, 2017-02-21 21:45:01]

Description was changed from

==========
Clean up the getLayoutSVGResource*ById helpers

Drop getLayoutSVGResourceContainerById helper, since we can just use
SVGTreeScopeResources::resourceById directly.

For getLayoutSVGResourceById, rather than calling through
TreeScope::ensureSVGTreeScopedResources, just pass the
SVGTreeScopeResources and do the lookup from there. Said object is
already available in the relevant places (only the layout tree dumping
needs to be updated wrt that.)

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

to

==========
Clean up the getLayoutSVGResource*ById helpers

Drop getLayoutSVGResourceContainerById helper, since we can just use
SVGTreeScopeResources::resourceById directly.

For getLayoutSVGResourceById, rather than calling through
TreeScope::ensureSVGTreeScopedResources, just pass the
SVGTreeScopeResources and do the lookup from there. Said object is
already available in the relevant places (only the layout tree dumping
needs to be updated wrt that.)

BUG=454767

Review-Url: https://codereview.chromium.org/2708543003
Cr-Commit-Position: refs/heads/master@{#451839}
Committed:
https://chromium.googlesource.com/chromium/src/+/7161f0e4a74110c45b7ca11fd329...
==========

[commit-bot: I haz the power, 2017-02-21 21:45:02]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/7161f0e4a74110c45b7ca11fd329...