Closed
Bug 1309048
Opened 8 years ago
Closed 8 years ago
move embedding/ files out of that directory
Categories
(Core Graveyard :: Embedding: APIs, defect)
Core Graveyard
Embedding: APIs
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1309049
People
(Reporter: myk, Assigned: myk)
Details
Attachments
(1 file)
8 years ago
64.88 KB,
patch
|
Details | Diff | Splinter Review |
bz: this is a strawman proposal for moving embedding/ files out of that directory. Roughly, it makes these changes: embedding/components/ -> docshell/components/ most of embedding/browser/ -> docshell/components/browser/ embedding/browser/nsDocShellTreeOwner.*|ns*ContextMenu*|nsI*Tooltip* -> docshell/base/ embedding/nsIWindow* (and tests) -> docshell/components/windowcreator/ embedding/nsEmbedCID.h -> docshell/components/browser/ I chose docshell/ because some of the moved files (like nsDocShellTreeOwner) are related, and also because you're its module owner, which provides continuity of ownership over the files. Nevertheless, I know that docshell/ isn't a great fit for most of the files, ontologically, and I'm happy to move them elsewhere. Offhand, xpfe/ and toolkit/ seem like better fits, but the former seems moribund, and you aren't a peer of either.
Updated•8 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
Updated•5 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•