Support multiple text decoration styles.

Support multiple text decoration styles.

Currently, we draw only the last seen text decoration style during
decoration painting, even when multiple decorations
(e.g. underline + overline) are applied. Hence, we potentially draw
the wrong decoration style for one or more of the applied decorations.

Fixed by fetching the matching style for each decoration, similar
to how the matching text-decoration-color is fetched.

BUG=351282
TEST=text-decoration-style-multiple.html

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

[Stephen Chennney, 2014-03-11 15:22:36]

LGTM. For a follow up patch, would it make the code any cleaner if you break out
the drawing code for each of over/through/under into their own methods?

[andersr, 2014-03-11 16:10:05]

The CQ bit was checked by andersr@opera.com

[commit-bot: I haz the power, 2014-03-11 16:10:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/andersr@opera.com/195093002/1

[commit-bot: I haz the power, 2014-03-11 16:13:59]

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

[commit-bot: I haz the power, 2014-03-11 16:14:00]

Try jobs failed on following builders: blink_presubmit

[andersr, 2014-03-11 19:59:33]

The CQ bit was checked by andersr@opera.com

[commit-bot: I haz the power, 2014-03-11 20:00:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/andersr@opera.com/195093002/1

[commit-bot: I haz the power, 2014-03-11 20:49:55]

Change committed as 168946

[andersr, 2014-03-12 16:15:55]

On 2014/03/11 15:22:36, Stephen Chenney wrote:
> LGTM. For a follow up patch, would it make the code any cleaner if you break
out
> the drawing code for each of over/through/under into their own methods?

Yes, please see issue 197633002.