[META] embedding api idl changes

VERIFIED FIXED in M18

Status

()

P3
normal
VERIFIED FIXED
18 years ago
17 years ago

People

(Reporter: jud, Assigned: adamlock)

Tracking

({embed})

Trunk
x86
Windows 98
embed
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [nsbeta3+])

Attachments

(1 attachment)

Comment hidden (empty)
(Reporter)

Updated

18 years ago
Keywords: embed
(Reporter)

Comment 1

18 years ago
nsIWebBrowser -
 topLevelWindow -> containerWindow

nsIWebBrowserChrome -
 setJSStatus, setJSDefaultStatus, setOverlink should be collapsed into a single 
method.
 constant names should be java style
 chromeMask -> chromeFlags

nsIWebNavigation -
 add a goto method which takes a relative index into the session history.
 currentURI() should take a URI.
 reload name flags need to be java style.

nsIWebProgress
 Get rid of progressStatusFlags, curSelfProgress, maxSelfProgress, 
curTotalProgress, maxTotalProgress (and all related flag constants). If you want 
 this information, you have to be a listener. 

nsIWebProgressListener - 
 constant names need to be java style

nsIDOMSelection
 needs to be xpidl'ized instead of idlc

Updated

18 years ago
Keywords: nsbeta3

Comment 2

18 years ago
Yoink
Assignee: valeski → adamlock
(Reporter)

Comment 3

18 years ago
+
Whiteboard: nsbeta3+

Updated

18 years ago
Whiteboard: nsbeta3+ → [nsbeta3+]
(Reporter)

Updated

18 years ago
Target Milestone: --- → M18
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 4

18 years ago
Editor team are grabbing nsIDOMSelection work
Depends on: 48575
(Reporter)

Comment 5

18 years ago
adam where are we on this?
(Assignee)

Comment 6

18 years ago
It's almost finished, the only thing left to do is to change the currentURI 
property on nsIWebNavigation to return an uri. I'm thinking of deleting the 
currentURI property on nsIDocShell at the same time - it's hardly called from 
anywhere.
(Assignee)

Comment 7

18 years ago
Created attachment 13687 [details] [diff] [review]
currentURI changes. Review please
(Reporter)

Comment 8

18 years ago
r=valeski
(Assignee)

Comment 9

18 years ago
Fix checked in 
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 10

18 years ago
verified fixed.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.