Last Comment Bug 689337 - Desktop notification interfaces should not be marked 'function'
: Desktop notification interfaces should not be marked 'function'
Product: Core
Classification: Components
Component: DOM (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla12
Assigned To: :Ms2ger
Depends on:
  Show dependency treegraph
Reported: 2011-09-26 15:25 PDT by Kyle Huey [:khuey] (
Modified: 2012-01-25 07:34 PST (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Patch v1 (1.13 KB, patch)
2012-01-13 01:38 PST, :Ms2ger
khuey: review+
Details | Diff | Review

Description Kyle Huey [:khuey] ( 2011-09-26 15:25:34 PDT

Comment 1 Doug Turner (:dougt) 2011-09-26 20:26:15 PDT
Comment 2 Kyle Huey [:khuey] ( 2011-09-27 04:39:14 PDT
Because 'function' means you can pass in a JS function and XPConnect will coerce it into the interface.  Both these interfaces are only implemented in C++, so this is unnecessary.
Comment 3 Doug Turner (:dougt) 2011-09-27 07:40:00 PDT
yes, 'function' should be removed.
Comment 4 Doug Turner (:dougt) 2012-01-12 14:02:05 PST
i am not working on these right now.  resetting assignee.
Comment 5 :Ms2ger 2012-01-13 01:38:01 PST
Created attachment 588344 [details] [diff] [review]
Patch v1

Note You need to log in before you can comment on or make changes to this bug.