Closed Bug 770477 Opened 13 years ago Closed 9 years ago

FUEL: Remove code to support tab move between windows

Categories

(Firefox :: General, defect)

defect
Not set
minor

Tracking

()

RESOLVED WONTFIX

People

(Reporter: sgautherie, Unassigned)

References

()

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