WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
143793
Web Inspector: "Log Value" of a value inside of an array, does not log the innermost value
https://bugs.webkit.org/show_bug.cgi?id=143793
Summary
Web Inspector: "Log Value" of a value inside of an array, does not log the in...
Joseph Pecoraro
Reported
2015-04-15 13:06:08 PDT
* SUMMARY "Log Value" of a value inside of an array, does not log the innermost value. * STEPS TO REPRODUCE 1. js> myObject = { array: [1, {name:"Joe", age:123}] }; dir(myObject) 2. Expand all the way to the inner object 3. Right click "name" and choose "Log Value" => Expected "Joe", but got the containing object {name: "Joe", age: 123}
Attachments
[PATCH] Proposed Fix
(5.43 KB, patch)
2015-04-15 21:32 PDT
,
Joseph Pecoraro
burg
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2015-04-15 13:06:45 PDT
<
rdar://problem/20558139
>
Joseph Pecoraro
Comment 2
2015-04-15 13:18:51 PDT
This happens because the GeneralTreeElement event handlers are "useCapture" and thus capture top down. Removing capture mode things work as expected.
Joseph Pecoraro
Comment 3
2015-04-15 21:08:13 PDT
Reusing this to improve context menus in the console in general.
Joseph Pecoraro
Comment 4
2015-04-15 21:32:29 PDT
Created
attachment 250898
[details]
[PATCH] Proposed Fix
Brian Burg
Comment 5
2015-04-16 08:08:31 PDT
Comment on
attachment 250898
[details]
[PATCH] Proposed Fix r=me Any intuition for why this used to be a capturing handler?
Timothy Hatcher
Comment 6
2015-04-16 10:18:04 PDT
(In reply to
comment #5
)
> Comment on
attachment 250898
[details]
> [PATCH] Proposed Fix > > r=me > > Any intuition for why this used to be a capturing handler?
Joe asked me about it before posting this and I had no clue. Likely legacy and a mistake.
Joseph Pecoraro
Comment 7
2015-04-16 13:55:55 PDT
http://trac.webkit.org/changeset/182909
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