Chromium Code Reviews
Help | Chromium Project | Sign in
(470)

Issue 7780033: Debugger: fix stepping next with trycatch recursion (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 7 months ago by indutny
Modified:
2 years, 7 months ago
CC:
v8-dev_googlegroups.com
Base URL:
gh:indutny/v8@master
Visibility:
Public.

Description

Debugger: fix stepping next with trycatch recursion

Added depth check to StepNextContinue. Do step out and queue actual StepNext if check failed.

R=yangguo@chromium.org,danno@chromium.org,vegorov@chromium.org
BUG=v8:1639
TEST=

Committed: http://code.google.com/p/v8/source/detail?r=9264

Patch Set 1 : v8: Debug StepOut if StepNext gone deeper #

Patch Set 2 : v8: Debug StepOut if StepNext gone deeper (skipBreak_ => skipBreak) #

Patch Set 3 : fix depth change check #

Patch Set 4 : fifixed code style #

Patch Set 5 : sort authors #

Total comments: 1

Patch Set 6 : remove queued_step_action_ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+49 lines, -2 lines) Lint Patch
M AUTHORS View 1 2 3 4 1 chunk +1 line, -0 lines 0 comments ? errors Download
M src/debug.h View 1 2 3 4 5 1 chunk +3 lines, -0 lines 0 comments ? errors Download
M src/debug.cc View 1 2 3 4 5 3 chunks +45 lines, -2 lines 0 comments ? errors Download
Trybot results:
Commit:

Messages

Total messages: 19
indutny
2 years, 7 months ago #1
Yang
On 2011/09/12 10:09:16, indutny wrote: Thanks for the patch. I'll take a look into it ...
2 years, 7 months ago #2
indutny
On 2011/09/12 10:42:37, Yang wrote: > On 2011/09/12 10:09:16, indutny wrote: > > Thanks for ...
2 years, 7 months ago #3
indutny
Sorry, lines 982-985 are useless and came from obsolete implementation of fix.
2 years, 7 months ago #4
Yang
On 2011/09/12 11:13:33, indutny wrote: > Sorry, lines 982-985 are useless and came from obsolete ...
2 years, 7 months ago #5
indutny
Oh, I think I know what causes a problem, I'll push changes in a 10-20 ...
2 years, 7 months ago #6
Yang
On 2011/09/12 12:50:02, indutny wrote: > Oh, I think I know what causes a problem, ...
2 years, 7 months ago #7
indutny
2 years, 7 months ago #8
indutny
Oh, git-cl created second patch set, removed old one. `tools/test.py mjsunit/debug*` reports 0 errors now. ...
2 years, 7 months ago #9
indutny
2 years, 7 months ago #10
Yang
On 2011/09/12 13:19:41, indutny wrote: The mjsunit tests are not the only ones. If you ...
2 years, 7 months ago #11
indutny
Can you please provide me those tests' names?
2 years, 7 months ago #12
indutny
Nevermind, I know names now
2 years, 7 months ago #13
indutny
Fixed patch, 100% passing now
2 years, 7 months ago #14
indutny
So what is a current status of this issue? That blocks some of my work ...
2 years, 7 months ago #15
Yang
Looks pretty good so far, except for one little thing remarked below. You might want ...
2 years, 7 months ago #16
indutny
Ok, sounds good for me. Fixed and uploaded changes, tests are still passing.
2 years, 7 months ago #17
Yang
On 2011/09/13 14:32:03, indutny wrote: > Ok, sounds good for me. > Fixed and uploaded ...
2 years, 7 months ago #18
indutny
2 years, 7 months ago #19
Great! Thank you!
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6