Closed
Bug 597716
Opened 14 years ago
Closed 14 years ago
Document pref values for max_concurrent_tabs
Categories
(Firefox :: Session Restore, defect)
Firefox
Session Restore
Tracking
()
RESOLVED
FIXED
Firefox 4.0b7
People
(Reporter: zpao, Assigned: zpao)
References
Details
Attachments
(1 file)
|
1.32 KB,
patch
|
dietrich
:
review+
|
Details | Diff | Splinter Review |
I meant to do that before I landed bug 586068, but forgot.
|
Assignee | |
Comment 1•14 years ago
|
||
Attachment #481974 -
Flags: review?(dietrich)
|
||
Comment 2•14 years ago
|
||
Comment on attachment 481974 [details] [diff] [review]
Patch v0.1
>+// The number of tabs that can restore concurrently:
>+// < 0 = All tabs can restore at the same time
>+// 0 = Only the selected tab in each window will load.
this one is not clear. the selected tab is restored, it doesn't just load, right? what happens to the other tabs?
>+// N = N tabs should restore at the same time
sheesh, the whole point of this is to convey meaning explicitly!
"N = The number of tabs to restore at the same time" i think is better.
r=me with whatever improvements you make.
Attachment #481974 -
Flags: review?(dietrich) → review+
|
|
Assignee | |
Comment 3•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b8
|
||
Updated•14 years ago
|
Target Milestone: Firefox 4.0b8 → Firefox 4.0b7
You need to log in
before you can comment on or make changes to this bug.
Description
•