Last Comment Bug 558639 - Port Bug 531519 [getBrowserState sometimes returns two entries for one window] to Seamonkey
: Port Bug 531519 [getBrowserState sometimes returns two entries for one window...
Product: SeaMonkey
Classification: Client Software
Component: Session Restore (show other bugs)
: Trunk
: All All
-- normal (vote)
: seamonkey2.1a1
Assigned To: Misak Khachatryan
Depends on: 531519
  Show dependency treegraph
Reported: 2010-04-11 03:03 PDT by Misak Khachatryan
Modified: 2010-08-26 09:07 PDT (History)
0 users
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

patch (1.60 KB, patch)
2010-04-11 03:03 PDT, Misak Khachatryan
misak.bugzilla: review+
misak.bugzilla: superreview+
Details | Diff | Splinter Review
patch v2 (982 bytes, patch)
2010-04-11 10:00 PDT, Misak Khachatryan
neil: review+
neil: superreview+
dao+bmo: feedback-
Details | Diff | Splinter Review

Description User image Misak Khachatryan 2010-04-11 03:03:11 PDT
Created attachment 438335 [details] [diff] [review]

From parent bug: exposed a constant
failure on all platforms:
| number of open browser windows according to getBrowserState - Got 3, expected

This happens even when running browser_526613.js alone.

I think the second window (from _windows) and the third one (from
_statesToRestore) are the same, but I'm not sure.

I ran test on my Fedora 12, no failed test seen.
Comment 1 User image 2010-04-11 04:55:19 PDT
Comment on attachment 438335 [details] [diff] [review]

>+      if (this._windows[ix]._restoring) // window data is still in _statesToRestore
>+        continue;
Does checking (this._windows[ix].__SS_restoreID in this._statesToRestore) work? (That would avoid adding the extra _restoring flag.)
Comment 2 User image Misak Khachatryan 2010-04-11 10:00:06 PDT
Created attachment 438368 [details] [diff] [review]
patch v2

Well, seems to work, but as our browser_526613.js didn't fail on my Fedora 12, I'm requesting feedback from  Dão Gottwald, maybe he can add useful comment here.
Comment 3 User image 2010-04-11 10:06:55 PDT
Comment on attachment 438368 [details] [diff] [review]
patch v2

Let's prefer to go with this version if possible.
Comment 4 User image Dão Gottwald [:dao] 2010-04-12 03:35:34 PDT
Comment on attachment 438368 [details] [diff] [review]
patch v2

Looks wrong to me. __SS_restoreID is set on dom windows, this._windows[ix] is not a dom window.
Comment 5 User image Misak Khachatryan 2010-04-12 07:16:06 PDT
Comment on attachment 438335 [details] [diff] [review]

Thank You very much Dão, unobsoleting previous patch and setting r+ and sr+, as discussed with Neil on IRC.
Comment 6 User image Misak Khachatryan 2010-04-17 06:07:04 PDT
checking ping
Comment 7 User image Misak Khachatryan 2010-04-20 00:01:27 PDT
Comment 8 User image Misak Khachatryan 2010-08-26 07:50:05 PDT
Hmm, looks like i checked in wrong patch ...

Note You need to log in before you can comment on or make changes to this bug.