If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

FUEL: Remove code to support tab move between windows

RESOLVED WONTFIX

Status

()

Firefox
General
--
minor
RESOLVED WONTFIX
5 years ago
2 years ago

People

(Reporter: sgautherie, Unassigned)

Tracking

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

5 years ago
(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.

Updated

5 years ago
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
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.