[webkit-reviews] review requested: [Bug 38159] ChromiumDataObject::setURL allows empty URLs to be set : [Attachment 54357] Fix for ChromiumDataObject::setURL

[bugzilla-daemon at webkit.org]

Daniel Cheng <dcheng at chromium.org> has asked  for review:
Bug 38159: ChromiumDataObject::setURL allows empty URLs to be set
https://bugs.webkit.org/show_bug.cgi?id=38159

Attachment 54357: Fix for ChromiumDataObject::setURL
https://bugs.webkit.org/attachment.cgi?id=54357&action=review

------- Additional Comments from Daniel Cheng <dcheng at chromium.org>
This wouldn't be a problem except that it appends URLs to an internal list. The
fix should be to not append empty URLs; otherwise, asserts in getData will be
triggered.