WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
77291
Add back WTFURL to WebKit
https://bugs.webkit.org/show_bug.cgi?id=77291
Summary
Add back WTFURL to WebKit
Benjamin Poulain
Reported
2012-01-28 16:29:59 PST
Add back what was removed by
r86787
.
Attachments
Patch
(92.16 KB, patch)
2012-01-28 16:40 PST
,
Benjamin Poulain
abarth
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Benjamin Poulain
Comment 1
2012-01-28 16:40:26 PST
Created
attachment 124457
[details]
Patch
Adam Barth
Comment 2
2012-01-28 16:55:52 PST
Comment on
attachment 124457
[details]
Patch There's still an ongoing thread on webkit-dev about this work, but I think it's fine to proceed as long as you're aware of the issues being discussed there.
Benjamin Poulain
Comment 3
2012-01-28 17:40:07 PST
(In reply to
comment #2
)
> (From update of
attachment 124457
[details]
) > There's still an ongoing thread on webkit-dev about this work, but I think it's fine to proceed as long as you're aware of the issues being discussed there.
Yep, I'll wait until there is some clear path forward before landing this.
Benjamin Poulain
Comment 4
2012-02-11 19:24:50 PST
Are you OK if I land this? I think we have an idea of where to go from here: -no dependency on anything else in WTF -at some point WTFURL will have to be compiled as a static library for Chromium Next I think I am gonna export the headers for WebCore and make an empty skeleton of KURL based on ParsedURL + a WTFString for invalid URL.
Adam Barth
Comment 5
2012-02-11 19:30:59 PST
That sounds like a good plan. I'm not sure we have 100% buy-in from all the stakeholders, but if you're ok proceeding knowing that risk is out there, I'm happy to review your patches.
Benjamin Poulain
Comment 6
2012-02-11 22:32:49 PST
Committed
r107502
: <
http://trac.webkit.org/changeset/107502
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug