801819 - Remove nsIDOMCSS2Properties and CSS declaration classinfo bits

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: Unpref the CSSStyleDeclaration binding.

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 671658 [details] [diff] [review]
Unpref the CSSStyleDeclaration binding.

r=dbaron

Please merge with bug 115199 if it lands first or describe the necessary merging there if it doesn't.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Thanks for pointing that out.  Looks like the merge should be a no-op, actually, at first glance.  I'll double-check once Peter reviews.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I thought there would be some additional removals since that patch clones CSSFontFaceStyleDecl for another declaration implementation.

Comment 5

[Boris Zbarsky [:bzbarsky]]

It doesn't clone it correctly (leaves out the classinfo, which is the part this bug is removing), but the incorrectness only matters if this pref is flipped.

Comment 6

[Peter Van der Beken [:peterv]]

Comment on attachment 671658 [details] [diff] [review]
Unpref the CSSStyleDeclaration binding.

Review of attachment 671658 [details] [diff] [review]:
-----------------------------------------------------------------

::: layout/doc/adding-style-props.html
@@ +458,2 @@
>  <a href="#CSSPropList">add your property</a> to <a href="http://lxr.mozilla.org/seamonkey/source/content/shared/public/nsCSSPropList.h">
> +nsCSSPropList.h</a>. 

Trailing whitespace.

Comment 7

[Boris Zbarsky [:bzbarsky]]

Looks like this actually got unpreffed in bug 801083.

Comment 8

[Boris Zbarsky [:bzbarsky]]

http://hg.mozilla.org/integration/mozilla-inbound/rev/316f2d6d7cbd

Comment 9

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/316f2d6d7cbd