110535 - Implement the code to copy a window for bug 18808

Status: RESOLVED WORKSFORME

(SeaMonkey :: Session Restore, enhancement)

Description

[Brian 'netdragon' Bober]

No not a dupe of Bug 18808.

We have been talking about this for over a year and still haven't come up to any
solution. It is one of only about 10 bugs that have this many or more votes!!!

I like Andrew's compromise.. its very thoughtful... but heheh its not practical
imho. I feel instead of making both parties happy it will make one semi-happy
and one totally angry as the back history has a rogue member in it.

What we need is someone who is willing to take this under their wings (not me -
please don't even ask :) - If someone implements it - it will probably be added
because of the number of votes on this bug. So this is a chance for someone to
make a real impact. I mean a HUGE impact.

Please correct me if I'm wrong... but what we need is:

A) Code to implement the copying of a window.
1) HTML data
2) Forward history
3) Back history
4) Javascript/images/etc...
5) Make sure popup windows for ads etc don't do this or it would slow things
down. (Maybe later)

C) Test its performance and fine-tune it. Copy memory in bulk, etc.


2+3 are probably best done every time in order not to complicate things.

Bug 18808 should only have to do with how to trigger it because otherwise it is
way too broad. 

Therefore... I am opening a new bug Called "Implement the code to copy a window"
and copying this text in. This code will have nothing to do with the
implementation of the UI for the bug but only have to do with the actual copying
of the window. It can be tested using javascript or whatever. i.e. :
copywindow(); in the URL bar.

Questions:
Would it be best if it copied the page as-is including pages recieved from CGI
without re-sending the CGI? Therefore data entered into forms would be copied
along with data recieved so the user wouldn't have to wait... Or would this
cause a mess?

Comment 1

[Claudius Gayle]

sharing the love. cc jrgm for when this too complicated for me to keep track of :-)

Comment 2

[Radha on family leave (not reading bugmail)]

this is a broad browser feature not a feature of session history. Session
history apis have all the methods required to make a copy of it. 

Comment 3

[Peter Trudelle]

We are trying to make windows open faster and be easier to use, not slower and
more complicated.  wontfix.

Comment 4

[Brian 'netdragon' Bober]

If you mark this bug WONTFIX then you should mark bug 18808 WONTFIX too. This
bug is only for implementing the code to copy a window. Code doesn't add any
memory footprint to windows as the code for a class only has one copy. As for
Windows opening faster... Speed will have to be a tradeoff for implementing bug
18808. As not ALL windows openened will need to have the back/forward history,
etc. This will not affect the speed for all windows. This bug was made to break
up the tasks necessary for implementing bug 18808 into more specific tasks. As
this bug is necessary to be resolved for 18808 and bug 18808 hasn't been
resolved WONTFIX then this one shouldn't either. 

Comment 5

[Peter Trudelle]

gee, thanks.  wontfix.

Comment 6

[Brian 'netdragon' Bober]

Reopening as per 18808

Comment 7

[Brian 'netdragon' Bober]

-> nobody

Comment 8

[Brian 'netdragon' Bober]

Note: this code should in no way change the performance of opening a new window
the old way.

Comment 9

[neutral]

This should also apply to tabs.

Comment 10

[John]

This is good! Make the code *first*, leave the implementation up to someone else
because those guys in 18808 could argue forever. It gets my vote.

Comment 11

[John]

EEK! I ran out of votes. Sorry. I'm in the cc list tho.

Comment 12

[Robert Kaiser]

MASS-CHANGE:
This bug report is registered in the SeaMonkey product, but has been without a comment since the inception of the SeaMonkey project. This means that it was logged against the old Mozilla suite and we cannot determine that it's still valid for the current SeaMonkey suite. Because of this, we are setting it to an UNCONFIRMED state.

If you can confirm that this report still applies to current SeaMonkey 2.x nightly builds, please set it back to the NEW state along with a comment on how you reproduced it on what Build ID, or if it's an enhancement request, why it's still worth implementing and in what way.
If you can confirm that the report doesn't apply to current SeaMonkey 2.x nightly builds, please set it to the appropriate RESOLVED state (WORKSFORME, INVALID, WONTFIX, or similar).
If no action happens within the next few months, we move this bug report to an EXPIRED state.

Query tag for this change: mass-UNCONFIRM-20090614

Comment 14

[Philip Chee]

> Therefore... I am opening a new bug Called "Implement the code to copy a window"
> and copying this text in. This code will have nothing to do with the
> implementation of the UI for the bug but only have to do with the actual copying
> of the window. It can be tested using javascript or whatever. i.e. :
> copywindow(); in the URL bar.

The SessionStore backend which we now have provides all the functionality needed so marking as WORKSFORME.

The front end UI will be fixed in Bug 449728 - Write front-end code to use backend Drag & Drop tabs between browser windows functionality.