Closed Bug 1309048 Opened 8 years ago Closed 8 years ago

move embedding/ files out of that directory

Categories

(Core Graveyard :: Embedding: APIs, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1309049

People

(Reporter: myk, Assigned: myk)

Details

Attachments

(1 file)

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.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: