[webkit-reviews] review requested: [Bug 127532] Removing the need for Debugger* and m_shouldPause op_debug check : [Attachment 222084] patch 2: fixed initialization bug.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jan 24 00:44:48 PST 2014


Mark Lam <mark.lam at apple.com> has asked  for review:
Bug 127532: Removing the need for Debugger* and m_shouldPause op_debug check
https://bugs.webkit.org/show_bug.cgi?id=127532

Attachment 222084: patch 2: fixed initialization bug.
https://bugs.webkit.org/attachment.cgi?id=222084&action=review

------- Additional Comments from Mark Lam <mark.lam at apple.com>
This patch fixes the initialization of CodeBlock::m_steppingMode.  The Octane
scores are now:

baseline w/ WebInspector + breakpoint:	 2640
patched w/ WebInspector + breakpoint:	 2830	 1.07x progression

This patched has passed the layout tests.


More information about the webkit-reviews mailing list