Closed
Bug 605884
Opened 14 years ago
Closed 12 years ago
test_placesTxn.js leaks like a sieve (editBookmarkMicrosummary loads remote content)
Categories
(Toolkit :: Places, defect)
Tracking
()
RESOLVED
INVALID
People
(Reporter: jdm, Unassigned)
References
Details
(Keywords: memory-leak, Whiteboard: [xpcshell-leak])
Attachments
(1 file)
|
(deleted),
text/plain
|
Details |
No description provided.
|
||
Updated•14 years ago
|
|
Reporter | |
Updated•14 years ago
|
|
||
Comment 1•14 years ago
|
||
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.
|
|
||
Updated•14 years ago
|
Version: unspecified → Trunk
|
||
Updated•14 years ago
|
Summary: test_placesTxn.js leaks like a sieve → test_placesTxn.js leaks like a sieve (editBookmarkMicrosummary loads remote content)
|
|
||
Comment 2•14 years ago
|
||
Is that test really loading remote content? Those URLs are 404 for me.
Is this in fact just bug 626999 comment 3?
|
|
||
Comment 3•14 years ago
|
||
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.
|
|
||
Comment 4•14 years ago
|
||
Aha, okay. FWIW, in bug 626999 I did run xpcshell tests with Wireshark running, and I didn't see any external network access.
|
||
Comment 5•14 years ago
|
||
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
|
|
||
Comment 6•12 years ago
|
||
microsummaries died so this is now invalid
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•