633833 - Add 'function' keyword to all the single function JSD interfaces

Status: RESOLVED FIXED

(Core :: JavaScript Engine, enhancement)

Description

[:Gijs (he/him)]

In particular:

jsdIActivationCallback
jsdINestCallback
jsdIFilterEnumerator
jsdIScriptEnumerator
jsdIContextEnumerator
jsdICallHook
jsdIErrorHook
jsdIExecutionHook


I'd even argue that onScriptCreated/onScriptDestroyed should be refactored to be a single function with an argument indicating whether a script was created or destroyed, but I suppose that is debatable, and would constitute an actual change in the interface...

Comment 1

[:Gijs (he/him)]

(In reply to comment #0)
> ... and would constitute an actual
> change in the interface...

Which reminds me, I can do a patch, but would I need to regenerate the CIDs for these interfaces?

Comment 2

[timeless]

so, the problem is that a caller which relies on [function] being present will fail on a version where it isn't present, being able to detect that it needs to have an object is thus vaguely necessary.

note that if you do bump an iid, you need to bump the interfaces which use them as arguments/attributes/return types and iteratively for those changed interfaces.

i'm ok with you not bumping the iids for this case, someone can also detect the lack of this by trying with a function and if it throws, knowing that it needs to wrap.

Comment 3

[:Gijs (he/him)]

Attachment: Patch

Good, here we go. :-)

Comment 4

[:Gijs (he/him)]

http://hg.mozilla.org/mozilla-central/rev/1a26bfb69598