617967 - check: TestTXMgr.cpp (nsITransactionManager tests) reports some warnings in nsTransactionManager.cpp and nsTransactionItem.cpp

Status: NEW

(Core :: DOM: Editor, defect, P5)

Description

[Serge Gautherie (:sgautherie)]

(Noticed on my local Windows 2000 while test bug 615045 fix.)

{
Call DoTransaction() with null transaction ... WARNING: NS_ENSURE_TRUE(aTransact
ion) failed: file e:/Dvlp/Mozilla/Hg/comm-central/mozilla/editor/txmgr/src/nsTra
nsactionManager.cpp, line 100
passed
...
Call AddListener() with null listener ... WARNING: NS_ENSURE_TRUE(aListener) fai
led: file e:/Dvlp/Mozilla/Hg/comm-central/mozilla/editor/txmgr/src/nsTransaction
Manager.cpp, line 632
passed
Call RemoveListener() with null listener ... WARNING: NS_ENSURE_TRUE(aListener)
failed: file e:/Dvlp/Mozilla/Hg/comm-central/mozilla/editor/txmgr/src/nsTransact
ionManager.cpp, line 646
passed
...
Test transaction RedoTransaction() error ... WARNING: NS_ENSURE_SUCCESS(result,
result) failed with result 0x80004005: file e:/Dvlp/Mozilla/Hg/comm-central/mozi
lla/editor/txmgr/src/nsTransactionItem.cpp, line 318
...
WARNING: Failed to get XPConnect?!: 'rts', file e:/Dvlp/Mozilla/Hg/comm-central/
mozilla/xpcom/base/nsCycleCollector.cpp, line 3415
}

Comment 1

[(no longer active)]

None of these seem very important, except for the last one.

Comment 2

[Serge Gautherie (:sgautherie)]

"WARNING: Failed to get XPConnect?!: 'rts'" was fixed by bug 618365.

*****

Bug still there on SM 2.12a1.


http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1337791331.1337796853.4286.gz&fulltext=1
Linux comm-central-trunk leak test build on 2012/05/23 09:42:11
s: cb-sea-linux-tbox
{
make[4]: Entering directory `/builds/slave/comm-cen-trunk-lnx-dbg/build/objdir/mozilla/editor/txmgr/tests'
WARNING: NS_ENSURE_TRUE(aTransaction) failed: file /builds/slave/comm-cen-trunk-lnx-dbg/build/mozilla/editor/txmgr/src/nsTransactionManager.cpp, line 59
WARNING: NS_ENSURE_TRUE(aListener) failed: file /builds/slave/comm-cen-trunk-lnx-dbg/build/mozilla/editor/txmgr/src/nsTransactionManager.cpp, line 466
WARNING: NS_ENSURE_TRUE(aListener) failed: file /builds/slave/comm-cen-trunk-lnx-dbg/build/mozilla/editor/txmgr/src/nsTransactionManager.cpp, line 474
WARNING: NS_ENSURE_SUCCESS(result, result) failed with result 0x80004005: file /builds/slave/comm-cen-trunk-lnx-dbg/build/mozilla/editor/txmgr/src/nsTransactionItem.cpp, line 277
WARNING: NS_ENSURE_TRUE(aTransaction) failed: file /builds/slave/comm-cen-trunk-lnx-dbg/build/mozilla/editor/txmgr/src/nsTransactionManager.cpp, line 59
WARNING: NS_ENSURE_TRUE(aListener) failed: file /builds/slave/comm-cen-trunk-lnx-dbg/build/mozilla/editor/txmgr/src/nsTransactionManager.cpp, line 466
WARNING: NS_ENSURE_TRUE(aListener) failed: file /builds/slave/comm-cen-trunk-lnx-dbg/build/mozilla/editor/txmgr/src/nsTransactionManager.cpp, line 474
WARNING: NS_ENSURE_SUCCESS(result, result) failed with result 0x80004005: file /builds/slave/comm-cen-trunk-lnx-dbg/build/mozilla/editor/txmgr/src/nsTransactionItem.cpp, line 277
Running nsITransactionManager tests...
[...]
TEST-PASS | Test unbalanced EndBatch() with empty undo stack
TEST-PASS | Test empWARNING: NS_ENSURE_SUCCESS(result, result) failed with result 0x80004005: file /builds/slave/comm-cen-trunk-lnx-dbg/build/mozilla/editor/txmgr/src/nsTransactionItem.cpp, line 277
WARNING: NS_ENSURE_SUCCESS(result, result) failed with result 0x80004005: file /builds/slave/comm-cen-trunk-lnx-dbg/build/mozilla/editor/txmgr/src/nsTransactionItem.cpp, line 277
ty batch
TEST-PASS | Execute 20 batched transactions
}


http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1337779595.1337789355.28349.gz&fulltext=1
OS X 10.6 comm-central-trunk leak test build on 2012/05/23 06:26:35
s: cb-sea-miniosx64-02

Better log: warnings are logged synchronously (= when they actually happen) :-)


http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1337776589.1337790413.29795.gz&fulltext=1
WINNT 5.2 comm-central-trunk leak test build on 2012/05/23 05:36:29
s: cb-seamonkey-win32-01

Same as Linux.

Comment 3

[Jens Stutte [:jstutte]]

Bulk-downgrade of unassigned, untouched DOM/Storage bug's priority.

If you have reason to believe, this is wrong, please write a comment and ni :jstutte.