Closed Bug 1213030 Opened 9 years ago Closed 8 years ago

Intermittent testEventDispatcher | Should have completed event before timeout -

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox45 fixed, firefox46 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 46
Tracking Flags:
Tracking Status
firefox45 --- fixed
firefox46 --- fixed

People

(Reporter: KWierso, Assigned: gbrown)

References

Details

(Keywords: intermittent-failure)

tracking-fennec: --- → ?
Assignee: nobody → margaret.leibovic
tracking-fennec: ? → ---
To see if increasing the wait time helps: https://treeherder.mozilla.org/#/jobs?repo=try&revision=eb3d1db9ae2e
(In reply to Geoff Brown [:gbrown] from comment #13)
> To see if increasing the wait time helps:
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=eb3d1db9ae2e

That does seem effective (the one failure is in a different test and unrelated). 

Compare to: https://treeherder.mozilla.org/#/jobs?repo=try&revision=fd533eff7f96 (no changes, same parent rev)

:margaret -- Do you have a better solution?
(In reply to Geoff Brown [:gbrown] from comment #14)
> (In reply to Geoff Brown [:gbrown] from comment #13)
> > To see if increasing the wait time helps:
> > https://treeherder.mozilla.org/#/jobs?repo=try&revision=eb3d1db9ae2e
> 
> That does seem effective (the one failure is in a different test and
> unrelated). 
> 
> Compare to:
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=fd533eff7f96 (no
> changes, same parent rev)
> 
> :margaret -- Do you have a better solution?

I have not had a chance to look into this today. Maybe jchen has ideas about what could be causing these async events to take so much time? Looks like this code was added in bug 1199444.

But r=me if you want to go ahead and land that fix in the meantime!
Blocks: 1199444
Flags: needinfo?(nchen)
Not sure why. Maybe the background thread is busy doing other unrelated tasks post-startup. I'm fine with the change if it works!
Flags: needinfo?(nchen)
gbrown, want to land that change then?
Assignee: margaret.leibovic → gbrown
Flags: needinfo?(gbrown)
Flags: needinfo?(gbrown)
https://hg.mozilla.org/mozilla-central/rev/00e82b640de3
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox46: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.