ui/button/view code is too slow

RESOLVED FIXED in Firefox 55

Status

Add-on SDK
General
RESOLVED FIXED
a month ago
a month ago

People

(Reporter: kmag, Assigned: kmag)

Tracking

(Blocks: 1 bug)

unspecified
mozilla55

Firefox Tracking Flags

(firefox55 fixed)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

a month ago
All of the merge and diff and filter and map and emit gunk in between events and updates is the biggest performance suck, but the actual UI update code has way more overhead than it needs, too.
Comment hidden (mozreview-request)

Comment 2

a month ago
mozreview-review
Comment on attachment 8851284 [details]
Bug 1350617: Remove unnecessary overhead from ui/button/view code.

https://reviewboard.mozilla.org/r/123640/#review126432
Attachment #8851284 - Flags: review?(dtownsend) → review+

Comment 3

a month ago
Pushed by maglione.k@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/974ec8fa169b
Remove unnecessary overhead from ui/button/view code. r=mossop

Comment 4

a month ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/974ec8fa169b
Status: NEW → RESOLVED
Last Resolved: a month 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.