[webkit-reviews] review granted: [Bug 42102] Moving BOM stripping from JSC specific code to V8+JSC shared code caused V8 crashes : [Attachment 61289] Patch

[bugzilla-daemon at webkit.org]

Oliver Hunt <oliver at apple.com> has granted Tony Gentilcore
<tonyg at chromium.org>'s request for review:
Bug 42102: Moving BOM stripping from JSC specific code to V8+JSC shared code
caused V8 crashes
https://bugs.webkit.org/show_bug.cgi?id=42102

Attachment 61289: Patch
https://bugs.webkit.org/attachment.cgi?id=61289&action=review

------- Additional Comments from Oliver Hunt <oliver at apple.com>
As long as the V8 lexer handles boms itself this is unlikely to be a win, you
should test impact after removing bom handling from the v8 lexer.