[SVGDom] Expose intrinsic size info

[SVGDom] Expose intrinsic size info

  * expose intrinsic size info on <svg> nodes.

  * tweak the SkSVGDOM constructor to no longer take an container size
    param, but instead default to intrinsic size
  * update clients to call SkSVGDOM::setContainerSize() explicitly, when
    needed

R=robertphillips@google.com,stephana@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2345533002

Committed: https://skia.googlesource.com/skia/+/e1baa7c105dad0f301ce34e5d6d1aa329334ef8c

[f(malita), 2016-09-14 18:06:12]

Description was changed from

==========
[SVGDom] Expose intrinsic size info

  * expose intrinsic size info on <svg> nodes.

  * tweak the SkSVGDOM constructor to no longer take an container size
    param, but instead default to intrinsic size
  * update clients to call SkSVGDOM::setContainerSize() explicitly, when
    needed

R=robertphillips@google.com,stephana@google.com
==========

to

==========
[SVGDom] Expose intrinsic size info

  * expose intrinsic size info on <svg> nodes.

  * tweak the SkSVGDOM constructor to no longer take an container size
    param, but instead default to intrinsic size
  * update clients to call SkSVGDOM::setContainerSize() explicitly, when
    needed

R=robertphillips@google.com,stephana@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2345533002
==========

[f(malita), 2016-09-14 18:12:29]

This is mainly to allow nanobench to raster at "native"/intrinsic SVG resolution
(for Android asset benching).

[f(malita), 2016-09-14 18:12:38]

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

[commit-bot: I haz the power, 2016-09-14 18:12:47]

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

[f(malita), 2016-09-14 18:15:37]

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

[commit-bot: I haz the power, 2016-09-14 18:15:47]

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, 2016-09-14 18:33:07]

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

[commit-bot: I haz the power, 2016-09-14 18:33:09]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2016-09-14 18:34:54]

lgtm

https://codereview.chromium.org/2345533002/diff/20001/experimental/svg/model/...
File experimental/svg/model/SkSVGDOM.h (right):

https://codereview.chromium.org/2345533002/diff/20001/experimental/svg/model/...
experimental/svg/model/SkSVGDOM.h:28: 
Should intrinsicSize be private?

[f(malita), 2016-09-14 18:40:05]

https://codereview.chromium.org/2345533002/diff/20001/experimental/svg/model/...
File experimental/svg/model/SkSVGDOM.h (right):

https://codereview.chromium.org/2345533002/diff/20001/experimental/svg/model/...
experimental/svg/model/SkSVGDOM.h:28: 
On 2016/09/14 18:34:53, robertphillips wrote:
> Should intrinsicSize be private?

Might be useful in the future, but yeah, for now there are no external clients
so it could be private.

Done.

[f(malita), 2016-09-14 18:43:43]

The CQ bit was checked by fmalita@chromium.org

[f(malita), 2016-09-14 18:43:43]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/2345533002/#ps40001
(title: "private intrinsicSize()")

[commit-bot: I haz the power, 2016-09-14 18:43:46]

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, 2016-09-14 19:04:32]

Description was changed from

==========
[SVGDom] Expose intrinsic size info

  * expose intrinsic size info on <svg> nodes.

  * tweak the SkSVGDOM constructor to no longer take an container size
    param, but instead default to intrinsic size
  * update clients to call SkSVGDOM::setContainerSize() explicitly, when
    needed

R=robertphillips@google.com,stephana@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2345533002
==========

to

==========
[SVGDom] Expose intrinsic size info

  * expose intrinsic size info on <svg> nodes.

  * tweak the SkSVGDOM constructor to no longer take an container size
    param, but instead default to intrinsic size
  * update clients to call SkSVGDOM::setContainerSize() explicitly, when
    needed

R=robertphillips@google.com,stephana@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2345533002

Committed:
https://skia.googlesource.com/skia/+/e1baa7c105dad0f301ce34e5d6d1aa329334ef8c
==========

[commit-bot: I haz the power, 2016-09-14 19:04:34]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/e1baa7c105dad0f301ce34e5d6d1aa329334ef8c