Here is an early try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=9c82be14e245f86c7ec820f14627d7a2b9338fd9 - The patch seems reasonable. about 7K +- lines because we're uprgrading reps. - We should be careful to not overwrite Gabe's photon changes in debugger.css

here's a new try run w/ the fixes to the mochitests (needed to include another commit w/ a reps bump) https://treeherder.mozilla.org/#/jobs?repo=try&revision=fda59e415cdbd70f86f2249acdc03c510d2156d7

okay, we've got a failure in the optimized builds... not a big deal. will investigate tomorrow

jlast: FYI the release includes a huge 10MB webpack-stats.html file. Make sure to remove.

Two issues here: 1/ we regressed the panel.isPaused method, it's now missing! 2/ the browser toolbox test injects in the browser toolbox process: - a new-debugger test - the new-debugger head.js But it doesn't have access to shared-head. And it just happens that waitForPaused in new-debugger head.js was modified to rely on waitUntil, which comes from shared-head. For now I suggest just importing waitUntil, without trying to inject the whole shared-head for this test. Try run with fixes for both issues: https://treeherder.mozilla.org/#/jobs?repo=try&revision=103261a0ce666d21761bc7c65ad933c36bd1d662

PR opened for the isPaused method, as I couldn't find it anywhere on the repo. https://github.com/devtools-html/debugger.html/pull/5194

If try is green with this, I would suggest to make a new release including isPaused and to keep my toolbox test patch separated.

Comment on attachment 8945437 [details] [diff] [review] release-10.patch Review of attachment 8945437 [details] [diff] [review]: ----------------------------------------------------------------- Needs a valid commit, and consistency with the versioning scheme. ::: devtools/client/debugger/new/README.mozilla @@ +1,4 @@ > This is the debugger.html project output. > See https://github.com/devtools-html/debugger.html > > +Version 10 10.0 please @@ +1,5 @@ > This is the debugger.html project output. > See https://github.com/devtools-html/debugger.html > > +Version 10 > +Commit: https://github.com/devtools-html/debugger.html/commit/a6351244d92145b https://github.com/devtools-html/debugger.html/commit/a6351244d92145b -> 404

Already reviewed by jlast!

Comment on attachment 8945562 [details] [diff] [review] release-10-2.patch Review of attachment 8945562 [details] [diff] [review]: ----------------------------------------------------------------- Looks good. Last try push with both patches: https://treeherder.mozilla.org/#/jobs?repo=try&revision=8f8a9c3367b17dc1d68ed8e708ec1157f2716886

As inbound is currently closed, adding checkin-needed. Both patches attached here should land together, the order doesn't matter. Thanks!

Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/aba21dd5668b Update Debugger Frontend v10. r=jdescottes https://hg.mozilla.org/integration/mozilla-inbound/rev/c3d2757a5343 duplicate waitUntil() in browser toolbox test;r=jlast

https://hg.mozilla.org/mozilla-central/rev/aba21dd5668b https://hg.mozilla.org/mozilla-central/rev/c3d2757a5343