Closed Bug 1371620 Opened 4 years ago Closed 4 years ago

Make arrowscrollbox._canScrollToElement not flush styles

Categories

(Toolkit :: XUL Widgets, enhancement)

Product:
Toolkit
Component:
XUL Widgets
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla55
Tracking Flags:
Tracking Status
firefox55 --- fixed

People

(Reporter: dao, Assigned: dao)

References

Details

(Keywords: perf)

Attachments

(1 file)

patch
2.09 KB, patch
Details | Diff | Splinter Review
Attached patch patch — — Splinter Review 
_canScrollToElement currently uses getComputedStyle to see if the element is hidden. We can use nsIDOMWindowUtils::getBoundsWithoutFlushing instead. mconley reviewed this in bug 1368208.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=93cd0404f8a1a7ac649077e79fbe347e592adb43
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/944c5cc643a6
Make arrowscrollbox._canScrollToElement not flush styles. r=mconley
https://hg.mozilla.org/mozilla-central/rev/944c5cc643a6
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.