Callers of JSString::getIndex should check for OOM exceptions
Created attachment 357327 [details] Patch
Comment on attachment 357327 [details] Patch r=me
rdar://problem/44451952
Comment on attachment 357327 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=357327&action=review > JSTests/stress/StringObject-define-length-getter-rope-string-oom.js:3 > +let char16 = decodeURI('%E7%9A%84'); > +let rope = char16.padEnd(2147483644, 1); > +rope.__defineGetter__(256, function () {}); Whoops forgot to wrap this in a try catch. Will do that before landing.
Created attachment 357330 [details] Patch for landing
Comment on attachment 357330 [details] Patch for landing Clearing flags on attachment: 357330 Committed r239227: <https://trac.webkit.org/changeset/239227>
All reviewed patches have been landed. Closing bug.