Closed Bug 1135498 Opened 10 years ago Closed 10 years ago

Remove browser.__SS_restore_data as it is unused

Categories

(Firefox :: Session Restore, defect)

Product:
Firefox
Component:
Session Restore
Type:
defect
Priority:
Not set
Severity:
normal
Points:
1

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 39
Iteration:
39.1 - 9 Mar
Tracking Flags:
Tracking Status
firefox39 --- fixed

People

(Reporter: ttaubert, Assigned: ttaubert)

References

Details

Attachments

(1 file, 1 obsolete file)

0001-Bug-1135498-Get-rid-of-the-unused-browser.__SS_resto.patch, v2
3.67 KB, patch
smacleod
: review+
Details | Diff | Splinter Review 
Stumbled upon the "__SS_restore_data" property we are still setting and removing although it's unused now that all that happens in the content process. Let's get rid of that \o/
Assignee: nobody → ttaubert
Status: NEW → ASSIGNED
Iteration: --- → 38.3 - 23 Feb
Points: --- → 1
Flags: qe-verify-
Flags: firefox-backlog+
Thought it was worth fixing up the comment describing restoreTabContent() as that's horribly outdated.
Attachment #8567628 - Attachment is obsolete: true
Attachment #8567628 - Flags: review?(smacleod)
Attachment #8567629 - Flags: review?(smacleod)
Iteration: 38.3 - 23 Feb → 39.1 - 9 Mar
Comment on attachment 8567629 [details] [diff] [review]
0001-Bug-1135498-Get-rid-of-the-unused-browser.__SS_resto.patch, v2

Review of attachment 8567629 [details] [diff] [review]:
-----------------------------------------------------------------

Wow ya that comment was stale.
Attachment #8567629 - Flags: review?(smacleod) → review+
https://hg.mozilla.org/mozilla-central/rev/0f3823760e96
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox39: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 39
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: