.
Created attachment 446219 [details] Patch
Created attachment 446220 [details] Patch
Created attachment 446235 [details] Patch
This is not actually needed for bug 191379.
actually, this might be necessary
Comment on attachment 446235 [details] Patch (In reply to Devin Rousso from comment #5) > actually, this might be necessary It's not really clear from the ChangeLog (or the comments) why this is the case, but it does seem like a fairly straightforward refactoring. I would've thought that simply reading the item at index 0 (or none if there are no items) would achieve the same thing.
(In reply to Wenson Hsieh from comment #6) > (In reply to Devin Rousso from comment #5) > > actually, this might be necessary > > It's not really clear from the ChangeLog (or the comments) why this is the case, but it does seem like a fairly straightforward refactoring. I would've thought that simply reading the item at index 0 (or none if there are no items) would achieve the same thing. I agree that it's not super clear, but as you said since it's pretty straightforward I didn't see a huge benefit in digging into it, both because it would allow us to match WK1 and this approach does work.
Created attachment 446629 [details] Patch
Committed r286870 (245101@main): <https://commits.webkit.org/245101@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 446629 [details].
<rdar://problem/86336945>