Closed
Bug 730949
Opened 12 years ago
Closed 12 years ago
TextLayer.renderText is not needed on startup path
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sriram, Assigned: kats)
Details
Attachments
(1 file)
|
6.65 KB,
patch
|
sriram
:
review+
|
Details | Diff | Splinter Review |
The TextRenderer is taking around 30ms on startup, which is not needed at all.
|
Assignee | |
Comment 1•12 years ago
|
||
The frame rate layer is created on the startup path even though it is not used right away (and in fact not displayed unless it is preffed on). This should be made more lazy in LayerRenderer.
Assignee: nobody → bugmail.mozilla
OS: Mac OS X → Android
Hardware: x86 → All
Summary: TextRender is not needed on startup patch → TextLayer.renderText is not needed on startup path
|
|
Assignee | |
Comment 2•12 years ago
|
||
I'll be landing this on maple to avoid merging problems later.
Attachment #601421 -
Flags: review?(sriram)
|
Reporter | |
Comment 3•12 years ago
|
||
Comment on attachment 601421 [details] [diff] [review] Take frame rate layer init off the startup path Review of attachment 601421 [details] [diff] [review]: ----------------------------------------------------------------- Looks good to me.
Attachment #601421 -
Flags: review?(sriram) → review+
|
|
Assignee | |
Comment 4•12 years ago
|
||
https://hg.mozilla.org/projects/maple/rev/e116c86ea476
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
|
||
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•