Summary: Stepping highlight for dot/bracket expressions in if statements highlights subset of the expression Test: <script> var obj = {condition: true}; debugger; if (obj.condition) true; if (obj["condition"]) true; </script> Steps to Reproduce: 1. Inspect test page 2. Reload to pause at debugger statement 3. Step through => Weird highlights Notes: - Currently we are getting pause locations that look like: if (obj|.condition) if (obj|["condition"]) But what I would expect is: if (|obj.condition) if (|obj["condition"]) It seems the DotAccessNode and BracketAccessNode we create in these cases points to the location of the dot/bracket.
<rdar://problem/28749376>
Created attachment 291454 [details] [PATCH] Proposed Fix I ran a pretty heavy load of js tests and changing these Node's location didn't fail anything. In any case this gets us the better position for breakpoints / stepping.
Comment on attachment 291454 [details] [PATCH] Proposed Fix r=me
Comment on attachment 291454 [details] [PATCH] Proposed Fix Clearing flags on attachment: 291454 Committed r207312: <http://trac.webkit.org/changeset/207312>
All reviewed patches have been landed. Closing bug.