854510 - UNEXPECTED-PASS: js/xpconnect/tests/chrome/test_bug503926.xul - Assertion count 0 is less than expected range 1-1 assertions.

Status: RESOLVED FIXED

(Core :: XPConnect, defect)

Description

[Bobby Holley (:bholley)]

This has been going red on my local machine for several weeks now. I'm assuming the fact that it's not red on tinderbox means that it asserts there, so I guess this should be turned into a range. Patch coming up.

Comment 1

[Bobby Holley (:bholley)]

Attachment: Convert expected assertion into a 0-1 range. v1

Comment 2

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 729100 [details] [diff] [review]
Convert expected assertion into a 0-1 range. v1

So something in your patch queue fixes it then? Great! Just don't forget to drop the annotation when you land whatever it is that did.

Comment 3

[Ryan VanderMeulen [:RyanVM]]

Though if that's the case, I'd prefer that you just land that patch with the fix and not land this one.

Comment 4

[Bobby Holley (:bholley)]

(In reply to Ryan VanderMeulen [:RyanVM] from comment #2)

> So something in your patch queue fixes it then? Great! Just don't forget to
> drop the annotation when you land whatever it is that did.

No. The test just doesn't assert when run locally. I don't even know what the assertion is.

Comment 5

[Bobby Holley (:bholley)]

For reference, the assertion was:

ASSERTION: Existing entry in StartupCache.: 'entry == nullptr', file ../../startupcache/StartupCache.cpp

https://hg.mozilla.org/integration/mozilla-inbound/rev/78d672df644f

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/78d672df644f