2001613 - don't put ASRs into mActiveScrolledRoots multiple times

Status: RESOLVED FIXED

(Core :: Panning and Zooming, enhancement)

Description

[Timothy Nikkel (:tnikkel)]

This isn't strictly a problem, but it's better to avoid it and be more clear about these functions.

Comment 1

[Timothy Nikkel (:tnikkel)]

Attachment: Bug 2001613. Don't put ASRs into mActiveScrolledRoots multiple times. r?botond,hiro

This doesn't cause problems, but we don't need to do it, and we can clean up these functions a bit to be more clear.

Besides the only functional change of this patch (the first line of the commit message) this patch also renames the functions to GetOrCreate so it is more clear what they do.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/58ae5b2ad37a
https://hg.mozilla.org/integration/autoland/rev/b151f4bb267b
Don't put ASRs into mActiveScrolledRoots multiple times. r=layout-reviewers,emilio

Comment 3

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/b151f4bb267b