GitLab

It seems to be possible for shared references to CrostiniRestarter to
outlive CrostiniManager. Since all of CrostiniRestarter assumes that
CrostiniManager still exists, this can cause crashes. Switch
CrostiniRestarter to being exclusivly owned by CrostiniManager and
handing out weak pointers to callbacks.

Bug: 1040797
Change-Id: Ib65c79c7c81111aad88435b3c95da0c64226e791
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1996112Reviewed-by: Nic Hollingum <hollingum@google.com>
Commit-Queue: Fergus Dall <sidereal@google.com>
Cr-Commit-Position: refs/heads/master@{#733496}