Debugger test updated to avoid setting breakpoints into random scripts.

Debugger test updated to avoid setting breakpoints into random scripts.

Committed: https://crrev.com/edc61b26b412410c984e6c1bbc100d4cc3ace6c7
Cr-Commit-Position: refs/heads/master@{#29668}

[Igor Sheludko, 2015-07-14 13:54:19]

Patchset #1 (id:1) has been deleted

[Igor Sheludko, 2015-07-14 13:55:42]

ishell@chromium.org changed reviewers:
+ yangguo@chromium.org

[Igor Sheludko, 2015-07-14 13:55:42]

PTAL

[Igor Sheludko, 2015-07-14 14:21:06]

Patchset #1 (id:20001) has been deleted

[Igor Sheludko, 2015-07-14 14:48:45]

On 2015/07/14 13:55:42, Igor Sheludko wrote:
> PTAL

For some reason this updated test now fails the same way on x64.optdebug (only):

#
# Fatal error in ../src/debug.cc, line 228
# Check failed: code()->has_debug_break_slots().
#

[Yang, 2015-07-14 14:53:19]

On 2015/07/14 14:48:45, Igor Sheludko wrote:
> On 2015/07/14 13:55:42, Igor Sheludko wrote:
> > PTAL
> 
> For some reason this updated test now fails the same way on x64.optdebug
(only):
> 
> #
> # Fatal error in ../src/debug.cc, line 228
> # Check failed: code()->has_debug_break_slots().
> #

lgtm!

[Igor Sheludko, 2015-07-14 16:30:47]

On 2015/07/14 14:53:19, Yang wrote:
> On 2015/07/14 14:48:45, Igor Sheludko wrote:
> > On 2015/07/14 13:55:42, Igor Sheludko wrote:
> > > PTAL
> > 
> > For some reason this updated test now fails the same way on x64.optdebug
> (only):
> > 
> > #
> > # Fatal error in ../src/debug.cc, line 228
> > # Check failed: code()->has_debug_break_slots().
> > #
> 
> lgtm!

I found the reason. The original test was soooo outdated...

[Igor Sheludko, 2015-07-15 07:33:00]

The CQ bit was checked by ishell@chromium.org

[Igor Sheludko, 2015-07-15 07:33:00]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/1231893007/#ps60001
(title: "Test fixed")

[commit-bot: I haz the power, 2015-07-15 07:33:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1231893007/60001

[commit-bot: I haz the power, 2015-07-15 07:42:00]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2015-07-15 07:42:16]

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