aw: Avoid hold and wait deadlock

aw: Avoid hold and wait deadlock

when creating single instances. Instead of using more locks which
complicates things, just create the singletons during the webview init
phoase.

Only exception here is WebViewDatabas, which really is a standalone
database and has nothing to do with the webview itself, but takes a
Context in its constructor. The constructor currently is safe to be
called from non-UI thread, so just do that.

BUG=648784
Committed: https://crrev.com/7d528aede8d842cbfdf29d99ba6b6869a127fc81
Cr-Commit-Position: refs/heads/master@{#423156}

[boliu, 2016-10-04 21:25:46]

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

[boliu, 2016-10-04 21:25:46]

ptal

[boliu, 2016-10-04 21:33:43]

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

[commit-bot: I haz the power, 2016-10-04 21:34:40]

Dry run: 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, 2016-10-04 22:51:40]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-04 22:51:41]

Dry run: This issue passed the CQ dry run.

[Torne, 2016-10-05 13:56:28]

LGTM - this seems like the best option by far, and I can't see any reason to
expect that creating these objects eagerly is going to be a measurable
difference in performance (they're pretty simple).

[boliu, 2016-10-05 14:35:22]

The CQ bit was checked by boliu@chromium.org

[commit-bot: I haz the power, 2016-10-05 14:35:50]

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, 2016-10-05 14:42:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-05 14:44:25]

Description was changed from

==========
aw: Avoid hold and wait deadlock

when creating single instances. Instead of using more locks which
complicates things, just create the singletons during the webview init
phoase.

Only exception here is WebViewDatabas, which really is a standalone
database and has nothing to do with the webview itself, but takes a
Context in its constructor. The constructor currently is safe to be
called from non-UI thread, so just do that.

BUG=648784
==========

to

==========
aw: Avoid hold and wait deadlock

when creating single instances. Instead of using more locks which
complicates things, just create the singletons during the webview init
phoase.

Only exception here is WebViewDatabas, which really is a standalone
database and has nothing to do with the webview itself, but takes a
Context in its constructor. The constructor currently is safe to be
called from non-UI thread, so just do that.

BUG=648784

Committed: https://crrev.com/7d528aede8d842cbfdf29d99ba6b6869a127fc81
Cr-Commit-Position: refs/heads/master@{#423156}
==========

[commit-bot: I haz the power, 2016-10-05 14:44:26]

Patchset 1 (id:??) landed as
https://crrev.com/7d528aede8d842cbfdf29d99ba6b6869a127fc81
Cr-Commit-Position: refs/heads/master@{#423156}