Closed Bug 1488917 Opened 6 years ago Closed 6 years ago

Update Debugger Frontend v87

Categories

(DevTools :: Debugger, enhancement, P3)

enhancement

Tracking

(firefox64 fixed)

RESOLVED FIXED
Firefox 64
Tracking Status
firefox64 --- fixed

People

(Reporter: jlast, Assigned: jlast)

References

Details

Attachments

(1 file, 2 obsolete files)

Attached patch 87-1.patch (obsolete) — Splinter Review
Attachment #9006718 - Flags: review?(dwalsh)
Heads up, this new revision breaks browser/base/content/test/static/browser_parsable_css.js for 3 reasons:
- need to remove --theme-search-overlays-semitransparent from the test
- need to remove rules pointing to chrome://devtools/skin/images/debugger/back.svg in debugger.css (image was deleted)
- need to remove rules pointing to chrome://devtools/skin/images/debugger/forward.svg in debugger.css (image was deleted)
Attached patch 87-2.patch (obsolete) — Splinter Review
Attachment #9006718 - Attachment is obsolete: true
Attachment #9006718 - Flags: review?(dwalsh)
Attachment #9006894 - Flags: review?(dwalsh)
Attached patch 87-3.patchSplinter Review
Attachment #9006894 - Attachment is obsolete: true
Attachment #9006894 - Flags: review?(dwalsh)
Attachment #9006897 - Flags: review?(dwalsh)
Comment on attachment 9006897 [details] [diff] [review]
87-3.patch

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

There's no README update but functionality it's good to go.
Attachment #9006897 - Flags: review?(dwalsh) → review+
https://hg.mozilla.org/mozilla-central/rev/361fa9dbf7e9
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: