Summary: | WebPageProxy::didFinishDocumentLoadForFrame() crashing in WPE and GTK+ ports | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Claudio Saavedra <csaavedra> | ||||
Component: | New Bugs | Assignee: | Claudio Saavedra <csaavedra> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | achristensen, dbates, webkit-bug-importer, wilander | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Claudio Saavedra
2019-02-18 04:41:11 PST
Created attachment 362290 [details]
Patch
This was introduced in https://trac.webkit.org/changeset/241451/ Comment on attachment 362290 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=362290&action=review Null checking navigation before using it is fine, but the way you did it here makes it hard to read and see where the assignment is done. Could we keep the assignment on its own line? > Source/WebKit/ChangeLog:3 > + WebPageProxy::didFinishDocumentLoadForFrame() crashing in WPE and GTK+ ports Why is this only WPE and GTK? Under what conditions does it crash? Comment on attachment 362290 [details] Patch https://trac.webkit.org/changeset/241735/webkit |