Include additional translation in getBBox for SVGUseElement

Include additional translation in getBBox for SVGUseElement

LayoutObject::objectBoundingBox() can't be used, because the additional
translation is included in the local-to-parent transform.

BUG=512081
Committed: https://crrev.com/495b0fc68f462f3541919a38a15eed0d8fea6878
git-svn-id: svn://svn.chromium.org/blink/trunk@202136 bbb929c8-8fbe-4397-9dbb-9b2b20218538

[fs, 2015-09-08 15:34:58]

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

[fs, 2015-09-08 15:34:59]

https://codereview.chromium.org/1317563010/ should be allowed to land first
since this would otherwise affect the result of gBCR on <use>.

[fs, 2015-09-10 10:41:46]

On 2015/09/08 15:34:59, fs wrote:
> https://codereview.chromium.org/1317563010/ should be allowed to land first
> since this would otherwise affect the result of gBCR on <use>.

The above CL has landed, so PTAL.

[pdr., 2015-09-11 05:57:19]

LGTM

https://codereview.chromium.org/1310873004/diff/1/Source/core/layout/svg/Layo...
File Source/core/layout/svg/LayoutSVGTransformableContainer.h (right):

https://codereview.chromium.org/1310873004/diff/1/Source/core/layout/svg/Layo...
Source/core/layout/svg/LayoutSVGTransformableContainer.h:37: const FloatSize&
additionalTranslation() const { return m_lastTranslation; }
WDYT about renaming m_lastTranslation since "last" is a little ambiguous?

[fs, 2015-09-11 07:40:52]

On 2015/09/11 05:57:19, pdr wrote:
...
>
https://codereview.chromium.org/1310873004/diff/1/Source/core/layout/svg/Layo...
> File Source/core/layout/svg/LayoutSVGTransformableContainer.h (right):
> 
>
https://codereview.chromium.org/1310873004/diff/1/Source/core/layout/svg/Layo...
> Source/core/layout/svg/LayoutSVGTransformableContainer.h:37: const FloatSize&
> additionalTranslation() const { return m_lastTranslation; }
> WDYT about renaming m_lastTranslation since "last" is a little ambiguous?

SGTM, will make a patch.

[fs, 2015-09-11 07:40:58]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2015-09-11 07:41:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1310873004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1310873004/1

[commit-bot: I haz the power, 2015-09-11 09:15:35]

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

[commit-bot: I haz the power, 2015-09-11 09:15:37]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[fs, 2015-09-11 10:32:35]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2015-09-11 10:32:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1310873004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1310873004/1

[commit-bot: I haz the power, 2015-09-11 11:55:48]

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

[commit-bot: I haz the power, 2015-09-11 11:55:49]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[fs, 2015-09-11 13:08:38]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2015-09-11 13:08:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1310873004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1310873004/1

[commit-bot: I haz the power, 2015-09-11 14:02:48]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=202136

[commit-bot: I haz the power, 2015-09-23 12:22:12]

Patchset 1 (id:??) landed as
https://crrev.com/495b0fc68f462f3541919a38a15eed0d8fea6878