Closed
Bug 836086
Opened 11 years ago
Closed 11 years ago
Lazy load keys.js
Categories
(Firefox :: Sync, defect)
Firefox
Sync
Tracking
()
RESOLVED
FIXED
mozilla21
People
(Reporter: gps, Assigned: gps)
Details
(Whiteboard: [MemShrink])
Attachments
(1 file)
1.31 KB,
patch
|
rnewman
:
review+
|
Details | Diff | Splinter Review |
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 1•11 years ago
|
||
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+
Assignee | ||
Comment 2•11 years ago
|
||
https://hg.mozilla.org/services/services-central/rev/193ab8cee72a
Assignee: nobody → gps
Status: NEW → ASSIGNED
Whiteboard: [MemShrink] → [MemShrink][fixed in services]
Assignee | ||
Comment 3•11 years ago
|
||
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
Updated•6 years ago
|
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.
Description
•