648887 - Make imgIDecoderObserver non-scriptable

Status: RESOLVED WONTFIX

(Core :: Graphics: ImageLib, defect)

Description

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

I would like to make imgIDecoderObserver non-scriptable.  The only scriptable use in the tree is for a test, and it would let us fix Bug 604841 and similar issues.

Comment 1

[Bobby Holley (:bholley)]

(khuey and I just talked about this on IRC)

I don't think we should be noscript-ing interfaces (and reducing test coverage) just because XPConnect exposes too much stuff to content. I'd be happy to work on this XPConnect-side in august.

I vote WONTFIX. Joe, thoughts?

Comment 2

[Boris Zbarsky [:bzbarsky]]

Extensions use this interface, fwiw.

Comment 3

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

I had someone run a search of the addons MXR a while back and they told me there wasn't anything, but if people are actually using this I guess this is WONTFIX.

Comment 4

[Joe Drew (not getting mail)]

The scriptable-ness is absolutely essential for very important tests. I'd r+ a DEBUG-only scriptability, but not a wholesale removal.

Comment 5

[Bobby Holley (:bholley)]

(In reply to comment #4)
> The scriptable-ness is absolutely essential for very important tests. I'd r+ a
> DEBUG-only scriptability, but not a wholesale removal.

We run tests on release builds too, no?

Resolving WONTFIX.