Last Comment Bug 243579 - Change nsIOnReadystatechangeHandler to nsIOnReadyStateChangeHandler
: Change nsIOnReadystatechangeHandler to nsIOnReadyStateChangeHandler
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: XML (show other bugs)
: Trunk
: x86 Windows XP
: -- normal (vote)
: ---
Assigned To: timeless
: Ashish Bhatt
Mentors:
Depends on: 96459
Blocks:
  Show dependency treegraph
 
Reported: 2004-05-13 19:03 PDT by timeless
Modified: 2004-06-16 05:42 PDT (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
too many changes (5.64 KB, text/plain)
2004-05-18 19:15 PDT, timeless
no flags Details
per bug summary (1.86 KB, patch)
2004-05-19 18:23 PDT, timeless
jst: review+
jst: superreview+
Details | Diff | Review

Description timeless 2004-05-13 19:03:10 PDT
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 Johnny Stenback (:jst, jst@mozilla.com) 2004-05-18 17:44:37 PDT
I'll be happy to r+sr such a change if one is provided...
Comment 2 timeless 2004-05-18 19:15:23 PDT
Created attachment 148790 [details]
too many changes
Comment 3 timeless 2004-05-19 18:23:17 PDT
Created attachment 148902 [details] [diff] [review]
per bug summary
Comment 4 Johnny Stenback (:jst, jst@mozilla.com) 2004-05-19 18:30:05 PDT
Comment on attachment 148902 [details] [diff] [review]
per bug summary

r+sr=jst. Thanks timeless!
Comment 5 Johnny Stenback (:jst, jst@mozilla.com) 2004-05-20 16:46:46 PDT
Comment on attachment 148902 [details] [diff] [review]
per bug summary

1.8a1 shipped, wanna land this on the trunk?
Comment 6 timeless 2004-06-16 05:42:10 PDT
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

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