Revert of Add Reclaim Support to ThreadLocalStorage

Revert of Add Reclaim Support to ThreadLocalStorage (patchset #1 id:1 of https://codereview.chromium.org/2363783002/ )

Reason for revert:
Precautionary revert while waiting for https://codereview.chromium.org/2383833004/

Will reland after branch point.

Original issue's description:
> Add Reclaim Support to ThreadLocalStorage
>
> Previously, ThreadLocalStorage::StaticSlot::Free() did not actually
> release the corresponding ThreadLocalStorage slot. It simply cleared
> out the slot and did not reuse it. As a result, each process had a
> finite number of calls to ThreadLocalStorage::StaticSlot::Initialize()
> before running out of slots.
>
> This problem would manifest itself in tests where a single process runs
> many tests that each do their own initialization and uninitialization.
> Tests that involve TLS usage caused the process to run out of TLS slots
> because there was no free support in ThreadLocalStorage.
>
> This change adds in free support by doing what most operating systems
> do, lock and track metadata in an array.
>
> BUG=590907
>
> Committed: https://crrev.com/783d1f9630c86d816669ab23981d0f3722c96808
> Cr-Commit-Position: refs/heads/master@{#421320}

TBR=brettw@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=590907
Committed: https://crrev.com/75f2f7f6f9bdf3d851b5cc3914d32a3d0836c49d
Cr-Commit-Position: refs/heads/master@{#423575}

[robliao, 2016-10-06 02:08:43]

The CQ bit was checked by robliao@chromium.org

[robliao, 2016-10-06 02:08:43]

Created Revert of Add Reclaim Support to ThreadLocalStorage

[commit-bot: I haz the power, 2016-10-06 02:09:14]

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-06 04:31:47]

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

[commit-bot: I haz the power, 2016-10-06 04:31:48]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[robliao, 2016-10-06 17:04:43]

The CQ bit was checked by robliao@chromium.org

[commit-bot: I haz the power, 2016-10-06 17:05:04]

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-06 17:37:49]

Description was changed from

==========
Revert of Add Reclaim Support to ThreadLocalStorage (patchset #1 id:1 of
https://codereview.chromium.org/2363783002/ )

Reason for revert:
Precautionary revert while waiting for
https://codereview.chromium.org/2383833004/

Will reland after branch point.

Original issue's description:
> Add Reclaim Support to ThreadLocalStorage
> 
> Previously, ThreadLocalStorage::StaticSlot::Free() did not actually
> release the corresponding ThreadLocalStorage slot. It simply cleared
> out the slot and did not reuse it. As a result, each process had a
> finite number of calls to ThreadLocalStorage::StaticSlot::Initialize()
> before running out of slots.
> 
> This problem would manifest itself in tests where a single process runs
> many tests that each do their own initialization and uninitialization.
> Tests that involve TLS usage caused the process to run out of TLS slots
> because there was no free support in ThreadLocalStorage.
> 
> This change adds in free support by doing what most operating systems
> do, lock and track metadata in an array.
> 
> BUG=590907
> 
> Committed: https://crrev.com/783d1f9630c86d816669ab23981d0f3722c96808
> Cr-Commit-Position: refs/heads/master@{#421320}

TBR=brettw@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=590907
==========

to

==========
Revert of Add Reclaim Support to ThreadLocalStorage (patchset #1 id:1 of
https://codereview.chromium.org/2363783002/ )

Reason for revert:
Precautionary revert while waiting for
https://codereview.chromium.org/2383833004/

Will reland after branch point.

Original issue's description:
> Add Reclaim Support to ThreadLocalStorage
> 
> Previously, ThreadLocalStorage::StaticSlot::Free() did not actually
> release the corresponding ThreadLocalStorage slot. It simply cleared
> out the slot and did not reuse it. As a result, each process had a
> finite number of calls to ThreadLocalStorage::StaticSlot::Initialize()
> before running out of slots.
> 
> This problem would manifest itself in tests where a single process runs
> many tests that each do their own initialization and uninitialization.
> Tests that involve TLS usage caused the process to run out of TLS slots
> because there was no free support in ThreadLocalStorage.
> 
> This change adds in free support by doing what most operating systems
> do, lock and track metadata in an array.
> 
> BUG=590907
> 
> Committed: https://crrev.com/783d1f9630c86d816669ab23981d0f3722c96808
> Cr-Commit-Position: refs/heads/master@{#421320}

TBR=brettw@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=590907
==========

[commit-bot: I haz the power, 2016-10-06 17:37:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-06 17:40:32]

Description was changed from

==========
Revert of Add Reclaim Support to ThreadLocalStorage (patchset #1 id:1 of
https://codereview.chromium.org/2363783002/ )

Reason for revert:
Precautionary revert while waiting for
https://codereview.chromium.org/2383833004/

Will reland after branch point.

Original issue's description:
> Add Reclaim Support to ThreadLocalStorage
> 
> Previously, ThreadLocalStorage::StaticSlot::Free() did not actually
> release the corresponding ThreadLocalStorage slot. It simply cleared
> out the slot and did not reuse it. As a result, each process had a
> finite number of calls to ThreadLocalStorage::StaticSlot::Initialize()
> before running out of slots.
> 
> This problem would manifest itself in tests where a single process runs
> many tests that each do their own initialization and uninitialization.
> Tests that involve TLS usage caused the process to run out of TLS slots
> because there was no free support in ThreadLocalStorage.
> 
> This change adds in free support by doing what most operating systems
> do, lock and track metadata in an array.
> 
> BUG=590907
> 
> Committed: https://crrev.com/783d1f9630c86d816669ab23981d0f3722c96808
> Cr-Commit-Position: refs/heads/master@{#421320}

TBR=brettw@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=590907
==========

to

==========
Revert of Add Reclaim Support to ThreadLocalStorage (patchset #1 id:1 of
https://codereview.chromium.org/2363783002/ )

Reason for revert:
Precautionary revert while waiting for
https://codereview.chromium.org/2383833004/

Will reland after branch point.

Original issue's description:
> Add Reclaim Support to ThreadLocalStorage
>
> Previously, ThreadLocalStorage::StaticSlot::Free() did not actually
> release the corresponding ThreadLocalStorage slot. It simply cleared
> out the slot and did not reuse it. As a result, each process had a
> finite number of calls to ThreadLocalStorage::StaticSlot::Initialize()
> before running out of slots.
>
> This problem would manifest itself in tests where a single process runs
> many tests that each do their own initialization and uninitialization.
> Tests that involve TLS usage caused the process to run out of TLS slots
> because there was no free support in ThreadLocalStorage.
>
> This change adds in free support by doing what most operating systems
> do, lock and track metadata in an array.
>
> BUG=590907
>
> Committed: https://crrev.com/783d1f9630c86d816669ab23981d0f3722c96808
> Cr-Commit-Position: refs/heads/master@{#421320}

TBR=brettw@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=590907

Committed: https://crrev.com/75f2f7f6f9bdf3d851b5cc3914d32a3d0836c49d
Cr-Commit-Position: refs/heads/master@{#423575}
==========

[commit-bot: I haz the power, 2016-10-06 17:40:33]

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