Closed Bug 1260129 Opened 8 years ago Closed 7 years ago

Remove editor state and always load source text

Categories

(DevTools :: Debugger, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: jlong, Assigned: jlast)

Details

One bug 1233927 lands, we don't need any of these shenanigans: https://github.com/mozilla/gecko-dev/blob/master/devtools/client/debugger/debugger-view.js#L524-L526 We can just unconditionally load whatever text is there because the Documents cache speeds it up.

We still need to respect `opts` and make sure to set the cursor position in the correct place. But we are explicitly setting the cursor to `0, 0` current anyway, so I think we just need to pull the position from opts instead.
Assignee: nobody → jlaster
This looks like cleanup to me, so assigning P3 to this.
Priority: -- → P3
closing as this is not a priority in debugger.html
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.