Closed Bug 685878 Opened 13 years ago Closed 13 years ago

Cache the sidebar width in ViewableAreaObserver

Categories

(Firefox for Android Graveyard :: General, defect)

Firefox 9
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 686417

People

(Reporter: mbrubeck, Assigned: mbrubeck)

References

Details

(Keywords: perf)

Attachments

(1 file)

Attached patch patchSplinter Review
This will save some work each time we call ViewableAreaObserver.width.
Attachment #559471 - Flags: review?(mark.finkle)
Attachment #559471 - Flags: review?(mark.finkle) → review+
Wait, won't this need to change when we support portrait mode and it's hidden sidebar?
I guess we could set the tabletSidebarWidth to null when we sense a rotation
Comment on attachment 559471 [details] [diff] [review]
patch

Good point.
Attachment #559471 - Flags: review+ → review-
This is fixed by one of the patches in bug 686417.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: