69918 - simplify or remove nsIWebBrowserChrome::[SG]etPersistence

Status: VERIFIED FIXED

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

Description

[Dan M]

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.

Comment 1

[Dan M]

Attachment: move [GS]etPersistence to nsIDocShellTreeOwner and fix Fibonacci indentation

Comment 2

[Dan M]

.

Comment 3

[Michael Dunn]

Correction: Changing QA contact for the Embed API bugs to David Epstein.

Comment 4

[David Epstein]

* set/getPersistence no longer in nsIWebBrowserChrome.idl
* moved to nsIDocShellTreeOwner.idl