Bug 23733 - Assertion fails in SecurityOrigin.cpp
Summary: Assertion fails in SecurityOrigin.cpp
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: All OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-04 09:52 PST by Yael
Modified: 2009-02-05 06:34 PST (History)
0 users

See Also:

Attachments
Use the KURL constructor that takes 2 parameters instead of one. (1.18 KB, patch)
2009-02-04 09:54 PST, Yael 		ap: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Yael 2009-02-04 09:52:58 PST 
Assertion fails when creating a SecurityOrigin from DatabaseIdentifier. That is because a DatabaseIdentifier includes an origin without a path. The fix is to use a different constructor to KRUL, one that does not asert.
Comment 1 Yael 2009-02-04 09:54:32 PST 
Created attachment 27317 [details]
Use the KURL constructor that takes 2 parameters instead of one.
Comment 2 Alexey Proskuryakov 2009-02-04 10:14:15 PST 
Comment on attachment 27317 [details]
Use the KURL constructor that takes 2 parameters instead of one.

r=me

It's always better to have a bug URL in ChangeLog.
Comment 3 Yael 2009-02-04 10:27:38 PST 
Thank you for quick review. I will make sure to add the bug URL in the future. Now I need help in committing this :-)
Comment 4 Simon Hausmann 2009-02-05 06:34:40 PST 
Landed in r40669