Use atomic ops for managing g_allocator.

Use atomic ops for managing g_allocator.

BUG=684498
Review-Url: https://codereview.chromium.org/2655673002
Cr-Commit-Position: refs/heads/master@{#445805}
Committed: https://chromium.googlesource.com/chromium/src/+/a7d5177f1f8d47bcae75304d19b2728533e267f6

[bcwhite, 2017-01-24 16:24:38]

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

[commit-bot: I haz the power, 2017-01-24 16:24:53]

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

[bcwhite, 2017-01-24 16:27:06]

bcwhite@chromium.org changed reviewers:
+ asvitkine@chromium.org

[commit-bot: I haz the power, 2017-01-24 16:29:02]

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

[commit-bot: I haz the power, 2017-01-24 16:29:03]

Dry run: Try jobs failed on following builders:
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[bcwhite, 2017-01-24 17:24:52]

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

[commit-bot: I haz the power, 2017-01-24 17:25:09]

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

[Alexei Svitkine (slow), 2017-01-24 17:45:20]

lgtm

https://codereview.chromium.org/2655673002/diff/20001/base/metrics/persistent...
File base/metrics/persistent_histogram_allocator.cc (right):

https://codereview.chromium.org/2655673002/diff/20001/base/metrics/persistent...
base/metrics/persistent_histogram_allocator.cc:508: if
(subtle::NoBarrier_Load(&g_allocator)) {
Nit: Just use Get() here.

[bcwhite, 2017-01-24 18:52:21]

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

[commit-bot: I haz the power, 2017-01-24 18:53:15]

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

[bcwhite, 2017-01-24 18:53:55]

https://codereview.chromium.org/2655673002/diff/20001/base/metrics/persistent...
File base/metrics/persistent_histogram_allocator.cc (right):

https://codereview.chromium.org/2655673002/diff/20001/base/metrics/persistent...
base/metrics/persistent_histogram_allocator.cc:508: if
(subtle::NoBarrier_Load(&g_allocator)) {
On 2017/01/24 17:45:19, Alexei Svitkine (slow) wrote:
> Nit: Just use Get() here.

Done.

[commit-bot: I haz the power, 2017-01-24 20:39:10]

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

[commit-bot: I haz the power, 2017-01-24 20:39:12]

Dry run: This issue passed the CQ dry run.

[bcwhite, 2017-01-24 20:46:19]

The CQ bit was checked by bcwhite@chromium.org

[bcwhite, 2017-01-24 20:46:19]

The patchset sent to the CQ was uploaded after l-g-t-m from
asvitkine@chromium.org
Link to the patchset: https://codereview.chromium.org/2655673002/#ps40001
(title: "use Get() instead of g_allocator")

[commit-bot: I haz the power, 2017-01-24 20:47:02]

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-01-24 20:59:32]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1485290779288600,
"parent_rev": "0a362f539460ef5cfd4ca3143fc790b683dbec15", "commit_rev":
"a7d5177f1f8d47bcae75304d19b2728533e267f6"}

[commit-bot: I haz the power, 2017-01-24 21:00:03]

Description was changed from

==========
Use atomic ops for managing g_allocator.

BUG=684498
==========

to

==========
Use atomic ops for managing g_allocator.

BUG=684498

Review-Url: https://codereview.chromium.org/2655673002
Cr-Commit-Position: refs/heads/master@{#445805}
Committed:
https://chromium.googlesource.com/chromium/src/+/a7d5177f1f8d47bcae75304d19b2...
==========

[commit-bot: I haz the power, 2017-01-24 21:00:04]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/a7d5177f1f8d47bcae75304d19b2...