Revert of Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags()

Revert of Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags() (patchset #3 id:40001 of https://codereview.chromium.org/1419023008/ )

Reason for revert:
Asserts no longer necessary; the bug is due to blink reinitialisation (which is now prevented elsewhere) not an internal problem with PartitionAlloc.

Original issue's description:
> Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags()
>
> To diagnose the cause of crashes on WebView.
>
> BUG=514141
>
> Committed: https://crrev.com/e0a8f09ae45e944615cacd795a05b6243e3573e1
> Cr-Commit-Position: refs/heads/master@{#358295}

# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=514141
Committed: https://crrev.com/75b542feeeae9067b91349606c9dc48a9f50dd58
Cr-Commit-Position: refs/heads/master@{#427095}

[Torne, 2016-10-24 13:34:59]

Created Revert of Add RELEASE_ASSERT() for |bucket| in
partitionAllocGenericFlags()

[Torne, 2016-10-24 13:35:16]

Description was changed from

==========
Revert of Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags()
(patchset #3 id:40001 of https://codereview.chromium.org/1419023008/ )

Reason for revert:
Asserts no longer necessary; the bug is due to blink reinitialisation (which is
now prevented elsewhere) not an internal problem with PartitionAlloc.

Original issue's description:
> Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags()
> 
> To diagnose the cause of crashes on WebView.
> 
> BUG=514141
> 
> Committed: https://crrev.com/e0a8f09ae45e944615cacd795a05b6243e3573e1
> Cr-Commit-Position: refs/heads/master@{#358295}

TBR=haraken@chromium.org,bashi@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=514141
==========

to

==========
Revert of Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags()
(patchset #3 id:40001 of https://codereview.chromium.org/1419023008/ )

Reason for revert:
Asserts no longer necessary; the bug is due to blink reinitialisation (which is
now prevented elsewhere) not an internal problem with PartitionAlloc.

Original issue's description:
> Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags()
> 
> To diagnose the cause of crashes on WebView.
> 
> BUG=514141
> 
> Committed: https://crrev.com/e0a8f09ae45e944615cacd795a05b6243e3573e1
> Cr-Commit-Position: refs/heads/master@{#358295}

# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=514141
==========

[Torne, 2016-10-24 13:35:49]

Going to revert this as esprehn@ suggested in the bug; we know it's not an
allocator issue now.

[haraken, 2016-10-24 13:37:31]

LGTM

[Torne, 2016-10-24 13:40:09]

The CQ bit was checked by torne@chromium.org

[commit-bot: I haz the power, 2016-10-24 13:40:31]

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-24 13:42:48]

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

[commit-bot: I haz the power, 2016-10-24 13:42:49]

Try jobs failed on following builders:
  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_...)

[Torne, 2016-10-24 13:43:44]

Too optimistic about a year-old patch applying in reverse, I guess. I'll upload
a rebased revert :)

[Torne, 2016-10-24 13:48:12]

The CQ bit was checked by torne@chromium.org

[Torne, 2016-10-24 13:48:12]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2444913002/#ps40001
(title: "Rebase")

[commit-bot: I haz the power, 2016-10-24 13:48:25]

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-24 16:50:46]

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

[commit-bot: I haz the power, 2016-10-24 16:50:50]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Torne, 2016-10-24 17:01:34]

The CQ bit was checked by torne@chromium.org

[commit-bot: I haz the power, 2016-10-24 17:02:08]

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-24 17:53:53]

Description was changed from

==========
Revert of Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags()
(patchset #3 id:40001 of https://codereview.chromium.org/1419023008/ )

Reason for revert:
Asserts no longer necessary; the bug is due to blink reinitialisation (which is
now prevented elsewhere) not an internal problem with PartitionAlloc.

Original issue's description:
> Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags()
> 
> To diagnose the cause of crashes on WebView.
> 
> BUG=514141
> 
> Committed: https://crrev.com/e0a8f09ae45e944615cacd795a05b6243e3573e1
> Cr-Commit-Position: refs/heads/master@{#358295}

# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=514141
==========

to

==========
Revert of Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags()
(patchset #3 id:40001 of https://codereview.chromium.org/1419023008/ )

Reason for revert:
Asserts no longer necessary; the bug is due to blink reinitialisation (which is
now prevented elsewhere) not an internal problem with PartitionAlloc.

Original issue's description:
> Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags()
> 
> To diagnose the cause of crashes on WebView.
> 
> BUG=514141
> 
> Committed: https://crrev.com/e0a8f09ae45e944615cacd795a05b6243e3573e1
> Cr-Commit-Position: refs/heads/master@{#358295}

# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=514141
==========

[commit-bot: I haz the power, 2016-10-24 17:53:54]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-10-24 18:22:36]

Description was changed from

==========
Revert of Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags()
(patchset #3 id:40001 of https://codereview.chromium.org/1419023008/ )

Reason for revert:
Asserts no longer necessary; the bug is due to blink reinitialisation (which is
now prevented elsewhere) not an internal problem with PartitionAlloc.

Original issue's description:
> Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags()
> 
> To diagnose the cause of crashes on WebView.
> 
> BUG=514141
> 
> Committed: https://crrev.com/e0a8f09ae45e944615cacd795a05b6243e3573e1
> Cr-Commit-Position: refs/heads/master@{#358295}

# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=514141
==========

to

==========
Revert of Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags()
(patchset #3 id:40001 of https://codereview.chromium.org/1419023008/ )

Reason for revert:
Asserts no longer necessary; the bug is due to blink reinitialisation (which is
now prevented elsewhere) not an internal problem with PartitionAlloc.

Original issue's description:
> Add RELEASE_ASSERT() for |bucket| in partitionAllocGenericFlags()
>
> To diagnose the cause of crashes on WebView.
>
> BUG=514141
>
> Committed: https://crrev.com/e0a8f09ae45e944615cacd795a05b6243e3573e1
> Cr-Commit-Position: refs/heads/master@{#358295}

# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=514141

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

[commit-bot: I haz the power, 2016-10-24 18:22:37]

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

[bashi, 2016-10-24 23:56:29]

lgtm, thanks for reverting this!