1286232 - Add a scripted way of generating values in css-properties-db.js

Status: RESOLVED FIXED

(DevTools :: Framework, enhancement, P1)

Description

[Greg Tatum [:gregtatum]]

In the work to separate the client and server in devtools, we are creating a static list of CSS properties on the client. The list has been hand-generated every time. At this time we need a script to re-generate all of the values, and instructions on how to update the file. At this time this does not need to be included in the build system as that would be a lot of additional complexity. The script should be able to simply run from the Browser Console.

This bug came out of discussions from Bug 1265785.

Comment 1

[Greg Tatum [:gregtatum]]

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

Comment 2

[Greg Tatum [:gregtatum]]

Attachment: Add scripts and tests to automate the css-properties-db.

I ended up combining this bug and Bug 1286238 as the testing bug was already failing due to the fact that the CSS properties database has already changed! I'd have to land the scripting bug, then land the testing patch. In the interest of time they are combined here. I'm keeping this pretty manual, but I'm trying to be as explicit as I can in how to update the database so that anyone can come in and update it.

Comment 3

[Patrick Brosset <:pbro>]

Comment on attachment 8771054 [details] [diff] [review]
Add scripts and tests to automate the css-properties-db.

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

This looks great! Thanks a lot. Feels like we are in a much better position now to prevent future de-sync with platform and it'll be really easy to re-generate the lists.

Comment 5

[Greg Tatum [:gregtatum]]

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

Comment 6

[Greg Tatum [:gregtatum]]

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

Comment 7

[Greg Tatum [:gregtatum]]

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

Comment 8

[Greg Tatum [:gregtatum]]

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

Comment 9

[Greg Tatum [:gregtatum]]

Attachment: Add scripts and tests to automate the css-properties-db.

This patch removes OS-specific tags, and makes the failing test more verbose
and specific to what failed.

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/fx-team/rev/7e5277f1cf65
Add scripts and tests to automate the css-properties-db. r=pbro

Comment 11

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/7e5277f1cf65