[webkit-reviews] review granted: [Bug 99142] fileReader abort case causes Chromium renderer crash : [Attachment 168373] Patch

[bugzilla-daemon at webkit.org]

Kentaro Hara <haraken at chromium.org> has granted Ningxin Hu
<ningxin.hu at intel.com>'s request for review:
Bug 99142: fileReader abort case causes Chromium renderer crash
https://bugs.webkit.org/show_bug.cgi?id=99142

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

------- Additional Comments from Kentaro Hara <haraken at chromium.org>
Thanks for the clarification.

> If we could let Chromium (with multiprocess) instead of DumpRenderTree to run
the test, this crash should be covered. Do we have this way?

I don't know... Given that the change looks reasonable and that you've manually
tested, it would be OK to land it without tests. (It would be better to
describe the situation in ChangeLog, instead of "No new tests".)