[webkit-reviews] review canceled: [Bug 28293] [Chromium] event.datatransfer.getdata("text/uri-list") treated the same as getdata("URL") : [Attachment 38541] patch: layout test for getData("text/uri-list")

[bugzilla-daemon at webkit.org]

Roland Steiner <rolandsteiner at google.com> has canceled Roland Steiner
<rolandsteiner at google.com>'s request for review:
Bug 28293: [Chromium] event.datatransfer.getdata("text/uri-list") treated the
same as getdata("URL")
https://bugs.webkit.org/show_bug.cgi?id=28293

Attachment 38541: patch: layout test for getData("text/uri-list")
https://bugs.webkit.org/attachment.cgi?id=38541&action=review

------- Additional Comments from Roland Steiner <rolandsteiner at google.com>
(cf. comment #7)

You're right - the reason why I put it into http/tests/security/clipboard was
that I adapted the other layout test there and wanted to re-use the same
resource files. However, those same file are present in editing/pasteboard as
well, so I moved the test there as suggested.

In doing this I also had to adapt the test, since for reasons unknown to me the
file URL returned in the new location was suddenly absolute rather than
relative. However, relying on one or the other probably isn't a good idea (and
won't work iwth absolute URLs anyway), so I changed the test to just check for
"file://" at the beginning and the correct file name at the end.