Closed Bug 69918 Opened 23 years ago Closed 23 years ago

simplify or remove nsIWebBrowserChrome::[SG]etPersistence

Categories

(Core Graveyard :: Embedding: APIs, defect, P2)

defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.9

People

(Reporter: danm.moz, Assigned: danm.moz)

References

Details

(Keywords: embed)

Attachments

(1 file)

It strikes the embedding API review team that GetPersistence lives uneasily in 
the nsIWebBrowserChrome interface. We should move this somewhere else or, failing 
that, make it more palatable.

Current ideas are:
1) move it to nsIBaseWindow
2) move it to the Mozilla-FE-only nsIXULWindow (requiring using include files 
from Mozilla XPFE in basic components which shouldn't be doing that)
3) hide the functionality in something more generic like [SG]etMessage
4) simplify it to [SG]etPersistence(yes or no)

I'm leaning toward #4 right now. The argument being there's something wrong with 
all of them, but maybe #4 the least. And a [SG]etPersistence functionality could 
be construed to make sense on that interface.
Status: NEW → ASSIGNED
Keywords: embed
Target Milestone: --- → mozilla0.9
Blocks: 70229
Priority: -- → P2
.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Correction: Changing QA contact for the Embed API bugs to David Epstein.
QA Contact: mdunn → depstein
* set/getPersistence no longer in nsIWebBrowserChrome.idl
* moved to nsIDocShellTreeOwner.idl
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.