Bug 85724

Summary:

Implement the simple constructors WTFURL's KURL

Product:

WebKit

Reporter:

Benjamin Poulain <benjamin>

Component:

Web Template Framework

Assignee:

Benjamin Poulain <benjamin>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none
Patch	none
Patch	none
Patch	abarth: review+

Description Benjamin Poulain 2012-05-05 17:58:45 PDT

Next step: be able to build KURL based on NSUrl and relative URL parsed from page.

Comment 1 Benjamin Poulain 2012-05-05 18:33:02 PDT

Created attachment 140416 [details]
Patch

Comment 2 Benjamin Poulain 2012-05-05 18:33:51 PDT

Work in progress, I still need to add parsing of relative URLs.

Comment 3 Benjamin Poulain 2012-05-19 19:00:52 PDT

Created attachment 142888 [details]
Patch

Comment 4 Benjamin Poulain 2012-06-10 15:02:10 PDT

Created attachment 146750 [details]
Patch

Comment 5 Benjamin Poulain 2012-06-10 15:17:53 PDT

Unfortunately, even the simplest URL test require relative URL to work. So although I can now run the tests, none of them succeed.

Next I'll have to fix parsing of relative URL.

Comment 6 Build Bot 2012-06-10 15:18:06 PDT

Comment on attachment 146750 [details]
Patch

Attachment 146750 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/12938237

Comment 7 Benjamin Poulain 2012-06-10 15:25:08 PDT

Created attachment 146753 [details]
Patch

Comment 8 Adam Barth 2012-06-11 01:07:29 PDT

Yeah, adding relative URLs is probably high on the TODO list.

Comment 9 Benjamin Poulain 2012-06-13 14:46:20 PDT

Committed r120250: <http://trac.webkit.org/changeset/120250>