Closed Bug 1162179 Opened 4 years ago Closed 4 years ago

APZCTreeManager still created in non-e10s windows

Categories

(Core :: Panning and Zooming, defect)

All
Gonk (Firefox OS)
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: kats, Assigned: kats)

References

Details

Attachments

(1 file)

In bug 1154739 I changed things so that APZ would only be enabled in e10s windows. However it looks like we're still creating APZCTreeManager instances for non-e10s windows which is unnecessary and probably inefficient. We can move the condition into CompositorParent so as to avoid this.
Attachment #8602236 - Attachment description: WIP → Patch
Attachment #8602236 - Flags: review?(dvander)
Comment on attachment 8602236 [details] [diff] [review]
Patch

Review of attachment 8602236 [details] [diff] [review]:
-----------------------------------------------------------------

::: widget/nsIWidget.h
@@ +1750,5 @@
>       */
>      NS_IMETHOD DispatchEvent(mozilla::WidgetGUIEvent* event,
>                               nsEventStatus & aStatus) = 0;
>  
> +    virtual bool IsMultiProcessWindow() = 0;

micro-nit: the nsBaseWidget version should get "override" now
Attachment #8602236 - Flags: review?(dvander) → review+
https://hg.mozilla.org/mozilla-central/rev/8985391fa24c
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.