Closed Bug 853058 Opened 11 years ago Closed 11 years ago

Intermittent browser_dbg_bug723071_editor-breakpoints-pane.js | The correct script is selected - Didn't expect http://example.com/browser/browser/devtools/debugger/test/test-script-switching-02.js, but got it

Categories

(DevTools :: Debugger, defect, P3)

Product:
DevTools
Component:
Debugger
Platform:
x86_64
Linux
Type:
defect

Tracking

(firefox25 wontfix, firefox26 fixed, firefox-esr24 wontfix)

Status:
RESOLVED FIXED
Milestone:
Firefox 26
Tracking Flags:
Tracking Status
firefox25 --- wontfix
firefox26 --- fixed
firefox-esr24 --- wontfix

People

(Reporter: RyanVM, Assigned: vporof)

References

Details

(Keywords: intermittent-failure) 

https://tbpl.mozilla.org/php/getParsedLog.php?id=20888748&tree=Mozilla-Inbound

Rev3 Fedora 12x64 mozilla-inbound opt test mochitest-browser-chrome on 2013-03-20 09:22:11 PDT for push 5387c58141df
slave: talos-r3-fed64-069

09:35:32     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_bug723071_editor-breakpoints-pane.js | Console message: [JavaScript Warning: "Use of Mutation Events is deprecated. Use MutationObserver instead." {file: "chrome://browser/content/orion.js" line: 3408}]
09:35:32     INFO -  DBG-FRONTEND: Finished loading the DebuggerView editor
09:35:32     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_bug723071_editor-breakpoints-pane.js | Console message: [JavaScript Warning: "Use of Mutation Events is deprecated. Use MutationObserver instead." {file: "chrome://browser/content/orion.js" line: 3408}]
09:35:33     INFO -  DBG-FRONTEND: ThreadState is connecting...
09:35:33     INFO -  DBG-FRONTEND: Handling tab navigation in the ThreadState
09:35:33     INFO -  DBG-FRONTEND: StackFrames is connecting...
09:35:33     INFO -  DBG-FRONTEND: Handling tab navigation in the StackFrames
09:35:33     INFO -  DBG-FRONTEND: SourceScripts is connecting...
09:35:33     INFO -  DBG-FRONTEND: Handling tab navigation in the SourceScripts
09:35:33     INFO -  DBG-FRONTEND: Setting the DebuggerView editor source: http://example.com/browser/browser/devtools/debugger/test/test-script-switching-01.js, loaded: undefined
09:35:33     INFO -  DBG-FRONTEND: Updating the DebuggerView editor: http://example.com/browser/browser/devtools/debugger/test/test-script-switching-02.js @ 6, flags: ({})
09:35:33     INFO -  DBG-FRONTEND: Setting the DebuggerView editor source: http://example.com/browser/browser/devtools/debugger/test/test-script-switching-02.js, loaded: undefined
09:35:33     INFO -  DBG-FRONTEND: Updating the DebuggerView editor: http://example.com/browser/browser/devtools/debugger/test/test-script-switching-02.js @ 6, flags: ({noSwitch:true})
09:35:33     INFO -  TEST-PASS | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_bug723071_editor-breakpoints-pane.js | Should only be getting stack frames while paused.
09:35:33     INFO -  TEST-PASS | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_bug723071_editor-breakpoints-pane.js | Found the expected number of scripts.
09:35:33     INFO -  TEST-PASS | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_bug723071_editor-breakpoints-pane.js | The correct script was loaded initially.
09:35:33  WARNING -  TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_bug723071_editor-breakpoints-pane.js | The correct script is selected - Didn't expect http://example.com/browser/browser/devtools/debugger/test/test-script-switching-02.js, but got it
09:35:33     INFO -  Stack trace:
09:35:33     INFO -      JS frame :: chrome://mochikit/content/browser-test.js :: test_isnot :: line 506
09:35:33     INFO -      JS frame :: chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_bug723071_editor-breakpoints-pane.js :: performTest :: line 82
09:35:33     INFO -      native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0
09:35:33     INFO -  TEST-PASS | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_bug723071_editor-breakpoints-pane.js | no breakpoints
09:35:33     INFO -  TEST-PASS | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_bug723071_editor-breakpoints-pane.js | getBreakpoint('chocolate', 3) returns falsey
Summary: Intermittent → Intermittent browser_dbg_bug723071_editor-breakpoints-pane.js | The correct script is selected - Didn't expect http://example.com/browser/browser/devtools/debugger/test/test-script-switching-02.js, but got it
Anyone? Bueller?
Flags: needinfo?(past)
Panos is out this week for orthodox Easter, which is a national holiday in Greece.

Victor, probably knows this code the best.
Flags: needinfo?(vporof)
This doesn't make much sense to me. According to the flow, we're pretty clearly showing the right source by the time we get to the "The correct script was loaded initially." check (the test simply doesn't advance until test-script-switching-02.js is shown). Otherwise the whole test would've died in flames, nu just this one check.

This makes me suspect that gEditor.getText() is misbehaving.
An acceptable temporary change would be not to check the text contents, but the DebuggerView.Sources.selectedValue for the right url, but that would be pretty redundant anyway, and the fact that the source editor getText() is returning something different makes me nervous.

This wasn't so chatty until a couple of days a go. What landed that poked this orange?
Flags: needinfo?(vporof)
Flags: needinfo?(past)
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee: nobody → vporof
status-firefox25: --- → wontfix
status-firefox26: --- → fixed
status-firefox-esr24: --- → wontfix
Depends on: 876277
Target Milestone: --- → Firefox 26
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.