Closed Bug 1269653 Opened 4 years ago Closed 4 years ago

Remove LayerManagerComposite access to nsIWidget

Categories

(Core :: Graphics: Layers, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox49 --- fixed

People

(Reporter: dvander, Assigned: dvander)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

No description provided.
Use CompositorBridgeParent::mApzcTreeManager to check if APZ is enabled instead of nsIWidget.
Attachment #8748080 - Flags: review?(bugmail.mozilla)
Attachment #8748080 - Flags: review?(bugmail.mozilla) → review+
We pass nsIWidget to SetDispAcquireFence, but it's already set in Compositor, and only Android needs it. When/if Android does OOP compositing we'll have to fix this call, but this unblocks Desktop for now.
Attachment #8748312 - Flags: review?(bugmail.mozilla)
Comment on attachment 8748312 [details] [diff] [review]
part 2, SetDispAcquireFence

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

s/Android/Gonk/ - but yeah this looks fine.
Attachment #8748312 - Flags: review?(bugmail.mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/3a4e68d0112b
https://hg.mozilla.org/mozilla-central/rev/2872b9050cab
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Depends on: 1271749
You need to log in before you can comment on or make changes to this bug.