Avoid AddRef()ing the Range object in IMEContentObserver::InitWithEditor()

RESOLVED FIXED in Firefox 56

Status

()

Core
DOM
RESOLVED FIXED
11 months ago
11 months ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

unspecified
mozilla56
Points:
---

Firefox Tracking Flags

(firefox56 fixed)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

11 months ago
Created attachment 8888609 [details] [diff] [review]
Avoid AddRef()ing the Range object in IMEContentObserver::InitWithEditor()
Attachment #8888609 - Flags: review?(bugs)
(Assignee)

Comment 2

11 months ago
I noticed this in some Speedometer profiles, seemed like low hanging fruit to fix.

Updated

11 months ago
Assignee: nobody → ehsan

Updated

11 months ago
Attachment #8888609 - Flags: review?(bugs) → review+

Comment 3

11 months ago
Pushed by eakhgari@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f39688016fcb
Avoid AddRef()ing the Range object in IMEContentObserver::InitWithEditor(); r=smaug
https://hg.mozilla.org/mozilla-central/rev/f39688016fcb
Status: NEW → RESOLVED
Last Resolved: 11 months ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.