[webkit-reviews] review requested: [Bug 95949] [GTK] [WK2] Crash when navigating between pages : [Attachment 162478] A different patch

[bugzilla-daemon at webkit.org]

Carlos Garcia Campos <cgarcia at igalia.com> has asked  for review:
Bug 95949: [GTK] [WK2] Crash when navigating between pages
https://bugs.webkit.org/show_bug.cgi?id=95949

Attachment 162478: A different patch
https://bugs.webkit.org/attachment.cgi?id=162478&action=review

------- Additional Comments from Carlos Garcia Campos <cgarcia at igalia.com>
I've checked the problem and it's related to bug
https://bugs.webkit.org/show_bug.cgi?id=91478. For pages loaded from the
history cache, the parameter isMainResource is always true, so all subresources
are considered the main one. When the loca committed signal is emitted, the
mainResource variable doesn't point to the main resource, but to the last
subresources that is being loaded in this moment, and it can happen that the
response hasn't been received yet for such subresource. So, we can simply
assume that the main resource of the main frame is the main resource of the web
view, and that when the load has been committed the main resource has already
received a response.