Revert of Fix stack walking to notice if the frame is obviously not valid.

Revert of Fix stack walking to notice if the frame is obviously not valid. (patchset #2 id:20001 of https://codereview.chromium.org/2692123005/ )

Reason for revert:
This likely caused a consistent breakage on CrOS ASAN/LSAN bot: https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/builds/19604

More info on https://crbug.com/677302#c16

Original issue's description:
> Fix stack walking to notice if the frame is obviously not valid.
> 
> If the PC is too small, the frame can't be valid.
> 
> BUG=677302
> 
> Review-Url: https://codereview.chromium.org/2692123005
> Cr-Commit-Position: refs/heads/master@{#451355}
> Committed: https://chromium.googlesource.com/chromium/src/+/f51b7d1a5db6ae289736b84622494086c54cd49a

TBR=mark@chromium.org,wez@chromium.org,erikchen@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=677302
Review-Url: https://codereview.chromium.org/2704933004
Cr-Commit-Position: refs/heads/master@{#451774}
Committed: https://chromium.googlesource.com/chromium/src/+/27f8b7ea6bbee0b512ce820842ebe8bba019b050

[vabr (Chromium), 2017-02-20 13:21:03]

The CQ bit was checked by vabr@chromium.org

[vabr (Chromium), 2017-02-20 13:21:04]

Created Revert of Fix stack walking to notice if the frame is obviously not
valid.

[commit-bot: I haz the power, 2017-02-20 13:21:18]

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-02-20 14:34:23]

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

[commit-bot: I haz the power, 2017-02-20 14:34:24]

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

[vabr (Chromium), 2017-02-20 20:04:09]

Description was changed from

==========
Revert of Fix stack walking to notice if the frame is obviously not valid.
(patchset #2 id:20001 of https://codereview.chromium.org/2692123005/ )

Reason for revert:
This likely caused a consistent breakage on CrOS ASAN/LSAN bot:
https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%2...

More info on https://crbug.com/677302#c16

Original issue's description:
> Fix stack walking to notice if the frame is obviously not valid.
> 
> If the PC is too small, the frame can't be valid.
> 
> BUG=677302
> 
> Review-Url: https://codereview.chromium.org/2692123005
> Cr-Commit-Position: refs/heads/master@{#451355}
> Committed:
https://chromium.googlesource.com/chromium/src/+/f51b7d1a5db6ae289736b8462249...

TBR=mark@chromium.org,wez@chromium.org,erikchen@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=677302
==========

to

==========
Revert of Fix stack walking to notice if the frame is obviously not valid.
(patchset #2 id:20001 of https://codereview.chromium.org/2692123005/ )

Reason for revert:
This likely caused a consistent breakage on CrOS ASAN/LSAN bot:
https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%2...

More info on https://crbug.com/677302#c16

Original issue's description:
> Fix stack walking to notice if the frame is obviously not valid.
> 
> If the PC is too small, the frame can't be valid.
> 
> BUG=677302
> 
> Review-Url: https://codereview.chromium.org/2692123005
> Cr-Commit-Position: refs/heads/master@{#451355}
> Committed:
https://chromium.googlesource.com/chromium/src/+/f51b7d1a5db6ae289736b8462249...

TBR=mark@chromium.org,wez@chromium.org,erikchen@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=677302
==========

[fdoray, 2017-02-21 14:30:20]

On 2017/02/20 14:34:24, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
>
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

Failures in  android_n5x_swarming_rel are unrelated. Trying to reland.

[fdoray, 2017-02-21 14:30:25]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2017-02-21 14:30:41]

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

[vabr (Chromium), 2017-02-21 14:39:08]

On 2017/02/21 14:30:41, commit-bot: I haz the power wrote:
> CQ is trying da patch. Follow status at
>  
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

If this fails again on android_n5x_swarming_rel, then please do not force it
through. I left a message in https://crbug.com/677302#c18 hoping that the owners
would pick this up.

Cheers,
Vaclav

[fdoray, 2017-02-21 14:58:03]

On 2017/02/21 14:39:08, vabr (Chromium) wrote:
> On 2017/02/21 14:30:41, commit-bot: I haz the power wrote:
> > CQ is trying da patch. Follow status at
> >  
> >
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
> 
> If this fails again on android_n5x_swarming_rel, then please do not force it
> through. I left a message in https://crbug.com/677302#c18 hoping that the
owners
> would pick this up.
> 
> Cheers,
> Vaclav

I uploaded a CL to fix the unrelated gl_unittests failure
https://codereview.chromium.org/2708923003. I'll try to reland this afterwards.

[commit-bot: I haz the power, 2017-02-21 16:28:54]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487687425547040, "parent_rev":
"1719c3703669b04157f584424a9d56d5e57cab09", "commit_rev":
"27f8b7ea6bbee0b512ce820842ebe8bba019b050"}

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

Description was changed from

==========
Revert of Fix stack walking to notice if the frame is obviously not valid.
(patchset #2 id:20001 of https://codereview.chromium.org/2692123005/ )

Reason for revert:
This likely caused a consistent breakage on CrOS ASAN/LSAN bot:
https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%2...

More info on https://crbug.com/677302#c16

Original issue's description:
> Fix stack walking to notice if the frame is obviously not valid.
> 
> If the PC is too small, the frame can't be valid.
> 
> BUG=677302
> 
> Review-Url: https://codereview.chromium.org/2692123005
> Cr-Commit-Position: refs/heads/master@{#451355}
> Committed:
https://chromium.googlesource.com/chromium/src/+/f51b7d1a5db6ae289736b8462249...

TBR=mark@chromium.org,wez@chromium.org,erikchen@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=677302
==========

to

==========
Revert of Fix stack walking to notice if the frame is obviously not valid.
(patchset #2 id:20001 of https://codereview.chromium.org/2692123005/ )

Reason for revert:
This likely caused a consistent breakage on CrOS ASAN/LSAN bot:
https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%2...

More info on https://crbug.com/677302#c16

Original issue's description:
> Fix stack walking to notice if the frame is obviously not valid.
> 
> If the PC is too small, the frame can't be valid.
> 
> BUG=677302
> 
> Review-Url: https://codereview.chromium.org/2692123005
> Cr-Commit-Position: refs/heads/master@{#451355}
> Committed:
https://chromium.googlesource.com/chromium/src/+/f51b7d1a5db6ae289736b8462249...

TBR=mark@chromium.org,wez@chromium.org,erikchen@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=677302

Review-Url: https://codereview.chromium.org/2704933004
Cr-Commit-Position: refs/heads/master@{#451774}
Committed:
https://chromium.googlesource.com/chromium/src/+/27f8b7ea6bbee0b512ce820842eb...
==========

[commit-bot: I haz the power, 2017-02-21 16:29:31]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/27f8b7ea6bbee0b512ce820842eb...

[vabr (Chromium), 2017-02-21 20:34:00]

On 2017/02/21 14:58:03, fdoray wrote:
> On 2017/02/21 14:39:08, vabr (Chromium) wrote:
> > On 2017/02/21 14:30:41, commit-bot: I haz the power wrote:
> > > CQ is trying da patch. Follow status at
> > >  
> > >
> >
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
> > 
> > If this fails again on android_n5x_swarming_rel, then please do not force it
> > through. I left a message in https://crbug.com/677302#c18 hoping that the
> owners
> > would pick this up.
> > 
> > Cheers,
> > Vaclav
> 
> I uploaded a CL to fix the unrelated gl_unittests failure
> https://codereview.chromium.org/2708923003. I'll try to reland this
afterwards.

Thanks!
Vaclav