aw: Remove global AllowWait on UI thread

aw: Remove global AllowWait on UI thread

Two specific cases left for webview.

SynchronousCompositor needs to send synchronous IPC messages, in order
to maintain synchronous behavior for backward compatibility without
merging the UI thread with the renderer compositor thread. There is a
design doc for this in crbug.com/526842#c10

CookieManager, has has to run before a UI thread in webview has been
determined. So CookieManager starts on its own thread, and any thread,
including the UI thread, can wait for a result synchronously based on
the cookie manager thread.

BUG=240453
Review-Url: https://codereview.chromium.org/2812703003
Cr-Commit-Position: refs/heads/master@{#463849}
Committed: https://chromium.googlesource.com/chromium/src/+/2ce547e8ae0c6a41876f95aec2465a0d1031ae52

[boliu, 2017-04-10 22:11:40]

The CQ bit was checked by boliu@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-10 22:12:12]

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

[boliu, 2017-04-10 23:34:53]

The CQ bit was unchecked by boliu@chromium.org

[boliu, 2017-04-11 01:32:55]

The CQ bit was checked by boliu@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-11 01:34:29]

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

[boliu, 2017-04-11 01:36:08]

The CQ bit was unchecked by boliu@chromium.org

[boliu, 2017-04-11 15:10:27]

boliu@chromium.org changed reviewers:
+ jam@chromium.org, torne@chromium.org

[boliu, 2017-04-11 15:10:27]

ptal

SynchronousCompositor waits are spread across three classes which kinda sucks,
but somewhat unavoidable.

[Torne, 2017-04-11 20:36:03]

LGTM, both in general and the android_webview and CookieManager bits
specifically :)

[jam, 2017-04-11 23:25:24]

lgtm

[boliu, 2017-04-11 23:26:55]

The CQ bit was checked by boliu@chromium.org

[commit-bot: I haz the power, 2017-04-11 23:27:45]

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-04-12 00:08:13]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1491953215817100,
"parent_rev": "ef4c9d488edca70239e636d80b19daa1b4e03d86", "commit_rev":
"2ce547e8ae0c6a41876f95aec2465a0d1031ae52"}

[commit-bot: I haz the power, 2017-04-12 00:09:09]

Description was changed from

==========
aw: Remove global AllowWait on UI thread

Two specific cases left for webview.

SynchronousCompositor needs to send synchronous IPC messages, in order
to maintain synchronous behavior for backward compatibility without
merging the UI thread with the renderer compositor thread. There is a
design doc for this in crbug.com/526842#c10

CookieManager, has has to run before a UI thread in webview has been
determined. So CookieManager starts on its own thread, and any thread,
including the UI thread, can wait for a result synchronously based on
the cookie manager thread.

BUG=240453
==========

to

==========
aw: Remove global AllowWait on UI thread

Two specific cases left for webview.

SynchronousCompositor needs to send synchronous IPC messages, in order
to maintain synchronous behavior for backward compatibility without
merging the UI thread with the renderer compositor thread. There is a
design doc for this in crbug.com/526842#c10

CookieManager, has has to run before a UI thread in webview has been
determined. So CookieManager starts on its own thread, and any thread,
including the UI thread, can wait for a result synchronously based on
the cookie manager thread.

BUG=240453

Review-Url: https://codereview.chromium.org/2812703003
Cr-Commit-Position: refs/heads/master@{#463849}
Committed:
https://chromium.googlesource.com/chromium/src/+/2ce547e8ae0c6a41876f95aec246...
==========

[commit-bot: I haz the power, 2017-04-12 00:09:10]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/2ce547e8ae0c6a41876f95aec246...