Remove CSSRule.UNKNOWN_RULE and CSSUnknownRule interface

Remove CSSRule.UNKNOWN_RULE and CSSUnknownRule interface

The CSSUnknownRule interface was introduced in DOM Level 2 Style, but
has been removed in CSSOM:

http://www.w3.org/TR/DOM-Level-2-Style/css.html#CSS-CSSUnknownRule
http://dev.w3.org/csswg/cssom/
This patch removes it completely.

The link to Intent to Deprecate and Remove is here:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/zzeNGPP1DcA

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

[Sunil Ratnu, 2015-06-10 10:50:14]

sunil.ratnu@samsung.com changed reviewers:
+ philipj@opera.com, sanjoy.pal@samsung.com

[Sunil Ratnu, 2015-06-10 10:50:15]

Please have a look.

[philipj_slow, 2015-06-10 13:34:21]

LGTM % CSSRule.h cleanup

https://codereview.chromium.org/1154753006/diff/20001/Source/core/css/CSSRule.h
File Source/core/css/CSSRule.h (right):

https://codereview.chromium.org/1154753006/diff/20001/Source/core/css/CSSRule...
Source/core/css/CSSRule.h:44: STYLE_RULE = 1,
Can you spell out the constant for every case to match CSSRule.idl? Especially
since the last two entries have them this starts to look strange.

https://codereview.chromium.org/1154753006/diff/20001/Source/core/css/CSSRule...
Source/core/css/CSSRule.h:50: // 7 was VARIABLES_RULE; we now match other
browsers with 7 as
I think you can remove this comment too, this was fixed a long time ago...

[Sunil Ratnu, 2015-06-10 13:54:40]

Thanks for the review Philip. Made the changes. Please give it a look one more
time.

https://codereview.chromium.org/1154753006/diff/20001/Source/core/css/CSSRule.h
File Source/core/css/CSSRule.h (right):

https://codereview.chromium.org/1154753006/diff/20001/Source/core/css/CSSRule...
Source/core/css/CSSRule.h:44: STYLE_RULE = 1,
On 2015/06/10 13:34:21, philipj wrote:
> Can you spell out the constant for every case to match CSSRule.idl? Especially
> since the last two entries have them this starts to look strange.

Done.

https://codereview.chromium.org/1154753006/diff/20001/Source/core/css/CSSRule...
Source/core/css/CSSRule.h:50: // 7 was VARIABLES_RULE; we now match other
browsers with 7 as
On 2015/06/10 13:34:21, philipj wrote:
> I think you can remove this comment too, this was fixed a long time ago...

Done.

[philipj_slow, 2015-06-10 13:59:46]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-06-10 13:59:47]

lgtm

Looks great!

[commit-bot: I haz the power, 2015-06-10 13:59:57]

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

[commit-bot: I haz the power, 2015-06-10 15:24:46]

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