Closed
Bug 243579
Opened 21 years ago
Closed 21 years ago
Change nsIOnReadystatechangeHandler to nsIOnReadyStateChangeHandler
Categories
(Core :: XML, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: timeless, Assigned: timeless)
References
Details
Attachments
(1 file, 1 obsolete file)
|
1.86 KB,
patch
|
jst
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
The mscom method is just an idispatch method
dispinterface XMLDOMDocumentEvents
{
properties:
methods:
[id (DISPID_XMLDOMEVENT_ONDATAAVAILABLE)]
HRESULT ondataavailable(void);
[id (DISPID_XMLDOMEVENT_ONREADYSTATECHANGE)]
HRESULT onreadystatechange(void);
};
it has no case, so i'd like to change the mozilla interface to use a more
typical interCaps format.
|
||
Comment 1•21 years ago
|
||
I'll be happy to r+sr such a change if one is provided...
Attachment #148790 -
Flags: superreview?(jst)
Attachment #148790 -
Flags: review?(jst)
Attachment #148790 -
Attachment description: changes → too many changes
Attachment #148790 -
Attachment is obsolete: true
Attachment #148790 -
Attachment is patch: false
Attachment #148790 -
Flags: superreview?(jst)
Attachment #148790 -
Flags: review?(jst)
Attachment #148902 -
Flags: superreview?(jst)
Attachment #148902 -
Flags: review?(jst)
|
|
||
Comment 4•21 years ago
|
||
Comment on attachment 148902 [details] [diff] [review]
per bug summary
r+sr=jst. Thanks timeless!
Attachment #148902 -
Flags: superreview?(jst)
Attachment #148902 -
Flags: superreview+
Attachment #148902 -
Flags: review?(jst)
Attachment #148902 -
Flags: review+
Attachment #148902 -
Flags: approval1.8a1?
|
|
||
Comment 5•21 years ago
|
||
Comment on attachment 148902 [details] [diff] [review]
per bug summary
1.8a1 shipped, wanna land this on the trunk?
Attachment #148902 -
Flags: approval1.8a1?
mozilla/extensions/xmlextras/base/public/nsIXMLHttpRequest.idl 1.25
mozilla/extensions/xmlextras/base/src/nsXMLHttpRequest.cpp 1.109
mozilla/extensions/xmlextras/base/src/nsXMLHttpRequest.h 1.42
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•