[ProximityAuth] Remote static initializers from uWeave classes.

[ProximityAuth] Remote static initializers from uWeave classes.

Keeping a raw pointer around isn't that ideal, as it leaks memory, but the
alternative of refactoring all classes to take a factory in the constructor is a
lot more work.

Review-Url: https://codereview.chromium.org/2844773002
Cr-Commit-Position: refs/heads/master@{#467480}
Committed: https://chromium.googlesource.com/chromium/src/+/40efa1fd583885791e551dfcf1374f6925e1df6c

[Tim Song, 2017-04-26 19:17:24]

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

[commit-bot: I haz the power, 2017-04-26 19:18:04]

Dry run: CQ is trying da patch.

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

[Tim Song, 2017-04-26 19:18:31]

tengs@chromium.org changed reviewers:
+ khorimoto@chromium.org

[Kyle Horimoto, 2017-04-26 21:12:50]

LGTM

I don't think that keeping a raw pointer is an issue at all. We only will ever
create one factory instance (per process), so the amount of memory allocated is
O(1). Once the process is killed, the heap is cleared, so that object will be
destroyed.

[commit-bot: I haz the power, 2017-04-26 21:19:06]

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

[commit-bot: I haz the power, 2017-04-26 21:19:07]

Dry run: This issue passed the CQ dry run.

[Tim Song, 2017-04-26 22:08:49]

The CQ bit was checked by tengs@chromium.org

[commit-bot: I haz the power, 2017-04-26 22:09:40]

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-26 22:15:04]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493244529495700, "parent_rev":
"789b5e528fb3bd9ca44341de46547c45355652ab", "commit_rev":
"40efa1fd583885791e551dfcf1374f6925e1df6c"}

[commit-bot: I haz the power, 2017-04-26 22:15:15]

Description was changed from

==========
[ProximityAuth] Remote static initializers from uWeave classes.

Keeping a raw pointer around isn't that ideal, as it leaks memory, but the
alternative of refactoring all classes to take a factory in the constructor is a
lot more work.
==========

to

==========
[ProximityAuth] Remote static initializers from uWeave classes.

Keeping a raw pointer around isn't that ideal, as it leaks memory, but the
alternative of refactoring all classes to take a factory in the constructor is a
lot more work.

Review-Url: https://codereview.chromium.org/2844773002
Cr-Commit-Position: refs/heads/master@{#467480}
Committed:
https://chromium.googlesource.com/chromium/src/+/40efa1fd583885791e551dfcf137...
==========

[commit-bot: I haz the power, 2017-04-26 22:15:16]

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