Revert of [heap] Move slot filtering logic into sweeper.

Revert of [heap] Move slot filtering logic into sweeper. (patchset #4 id:60001 of https://codereview.chromium.org/2418773002/ )

Reason for revert:
[Sheriff] Speculative revert for heap corruption on all platforms, e.g.:
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20-%20debug/builds/12377
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20-%20debug/builds/12379
https://build.chromium.org/p/client.v8/builders/V8%20Win32/builds/4819
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20nosnap%20-%20shared/builds/16783
https://build.chromium.org/p/client.v8/builders/V8%20Mac64%20-%20debug/builds/10007

Original issue's description:
> [heap] Move slot filtering logic into sweeper.
>
> BUG=chromium:648568
>
> Committed: https://crrev.com/18db69c38c93450c1ae957999fc48c465f111f00
> Cr-Commit-Position: refs/heads/master@{#40267}

TBR=ulan@chromium.org,mlippautz@chromium.org,hpayer@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:648568
Committed: https://crrev.com/cdc3459a858a6b86e96cedf841f7b95e49fef272
Cr-Commit-Position: refs/heads/master@{#40292}

[Michael Achenbach, 2016-10-14 06:58:49]

The CQ bit was checked by machenbach@chromium.org

[Michael Achenbach, 2016-10-14 06:58:49]

Created Revert of [heap] Move slot filtering logic into sweeper.

[commit-bot: I haz the power, 2016-10-14 06:59: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, 2016-10-14 06:59:12]

Description was changed from

==========
Revert of [heap] Move slot filtering logic into sweeper. (patchset #4 id:60001
of https://codereview.chromium.org/2418773002/ )

Reason for revert:
[Sheriff] Speculative revert for heap corruption on all platforms, e.g.:
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20-%20debug/buil...
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20-%20debug/buil...
https://build.chromium.org/p/client.v8/builders/V8%20Win32/builds/4819
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20nosnap%20-%2...
https://build.chromium.org/p/client.v8/builders/V8%20Mac64%20-%20debug/builds...

Original issue's description:
> [heap] Move slot filtering logic into sweeper.
> 
> BUG=chromium:648568
> 
> Committed: https://crrev.com/18db69c38c93450c1ae957999fc48c465f111f00
> Cr-Commit-Position: refs/heads/master@{#40267}

TBR=ulan@chromium.org,mlippautz@chromium.org,hpayer@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:648568
==========

to

==========
Revert of [heap] Move slot filtering logic into sweeper. (patchset #4 id:60001
of https://codereview.chromium.org/2418773002/ )

Reason for revert:
[Sheriff] Speculative revert for heap corruption on all platforms, e.g.:
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20-%20debug/buil...
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20-%20debug/buil...
https://build.chromium.org/p/client.v8/builders/V8%20Win32/builds/4819
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20nosnap%20-%2...
https://build.chromium.org/p/client.v8/builders/V8%20Mac64%20-%20debug/builds...

Original issue's description:
> [heap] Move slot filtering logic into sweeper.
> 
> BUG=chromium:648568
> 
> Committed: https://crrev.com/18db69c38c93450c1ae957999fc48c465f111f00
> Cr-Commit-Position: refs/heads/master@{#40267}

TBR=ulan@chromium.org,mlippautz@chromium.org,hpayer@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:648568
==========

[commit-bot: I haz the power, 2016-10-14 06:59:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-14 06:59:45]

Description was changed from

==========
Revert of [heap] Move slot filtering logic into sweeper. (patchset #4 id:60001
of https://codereview.chromium.org/2418773002/ )

Reason for revert:
[Sheriff] Speculative revert for heap corruption on all platforms, e.g.:
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20-%20debug/buil...
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20-%20debug/buil...
https://build.chromium.org/p/client.v8/builders/V8%20Win32/builds/4819
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20nosnap%20-%2...
https://build.chromium.org/p/client.v8/builders/V8%20Mac64%20-%20debug/builds...

Original issue's description:
> [heap] Move slot filtering logic into sweeper.
> 
> BUG=chromium:648568
> 
> Committed: https://crrev.com/18db69c38c93450c1ae957999fc48c465f111f00
> Cr-Commit-Position: refs/heads/master@{#40267}

TBR=ulan@chromium.org,mlippautz@chromium.org,hpayer@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:648568
==========

to

==========
Revert of [heap] Move slot filtering logic into sweeper. (patchset #4 id:60001
of https://codereview.chromium.org/2418773002/ )

Reason for revert:
[Sheriff] Speculative revert for heap corruption on all platforms, e.g.:
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20-%20debug/buil...
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20-%20debug/buil...
https://build.chromium.org/p/client.v8/builders/V8%20Win32/builds/4819
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20nosnap%20-%2...
https://build.chromium.org/p/client.v8/builders/V8%20Mac64%20-%20debug/builds...

Original issue's description:
> [heap] Move slot filtering logic into sweeper.
>
> BUG=chromium:648568
>
> Committed: https://crrev.com/18db69c38c93450c1ae957999fc48c465f111f00
> Cr-Commit-Position: refs/heads/master@{#40267}

TBR=ulan@chromium.org,mlippautz@chromium.org,hpayer@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:648568

Committed: https://crrev.com/cdc3459a858a6b86e96cedf841f7b95e49fef272
Cr-Commit-Position: refs/heads/master@{#40292}
==========

[commit-bot: I haz the power, 2016-10-14 06:59:46]

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

[Michael Achenbach, 2016-10-14 09:52:48]

FYI: The redness stopped after this revert. Is it a problem getting a canary
with this commit in?

@hablich: Can you check if we rolled in the range where this CL was in?

[Michael Hablich, 2016-10-14 09:59:26]

On 2016/10/14 09:52:48, machenbach (slow) wrote:
> FYI: The redness stopped after this revert. Is it a problem getting a canary
> with this commit in?
> 
> @hablich: Can you check if we rolled in the range where this CL was in?

This CL is deployed on Canary 2890 in v8
https://chromium.googlesource.com/v8/v8/+log/ec798714..24369beb.

[Yang, 2016-10-14 11:36:08]

On 2016/10/14 09:59:26, Hablich wrote:
> On 2016/10/14 09:52:48, machenbach (slow) wrote:
> > FYI: The redness stopped after this revert. Is it a problem getting a canary
> > with this commit in?
> > 
> > @hablich: Can you check if we rolled in the range where this CL was in?
> 
> This CL is deployed on Canary 2890 in v8
> https://chromium.googlesource.com/v8/v8/+log/ec798714..24369beb.

Note: this revert seems to have fixed the flaky test failures.