Closed
Bug 1441635
Opened 7 years ago
Closed 7 years ago
Update Debugger Frontend v20
Categories
(DevTools :: Debugger, enhancement, P3)
DevTools
Debugger
Tracking
(firefox61 fixed)
RESOLVED
FIXED
Firefox 61
Tracking | Status | |
---|---|---|
firefox61 | --- | fixed |
People
(Reporter: jlast, Assigned: jlast)
References
Details
Attachments
(2 files, 3 obsolete files)
(deleted),
patch
|
jdescottes
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jlast
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•7 years ago
|
||
Assignee | ||
Comment 2•7 years ago
|
||
Assignee | ||
Comment 3•7 years ago
|
||
Assignee | ||
Comment 4•7 years ago
|
||
a friendly bisect run to help diagnose an issue (if it comes up) https://treeherder.mozilla.org/#/jobs?repo=try&revision=96d1a3df24248275531a325058ff56a9068ea0b7
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → jlaster
Blocks: debugger-bundle-updates
Assignee | ||
Comment 5•7 years ago
|
||
Attachment #8958258 -
Flags: review?(jdescottes)
Assignee | ||
Comment 6•7 years ago
|
||
Attachment #8958258 -
Attachment is obsolete: true
Attachment #8958258 -
Flags: review?(jdescottes)
Attachment #8958260 -
Flags: review?(jdescottes)
Assignee | ||
Comment 7•7 years ago
|
||
Attachment #8958260 -
Attachment is obsolete: true
Attachment #8958260 -
Flags: review?(jdescottes)
Attachment #8958263 -
Flags: review?(jdescottes)
Assignee | ||
Comment 8•7 years ago
|
||
Attachment #8958265 -
Flags: review?(jdescottes)
Assignee | ||
Comment 9•7 years ago
|
||
Initial try runs are great, here's a new one with all the re-runs
https://treeherder.mozilla.org/#/jobs?repo=try&revision=91a08d13a0e5d5b195cf328cdcf6f97288fbfcb8
Assignee | ||
Comment 10•7 years ago
|
||
Comment 11•7 years ago
|
||
Comment on attachment 8958263 [details] [diff] [review]
rel-19-2.4.patch
Review of attachment 8958263 [details] [diff] [review]:
-----------------------------------------------------------------
R+, can we double check the README though?
::: devtools/client/debugger/new/README.mozilla
@@ +1,4 @@
> This is the debugger.html project output.
> See https://github.com/devtools-html/debugger.html
>
> Version 19.2
So this release contains no commit, but rolls back react, react-dom and webpack?
Attachment #8958263 -
Flags: review?(jdescottes) → review+
Comment 12•7 years ago
|
||
Comment on attachment 8958265 [details] [diff] [review]
rel-20-1.patch
Review of attachment 8958265 [details] [diff] [review]:
-----------------------------------------------------------------
README update is missing.
Attachment #8958265 -
Flags: review?(jdescottes)
Comment 13•7 years ago
|
||
- Meta:
- assigned, P3, enhancement, blocks meta: OK
- commit message: OK
- less than 20 commits: ?
- Try runs
- try run green: OK
- try run has rebuild: OK
- Manual test: OK
- README review
- compare link: ?
- commit link: ?
- version number: ?
- Properties file review: OK
- New files: No new files
Severity: normal → enhancement
Status: NEW → ASSIGNED
Priority: -- → P3
Comment 14•7 years ago
|
||
Assignee | ||
Comment 15•7 years ago
|
||
Attachment #8958265 -
Attachment is obsolete: true
Attachment #8958423 -
Flags: review+
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Comment 16•7 years ago
|
||
Pushed by jlaster@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/eb3619ae3425
Re-align webpack module ordering. r=jdescottes
https://hg.mozilla.org/integration/mozilla-inbound/rev/907c98c2f3c8
Update Debugger Frontend v20. r=jdescottes
Keywords: checkin-needed
Comment 17•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/eb3619ae3425
https://hg.mozilla.org/mozilla-central/rev/907c98c2f3c8
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
Comment 18•7 years ago
|
||
Talos results:
- 2% slowdown on complicated.jsdebugger.open
https://treeherder.mozilla.org/perf.html#/comparesubtest?originalProject=try&originalRevision=2aef04d16f6be8b167ac6b67cf5b975a2664f654&newProject=try&newRevision=bc27f15b586c11f58dbefa24d05c9f306f48e8c8&originalSignature=f79b6a4f1f8f53326d3056f6c8008c0ff4de0a94&newSignature=f79b6a4f1f8f53326d3056f6c8008c0ff4de0a94&showOnlyConfident=1&framework=1
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•