Make APZCTreeManager::GetContentController and FlushApzRepaints static

RESOLVED FIXED in Firefox 66

Status

()

enhancement
RESOLVED FIXED
6 months ago
6 months ago

People

(Reporter: botond, Assigned: botond)

Tracking

Trunk
mozilla66
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox66 fixed)

Details

Attachments

(2 attachments)

Assignee

Description

6 months ago
Because they can be, and it makes it easier to reason about code when functions that can be static, are.
Assignee

Comment 1

6 months ago
The function does not depend on any state in the tree manager, since
the controller is looked up via sIndirectLayerTrees.
Assignee

Comment 2

6 months ago
Since we no longer throttle paints, this function just sends a notification,
which does not rely on any tree manager state.

Depends on D15592

Comment 3

6 months ago
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3fa4e6287438
Make APZCTreeManager::GetContentController() static. r=kats
https://hg.mozilla.org/integration/autoland/rev/d568afd4311e
Make APZCTreeManager::FlushApzRepaints() static. r=kats

Comment 4

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/3fa4e6287438
https://hg.mozilla.org/mozilla-central/rev/d568afd4311e
Status: NEW → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in before you can comment on or make changes to this bug.