Closed Bug 770477 Opened 12 years ago Closed 8 years ago

FUEL: Remove code to support tab move between windows

Categories

(Firefox :: General, defect)

Product:
Firefox
Component:
General
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: sgautherie, Unassigned)

References

(
URL
)

Details

(In reply to neil@parkwaycc.co.uk from bug 767682 comment #12)
> (In reply to Serge Gautherie from bug 767682 comment #10)
> > > >+  } else {
> > > >+    // This tab may have moved to another window, so make sure its cached
> > > >+    // window is up-to-date.
> > > >+    smileBrowserTab._window = aSMILEWindow;
> > > Actually I don't think this is possible. Even if an extension "moves" a tab
> > > to a new window, it would need to create new <tab> and <browser> elements in
> > > the new window's document, so that a <tab> element could never exist in more
> > > than one window. r=me with this removed. (Ensure that getWindow and
> > > getBrowserTab use the same formatting.)
> > Is this a difference between SMILE and FUEL, or would you say this code is
> > useless in FUEL too?
> As far as I can tell, this code is useless in FUEL too.
Summary: FUEL: confirm (or remove) code to support tab move between windows → FUEL: Remove code to support tab move between windows
Good riddance!
FUEL is being removed in bug 1090880, so this is now wontfix.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.