APZCTreeManager still created in non-e10s windows

RESOLVED FIXED in Firefox 40

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: kats, Assigned: kats)

Tracking

unspecified
mozilla40
All
Gonk (Firefox OS)
Points:
---

Firefox Tracking Flags

(firefox40 fixed)

Details

Attachments

(1 attachment)

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.
See Also: → bug 1162064
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
Last Resolved: 4 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.