Closed Bug 784060 Opened 13 years ago Closed 13 years ago

Port Bug 768568 to SeaMonkey

Categories

(SeaMonkey :: Tabbed Browser, defect, P2)

Product:
SeaMonkey
Component:
Tabbed Browser
Type:
defect

Tracking

(firefox-esr10 unaffected, seamonkey2.12 fixed, seamonkey2.13 fixed)

Status:
VERIFIED FIXED
Milestone:
seamonkey2.14
Tracking Flags:
Tracking Status
firefox-esr10 --- unaffected
seamonkey2.12 --- fixed
seamonkey2.13 --- fixed

People

(Reporter: philip.chee, Assigned: philip.chee)

References

Details

(Keywords: regression, sec-high, Whiteboard: [perma-orange])

Attachments

(1 file)

Patch v1.0 Do it.
1.27 KB, patch
neil
: review+
sgautherie
: feedback+
Callek
: approval-comm-aurora+
Callek
: approval-comm-beta+
Details | Diff | Splinter Review
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1345402578.1345407329.8417.gz TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | Got a notification for the front notifications listener - Got onSecurityChange, expected onStateChange TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | Got a notification for the all notifications listener - Got onSecurityChange, expected onStateChange TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | Got an expected notification for the front notifications listener TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | Got a notification for the front notifications listener - Got onStateChange, expected undefined TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | Got an expected notification for the all notifications listener TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | Got a notification for the all notifications listener - Got onStateChange, expected undefined TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | Saw the expected number of notifications - Got 5, expected 4 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | Saw the expected number of frontnotifications - Got 5, expected 4 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | Got a notification for the all notifications listener - Got onSecurityChange, expected onStateChange TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | Got an expected notification for the all notifications listener TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | Got a notification for the all notifications listener - Got onStateChange, expected undefined TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | Saw the expected number of notifications - Got 5, expected 4
Before: INFO TEST-START | Shutdown Browser Chrome Test Summary Passed: 111 Failed: 16 Todo: 0 After: finished in 538ms TEST-INFO | checking window state TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | leaked until shutdown [nsGlobalWindow #25 https://example.com/browser/suite/browser/test/alltabslistener.html] TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | leaked until shutdown [nsGlobalWindow #24 http://example.org/browser/suite/browser/test/alltabslistener.html] TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | leaked until shutdown [nsGlobalWindow #23 https://example.com/browser/suite/browser/test/alltabslistener.html] TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_alltabslistener.js | leaked until shutdown [nsGlobalWindow #22 http://example.org/browser/suite/browser/test/alltabslistener.html] INFO TEST-START | Shutdown Browser Chrome Test Summary Passed: 123 Failed: 4 Todo: 0
Attachment #653409 - Flags: review?(neil)
Attachment #653409 - Flags: feedback?(sgautherie.bz)
Ftr, this patch undoes bug 752211. I can't say more until I can access bug 768568 :-/ "You are not authorized to access bug #768568."
Severity: normal → major
Comment on attachment 653409 [details] [diff] [review] Patch v1.0 Do it. Bah, so Gecko has to work around bugs in Firefox front end code?
Attachment #653409 - Flags: review?(neil) → review+
(In reply to Serge Gautherie from comment #2) > Ftr, this patch undoes bug 752211. That's not surprising, since 768568 undoes parts of 745254.
Pushed to comm-central http://hg.mozilla.org/comm-central/rev/14441f0e2747 Leaving open for comm-aurora and comm-beta.
Target Milestone: --- → seamonkey2.14
Attachment #653409 - Flags: approval-comm-beta?
Attachment #653409 - Flags: approval-comm-beta+
Attachment #653409 - Flags: approval-comm-aurora?
Attachment #653409 - Flags: approval-comm-aurora+
http://hg.mozilla.org/releases/comm-beta/rev/97c9fb2486da
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
status-seamonkey2.12: --- → fixed
Resolution: --- → FIXED
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1345544030.1345547174.646.gz Linux comm-central-trunk debug test mochitest-other on 2012/08/21 03:13:50 V.Fixed
Status: RESOLVED → VERIFIED
status-seamonkey2.13: --- → affected
Flags: in-testsuite+
Group: core-security
No longer blocks: SmTestFail
Depends on: CVE-2012-3976
Summary: Port Bug 768568 to SeaMonkey. → Port Bug 768568 to SeaMonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: