Closed Bug 1477261 Opened 6 years ago Closed 6 years ago

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

Categories

(DevTools :: Inspector, enhancement, P2)

enhancement

Tracking

(firefox63 fixed)

RESOLVED FIXED
Firefox 63
Tracking Status
firefox63 --- fixed

People

(Reporter: rcaliman, Assigned: rcaliman)

References

Details

Attachments

(2 files)

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 on attachment 8993681 [details]
Bug 1477261 - (Part 1) rename browser_fontinspector.html to doc_browser_fontinspector.html.

https://reviewboard.mozilla.org/r/258364/#review265418
Attachment #8993681 - Flags: review?(pbrosset) → review+
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
Attachment #8993682 - Flags: review?(pbrosset) → review+
Pushed by rcaliman@mozilla.com:
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
https://hg.mozilla.org/mozilla-central/rev/19fe41443952
https://hg.mozilla.org/mozilla-central/rev/06704594e78c
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Component: Inspector: Fonts → Inspector
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: