Closed
Bug 1439763
Opened 7 years ago
Closed 7 years ago
Update Debugger Frontend v18
Categories
(DevTools :: Debugger, enhancement, P3)
DevTools
Debugger
Tracking
(firefox60 fixed)
RESOLVED
FIXED
Firefox 60
Tracking | Status | |
---|---|---|
firefox60 | --- | fixed |
People
(Reporter: jlast, Assigned: jlast)
References
Details
Attachments
(1 file)
(deleted),
patch
|
jdescottes
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → jlaster
Blocks: debugger-bundle-updates
Priority: -- → P3
Summary: Up → Update Debugger Frontend v18
Assignee | ||
Comment 1•7 years ago
|
||
Assignee | ||
Comment 2•7 years ago
|
||
Attachment #8952561 -
Flags: review?(jdescottes)
Updated•7 years ago
|
Severity: normal → enhancement
Status: NEW → ASSIGNED
Comment 3•7 years ago
|
||
Comment on attachment 8952561 [details] [diff] [review]
rel18.patch
Review of attachment 8952561 [details] [diff] [review]:
-----------------------------------------------------------------
- Meta:
- assigned, P3, enhancement, blocks meta: OK
- commit message: OK
- less than 20 commits: OK
- TRY
- try run green: OK, but many failures for the intermittent https://bugzilla.mozilla.org/show_bug.cgi?id=1434868, need to narrow it down
- try run has rebuild: OK
- Manual test
- OK
- README review
- compare link: OK
- version number: OK
- Properties file review: N/A
- New files: OK (but are the new example files used?)
::: devtools/client/debugger/new/README.mozilla
@@ +6,2 @@
>
> Packages:
I think the information below is outdated? Can we remove it for the next release?
::: devtools/client/debugger/new/test/mochitest/examples/doc-babel.html
@@ +41,5 @@
> + <button onclick="stepOverForOfClosure()">Run stepOverForOfClosure</button>
> + <script src="babel/fixtures/step-over-function-params/output.js"></script>
> + <button onclick="stepOverFunctionParams()">Run stepOverFunctionParams</button>
> + <script src="babel/fixtures/step-over-regenerator-await/output.js"></script>
> + <button onclick="stepOverRegeneratorAwait()">Run stepOverRegeneratorAwait</button>
I can't see where these new buttons (and all the new files) are being used? I assumed new test code should be added to browser-dbg-babel.js, but can't see anything there. There's been a lot of new source files added recently so I'd like to make sure we are using them.
Attachment #8952561 -
Flags: review?(jdescottes) → review+
Comment 4•7 years ago
|
||
Comment 5•7 years ago
|
||
Some other changes seem to pollute the results. Compare with baseline : https://treeherder.mozilla.org/perf.html#/compare?originalProject=try&originalRevision=047158625a62&newProject=try&newRevision=b12120663002c2b63f0e6a4df51df7c4b5fcfdfb&framework=1 (no results yet)
Comment 6•7 years ago
|
||
No performance improvement nor regression detected.
Pushed by jdescottes@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/bb629a5223e1
Update Debugger Frontend v18. r=jdescottes
Comment 8•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox60:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•