tree 53feaff09273d62eb77c3c1de35c5d0ad58dfdd7
parent bdae58348b2294710d63f1466915744410c1bca6
author Peter Boström <pbos@chromium.org> 1710266793 +0000
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1710266793 +0000

Avoid calls to partially-destructed SurfaceManager

This does not fix any underlying problem (why are the Surface objects
still present here?), but mitigates a use-after-destroy bug where the
ObserverList is accessed after it's been destructed (but not yet freed
afaict).

This was found with crrev.com/c/5319909 and the workaround should let us
add better sequence checking in ObserverList.

Bug: 823043
Change-Id: I10c8c03bbb16d06cae185b85e102e469e08d08cf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5363498
Auto-Submit: Peter Boström <pbos@chromium.org>
Reviewed-by: Kevin Haslett <khaslett@chromium.org>
Commit-Queue: Kevin Haslett <khaslett@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1271680}