Closed
Bug 846949
Opened 12 years ago
Closed 12 years ago
Clean up address bar/tabs button spinoff of bug 817828
Categories
(Firefox for Android Graveyard :: Theme and Visual Design, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 22
People
(Reporter: kbrosnan, Assigned: sriram)
References
Details
Attachments
(1 file)
|
6.75 KB,
patch
|
lucasr
:
review+
|
Details | Diff | Splinter Review |
From bug 817828 comment 38
1. invalidate() is expensive and doesn't trigger onMeasure() which is needed for calculating the path when the curve changes. Hence making it requestLayout().
2. That removes the redundant requestLayout() calls in BrowserToolbar.
3. Fix for not trying to animate mMenu when it is not even there.
|
Assignee | |
Comment 1•12 years ago
|
||
Patch from the other bug (without changes to PropertyAnimator)
Attachment #724470 -
Flags: review?(lucasr.at.mozilla)
|
|
||
Updated•12 years ago
|
Attachment #724470 -
Flags: review?(lucasr.at.mozilla) → review+
|
|
Assignee | |
Comment 2•12 years ago
|
||
|
|
||
Comment 3•12 years ago
|
||
Assignee: nobody → sriram
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 22
|
||
Updated•5 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
•