Comment on attachment 8960989 [details] [diff] [review] rel-26.patch Review of attachment 8960989 [details] [diff] [review]: ----------------------------------------------------------------- - commit message: KO - less than 20 commits: OK - try: KO - manual test: OK - readme: KO R+ with: - correct commit message - fixed readme - skip-if on windows for failing toolbox test, with a link to a bug or GH issue to reenable it ::: devtools/client/debugger/new/README.mozilla @@ +5,2 @@ > > +Comparison: https://github.com/devtools-html/debugger.html/compare/release-16...release-17 wrong versions

talos ongoing: https://treeherder.mozilla.org/perf.html#/compare?originalProject=try&originalRevision=7ece888ace2bd967c0377cbb7de01f8016c1be5e&newProject=try&newRevision=00bdb6010ee57f8af010e286d0c2032db6c058a2&framework=1

Wrong talos push. New link: https://treeherder.mozilla.org/perf.html#/compare?originalProject=try&originalRevision=42c1ca4d5b5edd84cebb17dfa232a826e9033ee8&newProject=try&newRevision=dd7035708eb168d37e7e115749b7ed960742b508&framework=1

No visible perf impact: https://treeherder.mozilla.org/perf.html#/comparesubtest?originalProject=try&originalRevision=42c1ca4d5b5edd84cebb17dfa232a826e9033ee8&newProject=try&newRevision=dd7035708eb168d37e7e115749b7ed960742b508&originalSignature=f79b6a4f1f8f53326d3056f6c8008c0ff4de0a94&newSignature=f79b6a4f1f8f53326d3056f6c8008c0ff4de0a94&showOnlyConfident=1&framework=1

Pushed by jlaster@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/714385f620c7 Update Debugger Frontend v26. r=jdescottes