Closed Bug 1022809 Opened 5 years ago Closed 5 years ago

[Camera] Remove lazy_loader.js script tag


(Firefox OS Graveyard :: Gaia::Camera, defect)

Gonk (Firefox OS)
Not set


(b2g-v2.0 fixed, b2g-v2.1 fixed)

2.0 S4 (20june)
Tracking Status
b2g-v2.0 --- fixed
b2g-v2.1 --- fixed


(Reporter: justindarc, Assigned: justindarc)


(Keywords: perf, Whiteboard: [c=startup p= s= u=])


(2 files)

lazy_loader.js is no longer used. There is a <script defer> tag for it that is likely introducing a bit of startup time.
Attached file pull-request (master)
Attachment #8437080 - Flags: review?(dmarcos)
Attachment #8437080 - Flags: review?(dmarcos) → review+
Keywords: perf
Whiteboard: [c=startup p= s= u=]
If bug 1014955 lands on 1.4, we should also uplift this since it improves start up time. Noming for 1.4.
blocking-b2g: --- → 1.4?
Depends on: 1014955
mchang: I already included that fix in my uplift patch for 1014955 for v1.4 (it was just one line in index.html that was supposed to have been removed a long time ago -- likely a merge conflict error)
blocking-b2g: 1.4? → ---
No longer depends on: 1014955
Landed on master:

Needs uplift to v2.0 (setting NI? for djf)
Closed: 5 years ago
Flags: needinfo?(dflanagan)
Resolution: --- → FIXED
Comment on attachment 8437080 [details] [review]
pull-request (master)

Approving for perf (start-up time) improvement
Attachment #8437080 - Flags: approval-gaia-v2.0+
Attached file pull-request (v2.0)
Pull request for v2.0 -- Carrying R+ forward
Attachment #8437824 - Flags: review+
Landed on v2.0:

Clearing NI? for djf, approved v2.0+ from hema
Flags: needinfo?(dflanagan)
Assignee: nobody → jdarcangelo
Target Milestone: --- → 2.0 S4 (20june)
You need to log in before you can comment on or make changes to this bug.