Closed Bug 597716 Opened 14 years ago Closed 14 years ago

Document pref values for max_concurrent_tabs

Categories

(Firefox :: Session Restore, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 4.0b7

People

(Reporter: zpao, Assigned: zpao)

References

Details

Attachments

(1 file)

I meant to do that before I landed bug 586068, but forgot.
Attached patch Patch v0.1 (deleted) — Splinter Review
Attachment #481974 - Flags: review?(dietrich)
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+
Pushed http://hg.mozilla.org/mozilla-central/rev/daa279802cb4
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b8
Target Milestone: Firefox 4.0b8 → Firefox 4.0b7
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: