WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 163428
Web Inspector: Improve debugger highlight when inside of getter/setter calls
https://bugs.webkit.org/show_bug.cgi?id=163428
Summary
Web Inspector: Improve debugger highlight when inside of getter/setter calls
Joseph Pecoraro
Reported
2016-10-13 20:43:14 PDT
Summary: Improve debugger highlight when inside of getter/setter calls. When paused inside of a getter/setter the user may select the parent call frame to see where the getter/setter was invoked. In these cases we should highlight the best part of the parent member expression that we can. For example: - * is the location JavaScriptCore tells us we are (the divot) - |...| is the range we want to highlight Triggering getter via dot access: foo*.x.y.z => inside x looking at parent call frame => |foo.x|.y.z foo.x*.y.z => inside y looking at parent call frame => |foo.x.y|.z Triggering getter via bracket access: foo*["x"]["y"]["z"] => inside x looking at parent call frame => |foo["x"]|["y"]["z"] foo["x"]*["y"]["z"] => inside y looking at parent call frame => |foo["x"]["y"]|["z"] We already highlight the most useful expression when inside of a function call: func*(foo.x) => inside func => |func(foo.x)| foo.func*(x) => inside func => |foo.func(x)|
Attachments
[PATCH] Proposed Fix
(6.07 KB, patch)
2016-10-13 20:46 PDT
,
Joseph Pecoraro
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2016-10-13 20:43:35 PDT
<
rdar://problem/28769061
>
Joseph Pecoraro
Comment 2
2016-10-13 20:46:17 PDT
Created
attachment 291562
[details]
[PATCH] Proposed Fix
WebKit Commit Bot
Comment 3
2016-10-14 16:06:45 PDT
Comment on
attachment 291562
[details]
[PATCH] Proposed Fix Clearing flags on attachment: 291562 Committed
r207358
: <
http://trac.webkit.org/changeset/207358
>
WebKit Commit Bot
Comment 4
2016-10-14 16:06:51 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug