Summary: | KURL creates duplicate strings when completing data: URIs. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andreas Kling <kling> | ||||
Component: | Page Loading | Assignee: | Andreas Kling <kling> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, kling, roger_fong, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar, Performance | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Andreas Kling
2013-07-21 09:02:52 PDT
Created attachment 207209 [details]
Patch
Comment on attachment 207209 [details] Patch Clearing flags on attachment: 207209 Committed r152951: <http://trac.webkit.org/changeset/152951> All reviewed patches have been landed. Closing bug. I think that this may have caused 4 tests to start failing. In particular, http://build-safari.apple.com/results/Cab%20Debug%20Tests/r153057_53185%20(21903)/results.html My guess is just based on looking at regression ranges. There also is a test or two in that set of 4 that involves URL's. Can you take a look? Thanks (In reply to comment #5) > I think that this may have caused 4 tests to start failing. > In particular, > http://build-safari.apple.com/results/Cab%20Debug%20Tests/r153057_53185%20(21903)/results.html > > My guess is just based on looking at regression ranges. > There also is a test or two in that set of 4 that involves URL's. > > Can you take a look? Thanks I can repro the failures locally, even with that change reverted :/ (In reply to comment #6) > (In reply to comment #5) > > I think that this may have caused 4 tests to start failing. > > In particular, > > http://build-safari.apple.com/results/Cab%20Debug%20Tests/r153057_53185%20(21903)/results.html > > > > My guess is just based on looking at regression ranges. > > There also is a test or two in that set of 4 that involves URL's. > > > > Can you take a look? Thanks > > I can repro the failures locally, even with that change reverted :/ Humm, nvm then, maybe its one of phil's jsc changes which were also in that area. I'll try reverting dsome of those. thx! |