1226570 - Remove usages of chrome://browser/skin/

Status: RESOLVED FIXED

(DevTools :: General, defect)

Description

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Our tools currently reference a few style sheets, etc. from chrome://browser/skin, but much of it is unnecessary.

Removing these makes it possible to use the front-end in non-browser environments.

Comment 1

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Attachment: MozReview Request: Bug 1226570 - Remove chrome://browser/skin/ from DevTools. r=bgrins

Bug 1226570 - Remove chrome://browser/skin/ from DevTools. r=bgrins

Comment 2

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

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

Comment 3

[Brian Grinstead [:bgrins]]

Comment on attachment 8690098 [details]
MozReview Request: Bug 1226570 - Remove chrome://browser/skin/ from DevTools. r=bgrins

https://reviewboard.mozilla.org/r/25765/#review23193

This looks good to me, I don't know why browser.css was included in any of these tools in the first place.  We should double check all platforms to make sure this doesn't cause any obvious visual issues before landing

Comment 5

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Did a brief spot check on Windows and Linux, looks okay to me, but it's possible I've missed something.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/80234c36f1b7

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/80234c36f1b7