Use acquire/relase for delayed persistent allocations.

Use acquire/relase for delayed persistent allocations.

Previously, iteration was the safe way to pass allocations between
threads & processes.  DelayedPersistentAllocations add a second way by
storing the reference in a common location so acquire/release must be
used when accessing that reference to ensure data integrity of the
allocation itself.

BUG=722169
Review-Url: https://codereview.chromium.org/2886453002
Cr-Commit-Position: refs/heads/master@{#471795}
Committed: https://chromium.googlesource.com/chromium/src/+/17aab96e1dfcdfdd4e4004dd3949b6b0b872dce2

[bcwhite, 2017-05-15 13:13:53]

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

[commit-bot: I haz the power, 2017-05-15 13:14:06]

Dry run: CQ is trying da patch.

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

[bcwhite, 2017-05-15 13:15:06]

Description was changed from

==========
Use acquire/relase for delayed persistent allocations.

Previously, MakeIterable was the safe way to pass allocations between
threads & processes.  DelayedPersistentAllocations add a second way by
storing the reference in a common location so acquire/release must be
used when accessing that reference to ensure data integrity of the
allocation itself.

BUG=722169
==========

to

==========
Use acquire/relase for delayed persistent allocations.

Previously, iteration was the safe way to pass allocations between
threads & processes.  DelayedPersistentAllocations add a second way by
storing the reference in a common location so acquire/release must be
used when accessing that reference to ensure data integrity of the
allocation itself.

BUG=722169
==========

[commit-bot: I haz the power, 2017-05-15 13:17:42]

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

[commit-bot: I haz the power, 2017-05-15 13:17:43]

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...)
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[bcwhite, 2017-05-15 13:22:21]

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

[commit-bot: I haz the power, 2017-05-15 13:22:38]

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, 2017-05-15 13:26:11]

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

[commit-bot: I haz the power, 2017-05-15 13:26:12]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[bcwhite, 2017-05-15 13:46:12]

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

[commit-bot: I haz the power, 2017-05-15 13:46:31]

Dry run: CQ is trying da patch.

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

[bcwhite, 2017-05-15 14:17:24]

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

[commit-bot: I haz the power, 2017-05-15 14:30:33]

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

[commit-bot: I haz the power, 2017-05-15 14:30:34]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Alexei Svitkine (slow), 2017-05-15 14:31:38]

lgtm

[bcwhite, 2017-05-15 14:38:28]

The CQ bit was checked by bcwhite@chromium.org

[commit-bot: I haz the power, 2017-05-15 14:39:00]

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-05-15 16:43:33]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1494859108633360,
"parent_rev": "70caeb066409953311ea8d4402f1ed6fca82f646", "commit_rev":
"17aab96e1dfcdfdd4e4004dd3949b6b0b872dce2"}

[commit-bot: I haz the power, 2017-05-15 16:43:43]

Description was changed from

==========
Use acquire/relase for delayed persistent allocations.

Previously, iteration was the safe way to pass allocations between
threads & processes.  DelayedPersistentAllocations add a second way by
storing the reference in a common location so acquire/release must be
used when accessing that reference to ensure data integrity of the
allocation itself.

BUG=722169
==========

to

==========
Use acquire/relase for delayed persistent allocations.

Previously, iteration was the safe way to pass allocations between
threads & processes.  DelayedPersistentAllocations add a second way by
storing the reference in a common location so acquire/release must be
used when accessing that reference to ensure data integrity of the
allocation itself.

BUG=722169

Review-Url: https://codereview.chromium.org/2886453002
Cr-Commit-Position: refs/heads/master@{#471795}
Committed:
https://chromium.googlesource.com/chromium/src/+/17aab96e1dfcdfdd4e4004dd3949...
==========

[commit-bot: I haz the power, 2017-05-15 16:43:44]

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