background-color and text-decoration are not preserved when merging paragraphs

background-color and text-decoration are not preserved when merging paragraphs

In mergeInlineAndImplicitStyleOfElement(), the element styles are retrieved.
If they contain either background-color or text-decoration they will be removed
by removeNonEditingProperties() due to copyEditingProperties() implicit
OnlyInheritableEditingProperties parameter.

WebKit merge from r151772 and r152286

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

[tonyg, 2013-08-06 17:59:21]

I suspect you are looking for tony@, not tonyg@.

[tony, 2013-08-06 18:06:19]

This code LGTM, but why did you only import some of the cleanups in r152286?

[grzegorz, 2013-08-07 07:30:36]

On 2013/08/06 18:06:19, tony wrote:
> This code LGTM, but why did you only import some of the cleanups in r152286?

The part of changes in r152286 are already in repo. Blink uses
'nonInheritedStaticPropertiesCountmagic' instead of magic number 2. The clue of
the r152286 is to use newly added extractEditingProperties(PropertiesToInclude)
instead of removeNonEditingProperties(PropertiesToInclude). Thanks to it, we can
get rid of not used and complicated removeNonEditingProperties().

Other changes in r152286 doesn't affect the bug/fix at all. Frankly speaking I
don't fully understand them as I am not author of r152286. Would you mind
landing the fix first? We can consider to do the cleanup of EditingStyle{h/cpp}
in a separate patch.

[commit-bot: I haz the power, 2013-08-07 16:34:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/g.czajkowski@samsung.com/22353004/1

[commit-bot: I haz the power, 2013-08-07 18:48:54]

Change committed as 155705