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

Status: RESOLVED INVALID

(Toolkit :: Places, defect)

Description

[Josh Matthews [:jdm]]

Attachment: Leaks

Comment 1

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

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.

Comment 2

[(not currently active) Ted Mielczarek]

Is that test really loading remote content? Those URLs are 404 for me.
Is this in fact just bug 626999 comment 3?

Comment 3

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

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

[(not currently active) Ted Mielczarek]

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

[Phil Ringnalda (:philor)]

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.

Comment 6

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

microsummaries died so this is now invalid