Closed Bug 624718 Opened 10 years ago Closed 10 years ago

Use contentprefs caching for zoom

Categories

(SeaMonkey :: UI Design, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1b2

People

(Reporter: kairo, Assigned: kairo)

References

Details

Attachments

(1 file)

Bug 559992 added a caching system for content prefs and used it for Firefox zoom, we should do the latter in SeaMonkey as well.
http://hg.mozilla.org/mozilla-central/rev/9365fe1a3165 is the relevant checkin, we only need to port the browser-fullZoom.js part into our view "zoom overlay".
From all I see, the actual change in this ported code is just that we don't need to re-check when the pref is cached, as we know it will be fast in that case.
Assignee: nobody → kairo
Status: NEW → ASSIGNED
Attachment #502802 - Flags: review?(neil)
Attachment #502802 - Flags: review?(neil) → review+
Pushed as http://hg.mozilla.org/comm-central/rev/dec8d99ccc4b
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b2
Can we use .bind(this) here?
You need to log in before you can comment on or make changes to this bug.