GitLab

This patch also fixes a race revealed by the new test. When the
extension was uninstalled, ExtensionHostQueue::ProcessOneHost
would race with ExtensionHostQueue::Remove (called from
~ExtensionHost), and if the former won it would crash trying to get the
missing extension's ID. This patch ensures that ExtensionPopup
immediately destroys its owned ExtensionHost when the extension is
uninstalled, avoiding the race.

Bug: 984654
Change-Id: Ib811a6fca559129f235e00f6091086cf7f2f968f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1980613Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Commit-Queue: Bret Sepulveda <bsep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#736818}