Jason Laster [:jlast] Assignee
	Description • 6 years ago

http://github.com/devtools-html/debugger.html/compare/release-50...release-51

	Jason Laster [:jlast] Assignee
	Comment 2 • 6 years ago

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

	Julian Descottes [:jdescottes]
	Comment 4 • 6 years ago

Waiting for a greener try to review. 

An overall comment is that if we could have file moves rather than delete/add, this would make the diff much easier to read. Not sure what you need to do from git-cinnabar to achieve that though.

	Jason Laster [:jlast] Assignee
	Comment 6 • 6 years ago

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

	Julian Descottes [:jdescottes]
	Comment 7 • 6 years ago

Comment on attachment 8974694 [details] [diff] [review]
rel-51-3.patch

Review of attachment 8974694 [details] [diff] [review]:
-----------------------------------------------------------------

Works for me.

As discussed: need to update the path in browser_parsable_css.
I would honestly feel better if we didn't land the src/test folder to avoid polluting the repo.
At the very least remove the 'test' entry from the DIRS in src/moz.build . Up to you.

Follow ups for next release:
- stop copying tests to src/test
- check what should be done about vendors.css

	Jason Laster [:jlast] Assignee
	Comment 9 • 6 years ago

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

	Pulsebot
	Comment 10 • 6 years ago

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

	Dorel Luca [:dluca]
	Comment 11 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/b54faaaa591c