[Webkit-unassigned] [Bug 46943] [chromium] getData('text/uri-list') should return the same thing that was passed to setData('text/uri-list')

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=46943


Tony Chang <tony at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #69763|review?                     |review-
               Flag|                            |




--- Comment #4 from Tony Chang <tony at chromium.org>  2010-10-05 09:39:52 PST ---
(From update of attachment 69763)
View in context: https://bugs.webkit.org/attachment.cgi?id=69763&action=review

> LayoutTests/editing/pasteboard/script-tests/dataTransfer-setData-getData.js:119
> +    test("text/uri-list", "# comment\r\n# comment 2\r\n# comment 3", 
> +         "URL", null);

The spec says this should return an empty string, not null:
http://dev.w3.org/html5/spec/dnd.html
"If there is no data with that format, or if there is but it has no URLs, then the method must return the empty string."

> WebCore/platform/chromium/ChromiumDataObject.cpp:145
>          success = !m_url.isEmpty();
>          return m_url.string();

Maybe the fix is as simple as
  success = !m_url.isEmpty() || !m_uriList.isEmpty();
?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.