Closed Bug 1361881 Opened 7 years ago Closed 7 years ago

Remove unused function PAPZCTreeManager::AdjustScrollForSurfaceShift

Categories

(Core :: Graphics: Layers, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: rbarker, Assigned: rbarker)

References

Details

Attachments

(1 file)

Now that the dynamic toolbar is animated in the compositor thread and calls APZCTreeManager::AdjustScrollForSurfaceShift directly, there are no users of the function PAPZCTreeManager::AdjustScrollForSurfaceShift so remove it.
Assignee: nobody → rbarker
Depends on: dynamic-toolbar-3
Comment on attachment 8864337 [details]
Bug 1361881 - Remove unused function AdjustScrollForSurfaceShift from APZCTreeManager IPC stack

https://reviewboard.mozilla.org/r/135974/#review139166

::: commit-message-f91fe:3
(Diff revision 1)
> +APZCTreeManager::AdjustScrollForSurfaceShift is only called on the
> +compositor thread now so there is no need to plumb this call from the
> +main thread into the compositor thread.

"APZCTreeManager::AdjustScrollForSurfaceShift is only called from the compositor now, so there is no need to expose this API for callers in widget code."
Attachment #8864337 - Flags: review?(bugmail) → review+
Pushed by rbarker@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/177335aa1d58
Remove unused function AdjustScrollForSurfaceShift from APZCTreeManager IPC stack r=kats
https://hg.mozilla.org/mozilla-central/rev/177335aa1d58
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: