Closed Bug 464816 Opened 11 years ago Closed 11 years ago

<test_384370.js> and 7 <test_places/bookmarks/*> leak |nsNavHistory*| objects

Categories

(Firefox :: Bookmarks & History, defect)

x86
Windows 2000
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: sgautherie, Unassigned)

References

()

Details

(Keywords: memory-leak)

Attachments

(1 file)

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1b2pre) Gecko/20081113 Minefield/3.1b2pre] (home, optim default) (W2Ksp4)
Flags: wanted-firefox3.1?
Attached file test_384370.js.log
Blocks: mlkTests
 PlacesUtils.tagging.tagURI(u, t);
this tells me could be same leak as bug 462545
Depends on: 456414
No longer depends on: 456414
Depends on: 456414
And 7 <_tests/xpcshell-simple/test_places/bookmarks/*> too:

test_405938_restore_queries.js
test_417228-exclude-from-backup.js
test_417228-other-roots.js
test_424958-json-quoted-folders.js
test_448584.js
test_458683.js
test_restore_guids.js

***

(In reply to comment #2)
> this tells me could be same leak as bug 462545

The leak(s) looks quite the same indeed.
Depends on: 462545
Summary: test_384370.js leaks → <test_384370.js> and 7 <test_places/bookmarks/*> leak |nsNavHistory*| objects
i can't see that anymore, Serge?
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.2a1pre) Gecko/20081210 Minefield/3.2a1pre] (home, optim default) (W2Ksp4)
(http://hg.mozilla.org/mozilla-central/rev/89a175c70c25)

Bug 463513 fixed these leaks.
Status: NEW → RESOLVED
Closed: 11 years ago
Depends on: 463513
No longer depends on: 462545
Flags: wanted-firefox3.1?
Resolution: --- → FIXED
(In reply to comment #5)

Ftr, new test_preventive_maintenance.js (from bug 431558) had same leak and was fixed too.
(In reply to comment #5)

This also fixed this part of test_placesTxn.js leak. (See bug 449240 comment 22.)
Depends on: 469523
No longer depends on: 456414
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h".

In Thunderbird 3.0b, you do that as follows:
Tools | Message Filters
Make sure the correct account is selected. Click "New"
Conditions: Body   contains   places-to-b-and-h
Change the action to "Delete Message".
Select "Manually Run" from the dropdown at the top.
Click OK.

Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter.

Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in before you can comment on or make changes to this bug.