Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(811)

Issue 193059: Find the correct function for script break points (Closed)

Created:
11 years, 3 months ago by Søren Thygesen Gjesse
Modified:
9 years, 7 months ago
CC:
v8-dev
Visibility:
Public.

Description

Find the correct function for script break points The algorithm for finding the inner-most function containing a script break point was not correct when the script only contained one function. In that case the script function and not the actual function in the script could be returned depending on the order of the objects in the heap. TEST=cctest/test-debug/ScriptBreakPointReload BUG=none Committed: http://code.google.com/p/v8/source/detail?r=2862

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -5 lines) Patch
M src/objects.h View 1 chunk +1 line, -3 lines 0 comments Download
M src/runtime.cc View 1 1 chunk +14 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Søren Thygesen Gjesse
11 years, 3 months ago (2009-09-09 15:13:41 UTC) #1
Mads Ager (chromium)
11 years, 3 months ago (2009-09-09 15:15:55 UTC) #2
LGTM!

Powered by Google App Engine
This is Rietveld 408576698