1837960 - remove array of all viewmanagers nsViewManager::gViewManagers

Status: RESOLVED FIXED

(Core :: Layout, enhancement)

Comment 1

[Timothy Nikkel (:tnikkel)]

Attachment: Bug 1837960. Remove array of all nsViewManagers. r?#layout-reviewers

The only thing it's used for is calling will paint on a document tree. To iterate every viewmanager looking for those in one document tree is a bit silly, we can just walk the view tree. Since there are only 3 types of views (root views, popup views for menupopupframes, and subdocument frame views (and their inner view)) we don't have to walk many pointers: menupopupframes are controlled by chrome code and are limited, and the other two types of views correspond to the number of view managers in the tree (one view manager per document).

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/30ecf9470a05
Remove array of all nsViewManagers. r=emilio

Comment 3

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/30ecf9470a05