Make layout.css.devPixelsPerPx a string and parse it as a float

RESOLVED FIXED in mozilla1.9.3a1

Status

()

RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: sylvain.pasche, Assigned: sylvain.pasche)

Tracking

(Depends on: 1 bug)

Trunk
mozilla1.9.3a1
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(status1.9.2 beta1-fixed)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

9 years ago
Created attachment 397440 [details] [diff] [review]
patch, v1

Transforms the layout.css.devPixelsPerPx pref to a string and parse it as a float, as discussed on the newsgroup.

I noticed that if that pref was set to a non default integer value in a profile, it is reset when running a build with that patch. I don't think that many Firefox users have set that pref and that it would be worth adding some pref migration code. However, applications such as OLPC will need to update it I guess.
Attachment #397440 - Flags: review?(roc)
(Assignee)

Updated

9 years ago
Depends on: 394103
(Assignee)

Comment 2

9 years ago
sorry for the spam, I messed up the dependency.
Depends on: 493202
No longer depends on: 394103
(Assignee)

Updated

9 years ago
Keywords: checkin-needed
roc pushed this:
http://hg.mozilla.org/mozilla-central/rev/9ed2161f1f4a
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a1
(Assignee)

Updated

9 years ago
Depends on: 515455

Comment 5

9 years ago
Comment on attachment 397440 [details] [diff] [review]
patch, v1

approved for 1.9.2 landing
Attachment #397440 - Flags: approval1.9.2+

Comment 6

9 years ago
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/fd1e4a430445
status1.9.2: --- → beta1-fixed
You need to log in before you can comment on or make changes to this bug.