Closed Bug 548228 Opened 10 years ago Closed 10 years ago

Port Bug 528776 [getBrowserState considers closed windows as open] to SeaMonkey

Categories

(SeaMonkey :: Session Restore, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1a1

People

(Reporter: misak.bugzilla, Assigned: misak.bugzilla)

References

Details

Attachments

(1 file, 3 obsolete files)

From parent bug, this also can be the reason that some of tests failing:

Two things happened before this started:

http://hg.mozilla.org/mozilla-central/rev/4f62c9d94957
browser_477657.js, seems to open only one window?

http://hg.mozilla.org/mozilla-central/rev/01adc20ea792
browser_526613.js, backed out already

http://hg.mozilla.org/mozilla-central/rev/fa212b6a9d72
browser_394759.js
Attached patch patch plus test v1 (obsolete) — Splinter Review
Attachment #428673 - Flags: superreview?(neil)
Attachment #428673 - Flags: review?(neil)
Comment on attachment 428673 [details] [diff] [review]
patch plus test v1

>diff --git a/suite/common/tests/browser/Makefile.in b/suite/common/tests/browser/Makefile.in
>--- a/suite/common/tests/browser/Makefile.in
>+++ b/suite/common/tests/browser/Makefile.in
>@@ -55,6 +55,7 @@ _BROWSER_FILES = \
> 	browser_bug524371.js \
> 	browser_bug524371_sample.html \
> 	browser_bug548211.js \
>+        browser_bug548228.js \
> 	$(NULL)
Nit: spaces where the other lines have a tab.
Attachment #428673 - Flags: superreview?(neil)
Attachment #428673 - Flags: superreview+
Attachment #428673 - Flags: review?(neil)
Attachment #428673 - Flags: review+
Attached patch for checkin (obsolete) — Splinter Review
Nit fixed, carrying forward r+ and sr+ from Neil
Attachment #428673 - Attachment is obsolete: true
Attachment #428688 - Flags: superreview+
Attachment #428688 - Flags: review+
Keywords: checkin-needed
Comment on attachment 428688 [details] [diff] [review]
for checkin

>+  _handleClosedWidnows: function sss_handleClosedWidnows() {

Not "handleClosedWi_nd_ows"? (everywhere)
Sorry, fixed, carrying forward r+ and sr+ from Neil
Attachment #428688 - Attachment is obsolete: true
Attachment #428697 - Flags: superreview+
Attachment #428697 - Flags: review+
See bug 542946 comment 5 wrt the test name...
same as above, renamed test files as agreed, carrying forward r+ sr+ from Neil, got approval for renaming on meeting over IRC.
Attachment #428697 - Attachment is obsolete: true
Attachment #432100 - Flags: superreview+
Attachment #432100 - Flags: review+
Pushed as http://hg.mozilla.org/comm-central/rev/466f577ff66d - thanks for your work!
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a1
Blocks: 593681
You need to log in before you can comment on or make changes to this bug.