Implement nsIDOMStorageEvent using event implementation codegen

RESOLVED FIXED

Status

()

Core
DOM: Events
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: smaug, Assigned: smaug)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

5 years ago
patch coming
(Assignee)

Comment 1

5 years ago
Created attachment 645692 [details] [diff] [review]
patch
Attachment #645692 - Flags: review?(honzab.moz)
There is NS_NewDOMStorageEvent function used, but it is not defined in the patch either is not in mxr.  I presume there is some dependency that needs to land prior this patch.  Could you please add that dep to the list?  Thanks.
No, the function will be generated by the codegen in js/xpconnect/src.
(In reply to :Ms2ger from comment #3)
> No, the function will be generated by the codegen in js/xpconnect/src.

OK, can I know how that works?
(Assignee)

Comment 5

5 years ago
See bug 765163. It has example .h and .cpp
Comment on attachment 645692 [details] [diff] [review]
patch

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

r=honzab.
Attachment #645692 - Flags: review?(honzab.moz) → review+
(Assignee)

Comment 7

5 years ago
Created attachment 647978 [details] [diff] [review]
up-to-date
(Assignee)

Comment 8

5 years ago
https://hg.mozilla.org/mozilla-central/rev/476a189f5cda
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.