wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 546445 - Asynchronous getPref triggers debug assertion when shutting down sqlite. r=sdwilsh

Browse Source
This commit is contained in:
Ryan Flint 2010-02-23 12:19:36 -05:00
parent b4e4a16416
commit 47d53a6d69
2 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
toolkit/components/contentprefs/src/nsContentPrefService.js
View File

@ -101,6 +101,12 @@ ContentPrefService.prototype = {
_destroy: function ContentPrefService__destroy() {
this._observerSvc.removeObserver(this, "xpcom-shutdown");
// Finalize statements which may have been used asynchronously.
if (this.__stmtSelectPref)
this.__stmtSelectPref.finalize();
if (this.__stmtSelectGlobalPref)
this.__stmtSelectGlobalPref.finalize();
// Delete references to XPCOM components to make sure we don't leak them
// (although we haven't observed leakage in tests). Also delete references
// in _observers and _genericObservers to avoid cycles with those that

4
toolkit/components/contentprefs/tests/unit/test_getPrefAsync.js
View File

@ -38,10 +38,6 @@ var cps = Cc["@mozilla.org/content-pref/service;1"].
var uri = ContentPrefTest.getURI("http://www.example.com/");
function run_test() {
//XXX Temporarily disabled due to sqlite assertions
do_check_true(true);
return;
do_test_pending();
cps.setPref(uri, "asynctest", "pie");