[webkit-reviews] review granted: [Bug 55201] [Web Timing] loadEvent timing should refer to first load event if there are many : [Attachment 83758] Patch

[bugzilla-daemon at webkit.org]

Tony Gentilcore <tonyg at chromium.org> has granted James Simonsen
<simonjam at chromium.org>'s request for review:
Bug 55201: [Web Timing] loadEvent timing should refer to first load event if
there are many
https://bugs.webkit.org/show_bug.cgi?id=55201

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

------- Additional Comments from Tony Gentilcore <tonyg at chromium.org>
This fix looks good. But I'm wondering if it is ever correct to fire multiple
load events. If not, would it be better to fix that underlying bug instead of
this?