Closed Bug 243579 Opened 20 years ago Closed 20 years ago

Change nsIOnReadystatechangeHandler to nsIOnReadyStateChangeHandler

Categories

(Core :: XML, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: timeless, Assigned: timeless)

References

Details

Attachments

(1 file, 1 obsolete file)

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.
I'll be happy to r+sr such a change if one is provided...
Attached file too many changes (obsolete) —
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)
Attached patch per bug summarySplinter Review
Attachment #148902 - Flags: superreview?(jst)
Attachment #148902 - Flags: review?(jst)
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 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: 20 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.