Make CSS resize property non-inherited.

Make CSS resize property non-inherited.

Align with CSS UI spec[1] and Gecko.

[1] http://dev.w3.org/csswg/css-ui/#resize

R=andersr@opera.com,timloh@chromium.org
BUG=425552
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184176

[Timothy Loh, 2014-10-21 15:14:00]

Can you mention whether other browsers behave correctly in the patch
description?

https://codereview.chromium.org/639293008/diff/1/LayoutTests/fast/overflow/re...
File LayoutTests/fast/overflow/resize-inherit.html (right):

https://codereview.chromium.org/639293008/diff/1/LayoutTests/fast/overflow/re...
LayoutTests/fast/overflow/resize-inherit.html:13: resize: both;
Did you mean resize: inherit;?

https://codereview.chromium.org/639293008/diff/1/Source/core/css/CSSPropertie...
File Source/core/css/CSSProperties.in (right):

https://codereview.chromium.org/639293008/diff/1/Source/core/css/CSSPropertie...
Source/core/css/CSSProperties.in:240: // FIXME: This shouldn't be inherited, as
per css-ui
You should probably remove the FIXMEs you fix :)

[rune, 2014-10-21 15:26:04]

https://codereview.chromium.org/639293008/diff/1/LayoutTests/fast/overflow/re...
File LayoutTests/fast/overflow/resize-inherit.html (right):

https://codereview.chromium.org/639293008/diff/1/LayoutTests/fast/overflow/re...
LayoutTests/fast/overflow/resize-inherit.html:13: resize: both;
On 2014/10/21 15:14:00, Timothy Loh wrote:
> Did you mean resize: inherit;?

Done.

https://codereview.chromium.org/639293008/diff/1/Source/core/css/CSSPropertie...
File Source/core/css/CSSProperties.in (right):

https://codereview.chromium.org/639293008/diff/1/Source/core/css/CSSPropertie...
Source/core/css/CSSProperties.in:240: // FIXME: This shouldn't be inherited, as
per css-ui
On 2014/10/21 15:14:00, Timothy Loh wrote:
> You should probably remove the FIXMEs you fix :)

Amazingly enough, I didn't see it.

[rune, 2014-10-21 15:28:02]

On 2014/10/21 15:14:00, Timothy Loh wrote:
> Can you mention whether other browsers behave correctly in the patch
> description?

I don't have Windows/Trident here, but I tested Gecko and mentioned it in the
description.

[Timothy Loh, 2014-10-21 15:30:39]

On 2014/10/21 15:28:02, rune wrote:
> On 2014/10/21 15:14:00, Timothy Loh wrote:
> > Can you mention whether other browsers behave correctly in the patch
> > description?
> 
> I don't have Windows/Trident here, but I tested Gecko and mentioned it in the
> description.

FYI you can easily get screenshots in various browsers with modern.ie 

lgtm

[rune, 2014-10-21 15:32:51]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2014-10-21 15:33:40]

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

[rune, 2014-10-21 15:44:46]

On 2014/10/21 15:30:39, Timothy Loh wrote:

> FYI you can easily get screenshots in various browsers with modern.ie 

Cool. I didn't know. I won't add Trident to the description. Its result was
'both', 'undefined', 'inherit' instead of 'both', 'none', 'both'. But Presto,
otoh ;-)

[commit-bot: I haz the power, 2014-10-21 16:42:10]

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

[commit-bot: I haz the power, 2014-10-21 16:42:11]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/3...)

[rune, 2014-10-22 11:18:35]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2014-10-22 11:19:57]

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

[commit-bot: I haz the power, 2014-10-22 12:39:19]

Committed patchset #3 (id:40001) as 184176