1477261 - Font editor: add unit test for conversion of font-size values

Status: RESOLVED FIXED

(DevTools :: Inspector, enhancement, P2)

Description

[Razvan Caliman [:rcaliman]]

This replaces the integration test that was disabled in Bug 1476535 due to intermittent failures with a simplified unit test which just checks the math of the conversion operation.

Comment 1

[Razvan Caliman [:rcaliman]]

Attachment: Bug 1477261 - (Part 1) rename browser_fontinspector.html to doc_browser_fontinspector.html.

Review commit: https://reviewboard.mozilla.org/r/258364/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/258364/

Comment 2

[Razvan Caliman [:rcaliman]]

Attachment: Bug 1477261 - (Part 2) Replace integration test with unit test for font-size conversion.

Review commit: https://reviewboard.mozilla.org/r/258366/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/258366/

Comment 3

[Patrick Brosset <:pbro>]

Comment on attachment 8993681 [details]
Bug 1477261 - (Part 1) rename browser_fontinspector.html to doc_browser_fontinspector.html.

https://reviewboard.mozilla.org/r/258364/#review265418

Comment 4

[Patrick Brosset <:pbro>]

Comment on attachment 8993682 [details]
Bug 1477261 - (Part 2) Replace integration test with unit test for font-size conversion.

https://reviewboard.mozilla.org/r/258366/#review265424

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/19fe41443952
(Part 1) rename browser_fontinspector.html to doc_browser_fontinspector.html. r=pbro
https://hg.mozilla.org/integration/autoland/rev/06704594e78c
(Part 2) Replace integration test with unit test for font-size conversion. r=pbro

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/19fe41443952
https://hg.mozilla.org/mozilla-central/rev/06704594e78c