GitLab

Chrome sometimes needs to detect when an object has been deleted in
the middle of a member function. Manually maintaining a destroyed
pointer/flag is error prone and verbose so this change switches to using
WeakPtr.

In the NativeMenuWin case the destroyed pointer wasn't even used, making
deleting it particularly easy.

Idea from dcheng@

Bug: 1152152
Change-Id: Ie913fa6255c5db4667e59b724732df2dbedf5580
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2637268Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845538}