Closed Bug 836086 Opened 11 years ago Closed 11 years ago

Lazy load keys.js

Categories

(Firefox :: Sync, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla21

People

(Reporter: gps, Assigned: gps)

Details

(Whiteboard: [MemShrink])

Attachments

(1 file)

Sync is currently loading keys.js on startup even though it isn't needed unless Sync is active. We should lazy load it.

With compartment overhead, the attached patch saves ~98k.
Attachment #707867 - Flags: review?(rnewman)
Comment on attachment 707867 [details] [diff] [review]
Lazy load keys.js, v1

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

Hah, neat.
Attachment #707867 - Flags: review?(rnewman) → review+
https://hg.mozilla.org/services/services-central/rev/193ab8cee72a
Assignee: nobody → gps
Status: NEW → ASSIGNED
Whiteboard: [MemShrink] → [MemShrink][fixed in services]
https://hg.mozilla.org/mozilla-central/rev/193ab8cee72a
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [MemShrink][fixed in services] → [MemShrink]
Target Milestone: --- → mozilla21
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: