Closed Bug 1277107 Opened 5 years ago Closed 4 years ago

Intermittent browser_UITour_heartbeat.js | Heartbeat should appear in the most recent private window - Got false, expected true


(Firefox :: Tours, defect)

49 Branch
Not set



Tracking Status
firefox48 --- unaffected
firefox49 --- disabled
firefox50 --- disabled
firefox51 --- disabled


(Reporter: KWierso, Unassigned)



(Keywords: intermittent-failure)

Alessio, any idea why this might have gone of the rails in the last few days?
Flags: needinfo?(alessio.placitelli)
(In reply to Ryan VanderMeulen [:RyanVM] (Away June 4-20) from comment #2)
> Alessio, any idea why this might have gone of the rails in the last few days?

Mh, I'm afraid not. What I can think of is that we're not receiving something that makes |promisePageEvent| [1] resolve earlier.

Unfortunately I can't investigate this further right now. I will probably be able to in a couple of weeks. 

[1] -
Flags: needinfo?(alessio.placitelli)
This could be related to bug 1264108 as well.
This is awful.
Flags: needinfo?(alessio.placitelli)
Quite awful.
Whiteboard: [test disabled on Windows]
Keywords: leave-open
Pushed by
Skip browser_UITour_heartbeat.js on Windows where it fails constantly
Flags: needinfo?(alessio.placitelli)
Priority: -- → P2
Whiteboard: [test disabled on Windows] → [test disabled on Windows][measurement:client]
(In reply to Ryan VanderMeulen [:RyanVM] from comment #27)
> This is awful.

Yeah, quite awful. I took a quick look at the problem and seems to point to a real bug, on Windows.
I can reproduce the behaviour on Windows 10: if you run the test and then click elsewhere on the desktop, Heartbeat will fail to show in the most recent private window.

This happens because [1] returns null due to [2] not returning all the opened browser windows (in the specific case of the test, it only returns the top-most non private window).

I can't dig deeper into the issue due to time.

@Gregg, does your team have any engineer who can take it from there? Or, better, who owns that part? :-)

[1] -
[2] -
Flags: needinfo?(glind)
I claim this is a MattN, will NI him.
Flags: needinfo?(glind) → needinfo?(MattN+bmo)
Priority: P2 → P3
Priority: P3 → --
Whiteboard: [test disabled on Windows][measurement:client]
test doesn't exist in tree anymore
Closed: 4 years ago
Keywords: leave-open
Resolution: --- → INCOMPLETE
Flags: needinfo?(MattN+bmo)
You need to log in before you can comment on or make changes to this bug.