[Webkit-unassigned] [Bug 21073] JS Debugger doesn't handle 'for' loops correctly

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Sep 26 15:23:55 PDT 2008


kmccullough at apple.com changed:

           What    |Removed                     |Added
  Attachment #23848|review?                     |review-
               Flag|                            |

------- Comment #2 from kmccullough at apple.com  2008-09-26 15:23 PDT -------
(From update of attachment 23848)
Looks good however be aware that this will not fully fix the issue described in
this bug.

It will keep #1 & #3, but regress #2 in that subsequent "continues" will hit
the break point each time through the loop, where in VS and XCode it would not.

It will improve #4 & #5 in that you can now step over to the line with the
conditional of the for loop but you cannot step into the initialize or
increment statements of the for loop.

This patch would be fine without fixing all of these issues, however it would
also be good if we had a manual test that exercised each of the statement nodes
so that we could see the behavior and check against regressions.

Please make a test.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

More information about the webkit-unassigned mailing list