1432842 - Update Debugger Frontend v10

Status: RESOLVED FIXED

(DevTools :: Debugger, enhancement)

Comment 1

[Jason Laster [:jlast]]

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

Comment 2

[Jason Laster [:jlast]]

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

Comment 3

[Jason Laster [:jlast]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=80ca41934b4b653b6dcb26972a9de197dfe25079

Comment 4

[Jason Laster [:jlast]]

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

Comment 5

[Julian Descottes [:jdescottes]]

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

Comment 6

[Julian Descottes [:jdescottes]]

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

Comment 7

[Julian Descottes [:jdescottes]]

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

Comment 8

[Julian Descottes [:jdescottes]]

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 9

[Jason Laster [:jlast]]

Attachment: release-10.patch

Comment 10

[Jason Laster [:jlast]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=f3a0cebd90ad5bded81818785cff0a49b878c07f

Comment 11

[Julian Descottes [:jdescottes]]

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

Comment 12

[Jason Laster [:jlast]]

Attachment: release-10-1.patch

Comment 13

[Jason Laster [:jlast]]

Attachment: release-10-2.patch

Comment 14

[Julian Descottes [:jdescottes]]

Attachment: patch-toolbox-test.patch

Already reviewed by jlast!

Comment 15

[Julian Descottes [:jdescottes]]

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

Comment 16

[Julian Descottes [:jdescottes]]

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

Comment 17

[Pulsebot]

Pushed by [e-mail]:
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

Comment 18

[Andrei Ciure[:aciure]]

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