Closed
Bug 1504924
Opened 6 years ago
Closed 6 years ago
nsDisplayText::CanApplyOpacity() should use mIsFrameSelected to avoid nsIFrame::IsSelected() calls
Categories
(Core :: Web Painting, enhancement, P3)
Core
Web Painting
Tracking
()
RESOLVED
FIXED
mozilla65
| Tracking | Status | |
|---|---|---|
| firefox65 | --- | fixed |
People
(Reporter: MatsPalmgren_bugz, Assigned: mikokm)
Details
(Keywords: perf)
Attachments
(1 file)
https://searchfox.org/mozilla-central/rev/7c848ac7630df5baf1314b0c03e015683599efb9/layout/generic/nsTextFrame.cpp#4990 Instead of calling f->IsSelected() unconditionally it should check mIsFrameSelected, like so: https://searchfox.org/mozilla-central/rev/7c848ac7630df5baf1314b0c03e015683599efb9/layout/generic/nsTextFrame.cpp#6987-6989
|
Assignee | |
Updated•6 years ago
|
Assignee: nobody → mikokm
Status: NEW → ASSIGNED
Priority: -- → P3
|
|
Assignee | |
Comment 1•6 years ago
|
||
Pushed by mikokm@gmail.com: https://hg.mozilla.org/integration/autoland/rev/61c3a6ce72a0 Avoid nsIFrame::IsSelected() call in nsDisplayText::CanApplyOpacity() r=mats
|
||
Comment 3•6 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/61c3a6ce72a0
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in
before you can comment on or make changes to this bug.
Description
•