Closed Bug 1076830 Opened 10 years ago Closed 10 years ago

Intermittent browser_dbg_breakpoints-reload.js | Test timed out - expected PASS

Categories

(DevTools :: Debugger, defect)

x86
Windows 8
defect
Not set
normal

Tracking

(firefox33 unaffected, firefox34 unaffected, firefox35 fixed, firefox-esr31 unaffected)

RESOLVED FIXED
Firefox 35
Tracking Status
firefox33 --- unaffected
firefox34 --- unaffected
firefox35 --- fixed
firefox-esr31 --- unaffected

People

(Reporter: cbook, Assigned: fitzgen)

References

()

Details

(Keywords: intermittent-failure)

Attachments

(2 files)

WINNT 6.2 fx-team debug test mochitest-devtools-chrome-1

https://treeherder.mozilla.org/ui/logviewer.html#?job_id=803947&repo=fx-team

03:35:51 INFO - 270 INFO TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_breakpoints-reload.js | Test timed out - expected PASS
Assignee: nobody → nfitzgerald
Status: NEW → ASSIGNED
Please tell me this is a top priority for you. Otherwise I'm going to be looking for a backout/disabling in VERY short order.
Flags: needinfo?(nfitzgerald)
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #49)
> Please tell me this is a top priority for you. Otherwise I'm going to be
> looking for a backout/disabling in VERY short order.

Yes, I'm working on this.
Flags: needinfo?(nfitzgerald)
Disabled on Linux debug for frequent failures.
https://hg.mozilla.org/mozilla-central/rev/6ecd3345b311
Whiteboard: [test disabled on Linux debug]
Ok, so I'm 99% sure the test script was getting GC'd before the debugger opens and gets the list of sources.

FML AMIRITE?

https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=f1924a5dc801
Attachment #8501425 - Flags: review?(past)
Comment on attachment 8501425 [details] [diff] [review]
intermittent-name-test-function.patch

Review of attachment 8501425 [details] [diff] [review]:
-----------------------------------------------------------------

Fun! You also need to enable the test again.
Attachment #8501425 - Flags: review?(past) → review+
Attached patch re-enable.patchSplinter Review
Have multiple green pushes here with the other attachment fix (try push doesn't include the commit where the test was enabled): https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=f1924a5dc801
Attachment #8501891 - Flags: review?(ryanvm)
Comment on attachment 8501891 [details] [diff] [review]
re-enable.patch

Review of attachment 8501891 [details] [diff] [review]:
-----------------------------------------------------------------

Worth a shot.
Attachment #8501891 - Flags: review?(ryanvm) → review+
https://hg.mozilla.org/integration/fx-team/rev/d864cbdfee34
https://hg.mozilla.org/integration/fx-team/rev/424b31d77d2a
Keywords: checkin-needed
Whiteboard: [test disabled on Linux debug] → [test disabled on Linux debug][fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/d864cbdfee34
https://hg.mozilla.org/mozilla-central/rev/424b31d77d2a
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [test disabled on Linux debug][fixed-in-fx-team] → [test disabled on Linux debug]
Target Milestone: --- → Firefox 35
Whiteboard: [test disabled on Linux debug]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.