Update Debugger frontend (7/24/2017).

RESOLVED FIXED in Firefox 56

Status

enhancement
P3
normal
RESOLVED FIXED
2 years ago
Last year

People

(Reporter: jlast, Assigned: jlast)

Tracking

unspecified
Firefox 56
Dependency tree / graph

Firefox Tracking Flags

(firefox56 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

includes lots of polish and some fixes for breakpoints and the new Editor editor.state.editor functionality.
Posted patch 7-24-1.patch (obsolete) — Splinter Review
Attachment #8889503 - Flags: review?(jdescottes)
Assignee: nobody → jlaster
Status: NEW → ASSIGNED
Priority: -- → P3
Comment on attachment 8889503 [details] [diff] [review]
7-24-1.patch

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

Looks good to me, thanks. 
I only did very minimal testing, but looks like breakpoints no longer disappear, great!

::: devtools/client/sourceeditor/editor.js
@@ +462,5 @@
>    replaceDocument: function (doc) {
>      let cm = editors.get(this);
>      cm.swapDoc(doc);
> +    if (!Services.prefs.getBoolPref("devtools.debugger.new-debugger-frontend")) {
> +      this._updateLineNumberFormat();

(just as a sidenote, there is another call to updateLineNumberFormat in setText(), is this one fine for the new debugger?)
Attachment #8889503 - Flags: review?(jdescottes) → review+
Posted patch 7-24-2.patchSplinter Review
Attachment #8889503 - Attachment is obsolete: true
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a9721c2c4368
Update debugger frontend (v0.9.0). r=jdescottes
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/a9721c2c4368
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Blocks: 1380510
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.