1181856 - Migrate browser/devtools/styleeditor tests to shared-head.js

Status: RESOLVED WORKSFORME

(DevTools :: Style Editor, defect)

Description

[Brian Grinstead [:bgrins]]

We should import shared-head.js in styleeditor/test/head.js and remove any newly unnecessary code after that.

Comment 1

[Patrick Brosset <:pbro>]

Attachment: Bug_1181856_-_Use_shared-head.js_in_styleeditor_te.diff

I unfortunately don't have time to push this further, but I did start to work on this.
It's a little more complicated than just importing shared-head.js though. Some style-editor tests need to access the inspector's rule-view, and therefore, it makes sense to import the head.js from the inspector/rules/test directory instead (which itself imports the head.js from inspector/test, which itself imports shared-head.js), in order to remove code duplication.
This is what this patch does, and it's really close too. I think there was one remaining test failing.

Comment 2

[Patrick Brosset <:pbro>]

This appears to be done now. The style-editor's tests include the inspector's shared head file, which itself includes the shared-head.js file