Re-land "[css3-text] Implement text-decoration property shorthand"

Re-land "[css3-text] Implement text-decoration property shorthand"

Last time I reverted this patch, I forgot to test it in debug mode
*with the runtime feature disabled*. This caused a series of crashes
(see https://code.google.com/p/chromium/issues/detail?id=271234 for
details) and is now fixed - CSSPropertyTextDecoration was already
removed by filterEnabledCSSPropertiesIntoVector() when runtime feature
is disabled - my apologies for not checking it last time.

I spent more time testing this patch and found out a spin-off issue
while copying/pasting HTML code into content editable elements:
https://code.google.com/p/chromium/issues/detail?id=272363

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

[abinader, 2013-08-13 20:30:26]

Last time I re-reverted this patch, I forgot to test it in debug mode 
*with the runtime feature disabled*. This caused a series of crashes 
(see https://code.google.com/p/chromium/issues/detail?id=271234 for 
details) and is now fixed - CSSPropertyTextDecoration was already 
removed by filterEnabledCSSPropertiesIntoVector() when runtime feature 
is disabled - my apologies for not checking it last time. 

I spent more time testing this patch and found out a spin-off issue 
while copying/pasting HTML code into content editable elements: 
https://code.google.com/p/chromium/issues/detail?id=272363

[Julien - ping for review, 2013-08-16 22:24:37]

lgtm.

> This reverts commit ed65acf69bbd1eefcd53104ce5bf9b0e2fe41c79. 

Can we stop the revert madness? Having a chain of now 5+ patches linked together
to the original change makes blaming super confusing (revert of revert of revert
...).

This change is *NOT* a revert of any patch, it's relanding the original change,
amended with new information. Saying that makes it a lot more obvious what's
going on. Also this would make it obvious that the description should match the
original change, again amended with any new information.

[abinader, 2013-08-17 20:28:37]

On 2013/08/16 22:24:37, Julien Chaffraix wrote:
> lgtm.
> 
> > This reverts commit ed65acf69bbd1eefcd53104ce5bf9b0e2fe41c79. 
> 
> Can we stop the revert madness? Having a chain of now 5+ patches linked
together
> to the original change makes blaming super confusing (revert of revert of
revert
> ...).
> 
> This change is *NOT* a revert of any patch, it's relanding the original
change,
> amended with new information. Saying that makes it a lot more obvious what's
> going on. Also this would make it obvious that the description should match
the
> original change, again amended with any new information.

Agreed - This is an atypical situation for me, so I didn't knew well how to deal
with the reverts, thanks for the heads up.

[abinader, 2013-08-17 20:29:44]

On 2013/08/16 22:24:37, Julien Chaffraix wrote:
> lgtm.
> 
> > This reverts commit ed65acf69bbd1eefcd53104ce5bf9b0e2fe41c79. 
> 
> Can we stop the revert madness? Having a chain of now 5+ patches linked
together
> to the original change makes blaming super confusing (revert of revert of
revert
> ...).
> 
> This change is *NOT* a revert of any patch, it's relanding the original
change,
> amended with new information. Saying that makes it a lot more obvious what's
> going on. Also this would make it obvious that the description should match
the
> original change, again amended with any new information.

Agreed  - this is an atypical situation for me, so I didn't knew well how to
properly format the message, thanks for the heads up!

[commit-bot: I haz the power, 2013-08-17 20:31:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bruno.d@partner.samsung.com/23111004/1

[commit-bot: I haz the power, 2013-08-17 20:51:40]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_blink_rel.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_blin...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-08-17 22:23:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bruno.d@partner.samsung.com/23111004/5002

[commit-bot: I haz the power, 2013-08-17 23:29:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bruno.d@partner.samsung.com/23111004/5002

[commit-bot: I haz the power, 2013-08-18 00:19:35]

Change committed as 156266