Closed Bug 696940 Opened 13 years ago Closed 10 years ago

Unresponsive script dialog for nsSessionStore.js line 3145

Categories

(Firefox :: Session Restore, defect)

x86
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: Unfocused, Unassigned)

Details

Started getting this every time I restore my session. Line 3145 of nsSessionStore.js is:

        for (id = Date.now(); id in aIdMap.used; id++);


I only have 66 tabs open (in the past, I've had 200-300 tabs). My sessionstore.js file is 4.9Mb.
Hmm. I don't see why we would get stuck there unless Date.now() is hitting max int or is close enough that we end up hitting it as we remap ids...

I don't see why but perhaps related to 687710?

(side note: there's something mean in your session file keeping it at a larger-than-it-should-be-for-66-tabs size
(In reply to Paul O'Shannessy [:zpao] from comment #1)
> (side note: there's something mean in your session file keeping it at a
> larger-than-it-should-be-for-66-tabs size

Yea, I believe that's an extension doing that - Spool adds iframes to every link on Twitter, and some other sites). Which, thinking about it, could be the cause. I have Twitter in an AppTab, so it loads first - and my sessionrestore.js has a *huge* number of children for that tab (all about:blank).
quoth the raven, "omg i'm all over that ****."
and so it shall-eth be fix-eth!

(happy halloween everyone, I'll work on getting this fixed asap)
She's all over that.
Blair, do you still see this problem when using a current version?
Flags: needinfo?(bmcbride)
Nope.
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(bmcbride)
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.