Closed Bug 605884 Opened 10 years ago Closed 8 years ago

test_placesTxn.js leaks like a sieve (editBookmarkMicrosummary loads remote content)

Categories

(Toolkit :: Places, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: jdm, Unassigned)

References

Details

(Keywords: memory-leak, Whiteboard: [xpcshell-leak])

Attachments

(1 file)

Attached file Leaks
No description provided.
Blocks: 606067
No longer blocks: 606067
Depends on: 469523
Blocks: 469523
No longer depends on: 469523
fwiw, what leaks here are microsummaries, and they are trying to access remote content.
If you want to comment out "editBookmarkMicrosummary" section of the test and file a new bug to make it use local content rather than remote one, it will be fine for us.
Blocks: 626999
Version: unspecified → Trunk
Keywords: mlk
Summary: test_placesTxn.js leaks like a sieve → test_placesTxn.js leaks like a sieve (editBookmarkMicrosummary loads remote content)
Is that test really loading remote content? Those URLs are 404 for me.
Is this in fact just bug 626999 comment 3?
hm right this is a 404, it was not! but actually this test should not rely on stuff on dietrich's web page.
Btw, bug 626999 comment 3 seems to point to chrome tests, this is just a xpcshell test.
Aha, okay. FWIW, in bug 626999 I did run xpcshell tests with Wireshark running, and I didn't see any external network access.
I did too, and didn't see anything other than DNS queries for dietrich's site, so I don't think this actually blocks bug 626999.
No longer blocks: 626999
microsummaries died so this is now invalid
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.