Comment on attachment 8938964 [details] [diff] [review] patch-12-27-2.patch Review of attachment 8938964 [details] [diff] [review]: ----------------------------------------------------------------- Invalid (local?) commit in README. I also see a leak on try, but not sure the runs posted in bugzilla match the current patch. ::: devtools/client/debugger/new/README.mozilla @@ +1,4 @@ > This is the debugger.html project output. > See https://github.com/devtools-html/debugger.html > > +Taken from upstream commit: d96af0192084a3bdb66b916c7336aa80fcf3c69a Can't find this commit on debugger.html

updates the sha

Links: + Commit - https://github.com/devtools-html/debugger.html/commit/0f6983e5343bb7a63ee120c99dcbdb0ccdec2e17 + Branch - https://github.com/devtools-html/debugger.html/tree/release-3 + Comparison - https://github.com/devtools-html/debugger.html/compare/release-3...release-4?expand=1

Try Run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=01781f81e24a11f997718701701427e8ba33cbaa Comparison: https://github.com/devtools-html/debugger.html/compare/release-3...release-4 Branch: + Branch - https://github.com/devtools-html/debugger.html/tree/release-4 Commit: https://github.com/devtools-html/debugger.html/commit/0bcf3e687305960077e1255510e424d0437a3b69

Comment on attachment 8939894 [details] [diff] [review] 12-27-4.patch Review of attachment 8939894 [details] [diff] [review]: ----------------------------------------------------------------- LGTM https://github.com/devtools-html/debugger.html/compare/f98e56ff661460044382df55bd33cea601e093f0...0bcf3e687305960077e1255510e424d0437a3b69

Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/c2e9837df732 Update Debugger frontend (12-27). r=jdescottes