As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
Last Comment Bug 836086 - Lazy load keys.js
: Lazy load keys.js
Product: Cloud Services
Classification: Client Software
Component: Firefox Sync: Backend (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla21
Assigned To: Gregory Szorc [:gps]
Depends on:
  Show dependency treegraph
Reported: 2013-01-29 16:02 PST by Gregory Szorc [:gps]
Modified: 2013-02-06 03:41 PST (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

Lazy load keys.js, v1 (1.31 KB, patch)
2013-01-29 16:02 PST, Gregory Szorc [:gps]
rnewman: review+
Details | Diff | Splinter Review

Description User image Gregory Szorc [:gps] 2013-01-29 16:02:13 PST
Created attachment 707867 [details] [diff] [review]
Lazy load keys.js, v1

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.
Comment 1 User image Richard Newman [:rnewman] 2013-01-29 16:06:48 PST
Comment on attachment 707867 [details] [diff] [review]
Lazy load keys.js, v1

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

Hah, neat.
Comment 3 User image Gregory Szorc [:gps] 2013-01-30 16:14:59 PST

Note You need to log in before you can comment on or make changes to this bug.