Revert of Force chooser bubbles to close synchronously.

Revert of Force chooser bubbles to close synchronously. (patchset #3 id:40001 of https://codereview.chromium.org/2724903008/ )

Reason for revert:
Reverting this patchset because it was insufficiently tested and introduced a new bug.

Original issue's description:
> Force chooser bubbles to close synchronously.
> 
> This fixes a potential use-after-free because UsbChooserController holds
> a raw pointer to the RenderFrameHost and so must be destroyed as part of
> RenderFrameHost destruction.
> 
> BUG=697486
> 
> Review-Url: https://codereview.chromium.org/2724903008
> Cr-Commit-Position: refs/heads/master@{#454989}
> Committed: https://chromium.googlesource.com/chromium/src/+/692db28a688889a7946b04debfd890673ae2269f

TBR=msw@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=697486, 698927
Review-Url: https://codereview.chromium.org/2736793003
Cr-Commit-Position: refs/heads/master@{#455086}
Committed: https://chromium.googlesource.com/chromium/src/+/5e296353d65aef23ed1b524a4c1b01e568f6c3cd

[Reilly Grant (use Gerrit), 2017-03-07 15:50:01]

Created Revert of Force chooser bubbles to close synchronously.

[Reilly Grant (use Gerrit), 2017-03-07 15:50:16]

Description was changed from

==========
Revert of Force chooser bubbles to close synchronously. (patchset #3 id:40001 of
https://codereview.chromium.org/2724903008/ )

Reason for revert:
Reverting this patchset because it was insufficiently tested and introduced a
new bug.

Original issue's description:
> Force chooser bubbles to close synchronously.
> 
> This fixes a potential use-after-free because UsbChooserController holds
> a raw pointer to the RenderFrameHost and so must be destroyed as part of
> RenderFrameHost destruction.
> 
> BUG=697486
> 
> Review-Url: https://codereview.chromium.org/2724903008
> Cr-Commit-Position: refs/heads/master@{#454989}
> Committed:
https://chromium.googlesource.com/chromium/src/+/692db28a688889a7946b04debfd8...

TBR=msw@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=697486
==========

to

==========
Revert of Force chooser bubbles to close synchronously. (patchset #3 id:40001 of
https://codereview.chromium.org/2724903008/ )

Reason for revert:
Reverting this patchset because it was insufficiently tested and introduced a
new bug.

Original issue's description:
> Force chooser bubbles to close synchronously.
> 
> This fixes a potential use-after-free because UsbChooserController holds
> a raw pointer to the RenderFrameHost and so must be destroyed as part of
> RenderFrameHost destruction.
> 
> BUG=697486
> 
> Review-Url: https://codereview.chromium.org/2724903008
> Cr-Commit-Position: refs/heads/master@{#454989}
> Committed:
https://chromium.googlesource.com/chromium/src/+/692db28a688889a7946b04debfd8...

TBR=msw@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=697486,698927
==========

[Reilly Grant (use Gerrit), 2017-03-07 15:50:24]

The CQ bit was checked by reillyg@chromium.org

[commit-bot: I haz the power, 2017-03-07 15:51:39]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-07 16:06:27]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488901824252810, "parent_rev":
"6c65974da98f515df67bd24b354bf1a5ce468d60", "commit_rev":
"5e296353d65aef23ed1b524a4c1b01e568f6c3cd"}

[commit-bot: I haz the power, 2017-03-07 16:07:03]

Description was changed from

==========
Revert of Force chooser bubbles to close synchronously. (patchset #3 id:40001 of
https://codereview.chromium.org/2724903008/ )

Reason for revert:
Reverting this patchset because it was insufficiently tested and introduced a
new bug.

Original issue's description:
> Force chooser bubbles to close synchronously.
> 
> This fixes a potential use-after-free because UsbChooserController holds
> a raw pointer to the RenderFrameHost and so must be destroyed as part of
> RenderFrameHost destruction.
> 
> BUG=697486
> 
> Review-Url: https://codereview.chromium.org/2724903008
> Cr-Commit-Position: refs/heads/master@{#454989}
> Committed:
https://chromium.googlesource.com/chromium/src/+/692db28a688889a7946b04debfd8...

TBR=msw@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=697486,698927
==========

to

==========
Revert of Force chooser bubbles to close synchronously. (patchset #3 id:40001 of
https://codereview.chromium.org/2724903008/ )

Reason for revert:
Reverting this patchset because it was insufficiently tested and introduced a
new bug.

Original issue's description:
> Force chooser bubbles to close synchronously.
> 
> This fixes a potential use-after-free because UsbChooserController holds
> a raw pointer to the RenderFrameHost and so must be destroyed as part of
> RenderFrameHost destruction.
> 
> BUG=697486
> 
> Review-Url: https://codereview.chromium.org/2724903008
> Cr-Commit-Position: refs/heads/master@{#454989}
> Committed:
https://chromium.googlesource.com/chromium/src/+/692db28a688889a7946b04debfd8...

TBR=msw@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=697486,698927

Review-Url: https://codereview.chromium.org/2736793003
Cr-Commit-Position: refs/heads/master@{#455086}
Committed:
https://chromium.googlesource.com/chromium/src/+/5e296353d65aef23ed1b524a4c1b...
==========

[commit-bot: I haz the power, 2017-03-07 16:07:05]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/5e296353d65aef23ed1b524a4c1b...

[msw, 2017-03-07 18:31:39]

lgtm - can you elaborate on the bug this introduced?