Stop propagating text decorations on outermost SVG roots

Stop propagating text decorations on outermost SVG roots

Outermost SVG roots should be consider atomic inline-level, and hence
text decorations should not propagate into them from the outside.

BUG=406966
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=199132

[fs, 2015-07-15 17:07:57]

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

[Timothy Loh, 2015-07-17 01:03:02]

FYI LayoutObject::getTextDecorations has some logic which may or may not need to
be updated, but I don't really know :-)

[fs, 2015-07-17 07:42:30]

On 2015/07/17 01:03:02, Timothy Loh wrote:
> FYI LayoutObject::getTextDecorations has some logic which may or may not need
to
> be updated, but I don't really know :-)

AFAIK/CT this method does not need to be updated:

1) The SVG text painting code does not use it (it has a simplified version, and
getting text-decoration-color et.c. is of no use to it because it want's 'fill'
/ 'stroke' values)
2) The |decorations| input is generally supposed to be the
textDecorationsInEffect() value which is what the function in this CL affects.

As a side-note, looking at the Gecko patch,it seems they only have the
equivalent of LayoutObject::getTextDecorations.

[pdr., 2015-07-17 17:51:39]

The CQ bit was checked by pdr@chromium.org

[pdr., 2015-07-17 17:51:39]

lgtm

[commit-bot: I haz the power, 2015-07-17 17:52:35]

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

[commit-bot: I haz the power, 2015-07-17 19:52:16]

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