Bug 119587

Summary:

WKBundleFrameCopyProvisionalURL crashes when there is no provisional loader

Product:

WebKit

Reporter:

Alexey Proskuryakov <ap>

Component:

WebKit2

Assignee:

Alexey Proskuryakov <ap>

Status:

RESOLVED FIXED

Severity:

Normal

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
proposed fix	darin: review+

Description Alexey Proskuryakov 2013-08-08 11:56:44 PDT

WKBundleFrameCopyProvisionalURL crashes when there is no provisional loader.

Comment 1 Alexey Proskuryakov 2013-08-08 11:58:24 PDT

Created attachment 208359 [details]
proposed fix

Comment 2 Darin Adler 2013-08-08 12:16:02 PDT

Comment on attachment 208359 [details]
proposed fix

I’m going to assume that it’s better to return null rather than returning the committed string for that frame.

Comment 3 Alexey Proskuryakov 2013-08-08 12:23:21 PDT

Committed <http://trac.webkit.org/changeset/153839>.

Yes, I think that it would be misleading to return committed URL when asked for provisional one. This also matches how -[WebFrame provisionalDataSource] works.