[webkit-reviews] review requested: [Bug 59100] Web Inspector: Tiny improvement to UI for adding an XHR breakpoint : [Attachment 90529] Strawman patch for discussion.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Apr 21 07:34:49 PDT 2011


Mike West <mkwst at chromium.org> has asked  for review:
Bug 59100: Web Inspector: Tiny improvement to UI for adding an XHR breakpoint
https://bugs.webkit.org/show_bug.cgi?id=59100

Attachment 90529: Strawman patch for discussion.
https://bugs.webkit.org/attachment.cgi?id=90529&action=review

------- Additional Comments from Mike West <mkwst at chromium.org>
This is the first patch I'm attempting to contribute to WebKit, so please be
gentle.  Happily, it's tiny, so I hope there's not much I could have done
incorrectly. :)

Assuming that the functionality is acceptable, I'd appreciate a pointer to
which layout test I ought to modify to deal with the new addition. I didn't
find anything (and nothing broke... ?), but I'm happy to add tests.

Thanks!


More information about the webkit-reviews mailing list