Closed Bug 1173801 Opened 4 years ago Closed 4 years ago

Intermittent browser_privatebrowsing_DownloadLastDirWithCPS.js | Test timed out - expected PASS


(Firefox :: Private Browsing, defect)

Not set



Firefox 42
Tracking Status
firefox40 --- fixed
firefox41 --- fixed
firefox42 --- affected
firefox-esr31 --- unaffected
firefox-esr38 --- unaffected


(Reporter: cbook, Assigned: ehsan)




(Keywords: intermittent-failure)


(2 files)


06:15:25 INFO - 897 INFO TEST-UNEXPECTED-FAIL | browser/components/privatebrowsing/test/browser/browser_privatebrowsing_DownloadLastDirWithCPS.js | Test timed out - expected PASS
Felipe, any chance you could look into this recent spike on win8?
Flags: needinfo?(felipc)
Ryan, I had landed bug 1129957 and it was backed out for apparently having caused that. Does that mean that patch is likely not to blame here? (I was skeptical that those changes could cause anything related to this)

I can take a look but I don't have quick access to a win8 64bit env, as this is only happening there. I can try to add some more logging to get more details from it when it fails, but if it's bothering too much I vote to skip it on win8 64.
Flags: needinfo?(felipc) → needinfo?(ryanvm)
If you want to land a logging patch, be my guest. It's frequent enough that *somebody* needs to do something about it. If you want to disable, that's your choice.
Flags: needinfo?(ryanvm)
My main question was about whether it sounds right that backing out bug 1129957 was a red herring, or if I'm missing something (like: the backout hadn't been merged yet to the trees where this is happening).
As we discussed on IRC, the retriggers were pretty clearly pointing at your original push as when it started the first time around. I think the fact that it resurfaced later points to an underlying condition being perturbed into failing.
Comment on attachment 8622485 [details] [diff] [review]
Skip test on Win8 64bit

Review of attachment 8622485 [details] [diff] [review]:

You don't need the 64bits part or the os_version == 6.3 part. We're unlikely to ever run tests on 8.1 (will probably go straight to 10 at this point) and we only run win8 tests on 64bit at this point.
Attachment #8622485 - Flags: review?(ryanvm) → review+
Assignee: nobody → felipc
Whiteboard: [test disabled on Windows 8]