Bug 46597 - [Chromium] fast/dom/anchor-origin.html failing since r68354
Summary: [Chromium] fast/dom/anchor-origin.html failing since r68354
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: James Kozianski
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-26 18:37 PDT by James Kozianski
Modified: 2011-01-18 22:03 PST (History)
5 users (show)

See Also:


Attachments
Patch (2.94 KB, patch)
2010-09-26 21:55 PDT, James Kozianski
no flags Details | Formatted Diff | Diff
Patch (1.45 KB, patch)
2010-09-26 22:06 PDT, James Kozianski
no flags Details | Formatted Diff | Diff
Patch (2.11 KB, patch)
2010-09-26 22:54 PDT, James Kozianski
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description James Kozianski 2010-09-26 18:37:57 PDT
Seems to be failing because of URL parsing differences.

Here's a snippet from the diff:

 http://example.com/foo/bar => http://example.com
-HTTP://example.com/foo/bar => http://example.com
+http://example.com/foo/bar => http://example.com
 https://example.com/ttt?ggg => https://example.com
 ftp://example.com/ttt?ggg => ftp://example.com
 file:///home/abarth => file://
-data:text/html,%3Cb%3Efoo%3C/b%3E => null
+data:text/html,foo => null
Comment 1 Adam Barth 2010-09-26 18:48:59 PDT
Those results are fine.  We just need a Chromium baseline.
Comment 2 James Kozianski 2010-09-26 21:55:56 PDT
Created attachment 68879 [details]
Patch
Comment 3 Adam Barth 2010-09-26 22:01:03 PDT
Comment on attachment 68879 [details]
Patch

These results are all the same.  Then can go in platform/chromium, no?
Comment 4 James Kozianski 2010-09-26 22:06:37 PDT
Created attachment 68880 [details]
Patch
Comment 5 James Kozianski 2010-09-26 22:08:02 PDT
(In reply to comment #3)
> (From update of attachment 68879 [details])
> These results are all the same.  Then can go in platform/chromium, no?

Yes, you're right. Sorry - I trusted the rebaseline-chromium-webkit-tests script output.
Comment 6 Ojan Vafai 2010-09-26 22:29:34 PDT
Comment on attachment 68880 [details]
Patch

Removing cq+. James is modifying his patch to remove the line from test_expectations.txt.
Comment 7 James Kozianski 2010-09-26 22:54:59 PDT
Created attachment 68881 [details]
Patch
Comment 8 WebKit Commit Bot 2010-09-27 01:25:44 PDT
Comment on attachment 68881 [details]
Patch

Clearing flags on attachment: 68881

Committed r68376: <http://trac.webkit.org/changeset/68376>
Comment 9 WebKit Commit Bot 2010-09-27 01:25:51 PDT
All reviewed patches have been landed.  Closing bug.