Closed Bug 633126 Opened 13 years ago Closed 13 years ago

Port Bug 590268 [Provide access to sessionstore tab data sooner] to SeaMonkey

Categories

(SeaMonkey :: Session Restore, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1b3

People

(Reporter: misak.bugzilla, Assigned: misak.bugzilla)

References

Details

Attachments

(1 file)

Attached patch straight portSplinter Review
From parent bug:

Tab Candy needs this; otherwise we don't get our tab location data right at
start up.
Attachment #511327 - Flags: review?(neil)
Comment on attachment 511327 [details] [diff] [review]
straight port

I was thinking they could have just written them as rather long || statements, but somehow I doubt that would have passed review ;-)
Attachment #511327 - Flags: review?(neil) → review+
Just for the record, those rather long || statements look like this!

return (aTab.__SS_extdata ||
        (aTab.linkedBrowser.__SS_data &&
         aTab.linkedBrowser.__SS_data.extData) ||
        {})[aKey] || "";

(aTab.__SS_extdata ||
 (aTab.linkedBrowser.__SS_data && 
  aTab.linkedBrowser.__SS_data.extData) ||
 (aTab.__SS_extdata = {}))[aKey] = aStringValue;
Much more compact but clear as mud!
Pushed: http://hg.mozilla.org/comm-central/rev/c92d62353031
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
OS: Linux → All
Hardware: x86_64 → All
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: