Closed Bug 726609 Opened 13 years ago Closed 12 years ago

Intermittent browser_dbg_select-line.js | The correct line is selected. - Got 0, expected 5 OR | The correct script was loaded initially.

Categories

(DevTools :: Debugger, defect, P2)

x86
Windows 7
defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 13

People

(Reporter: philor, Assigned: past)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

https://tbpl.mozilla.org/php/getParsedLog.php?id=9284998&tree=Fx-Team Rev3 WINNT 6.1 fx-team pgo test mochitest-other on 2012-02-12 21:25:54 PST for push c17bad2abafe TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_select-line.js | The correct line is selected. - Got 0, expected 5 Stack trace: JS frame :: chrome://mochikit/content/browser-test.js :: test_is :: line 442 JS frame :: chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_select-line.js :: <TOP_LEVEL> :: line 48 https://tbpl.mozilla.org/php/getParsedLog.php?id=9288918&tree=Profiling Rev3 WINNT 6.1 profiling opt test mochitest-other on 2012-02-13 02:09:20 PST for push a0d9d4b9c5e2 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_select-line.js | The correct line is selected. - Got 0, expected 5 Stack trace: JS frame :: chrome://mochikit/content/browser-test.js :: test_is :: line 442 JS frame :: chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_select-line.js :: <TOP_LEVEL> :: line 48
Status: NEW → ASSIGNED
Priority: -- → P2
Attached patch [in-fx-team] Working patch (deleted) — — Splinter Review
This seems to have survived try so far, so we might as well land it: https://tbpl.mozilla.org/?tree=Try&rev=05662d0aef7f
Attachment #597081 - Flags: review?(mihai.sucan)
Comment on attachment 597081 [details] [diff] [review] [in-fx-team] Working patch Review of attachment 597081 [details] [diff] [review]: ----------------------------------------------------------------- Patch looks good!
Attachment #597081 - Flags: review?(mihai.sucan) → review+
(In reply to Panos Astithas [:past] from comment #8) > Created attachment 597081 [details] [diff] [review] > Working patch > > This seems to have survived try so far, so we might as well land it: > > https://tbpl.mozilla.org/?tree=Try&rev=05662d0aef7f 100 runs without this error. Landed in fx-team: https://hg.mozilla.org/integration/fx-team/rev/4f8f17eacf31
Whiteboard: [orange] → [orange][fixed-in-fx-team]
Attachment #597081 - Attachment description: Working patch → [in-fx-team] Working patch
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [orange][fixed-in-fx-team] → [orange]
Target Milestone: --- → Firefox 13
That last one was well after your merge :(
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Summary: Intermittent browser_dbg_select-line.js | The correct line is selected. - Got 0, expected 5 → Intermittent browser_dbg_select-line.js | The correct line is selected. - Got 0, expected 5 OR | The correct script was loaded initially.
We are temporarily disabling this test in bug 734872.
I expect a better fix for this test would be something like in bug 728926. executeSoon() will only make the failures less frequent.
FILTER ON PUMPKINS.
Status: REOPENED → RESOLVED
Closed: 13 years ago12 years ago
Resolution: --- → FIXED
Whiteboard: [orange]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: