The spec asks us to throw an error if the URL is invalid (it does not parse as a URL). Spec: https://drafts.css-houdini.org/css-typed-om-1/#dom-cssurlimagevalue-cssurlimagevalue See test failure: https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/typedcssom/stylevalue-subclasses/cssUrlImageValue-expected.txt
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b3f24f517983d42b35d3d37739aa474df1fb51b2 commit b3f24f517983d42b35d3d37739aa474df1fb51b2 Author: Shanmuga Pandi M <shanmuga.m@samsung.com> Date: Tue Dec 05 01:26:52 2017 CSSURLImageValue should validate URLs As per the spec[1], Constructing CSSURLImageValue should throw an error, if the URL is invalid. [1] https://drafts.css-houdini.org/css-typed-om-1/#dom-cssurlimagevalue-cssurlimagevalue Bug: 791434 Change-Id: I06ae5dad378ca91ce827f64ff56498a06c9963af Reviewed-on: https://chromium-review.googlesource.com/805635 Reviewed-by: Darren Shen <shend@chromium.org> Commit-Queue: Shanmuga Pandi <shanmuga.m@samsung.com> Cr-Commit-Position: refs/heads/master@{#521574} [modify] https://crrev.com/b3f24f517983d42b35d3d37739aa474df1fb51b2/third_party/WebKit/LayoutTests/typedcssom/stylevalue-subclasses/cssUrlImageValue-expected.txt [modify] https://crrev.com/b3f24f517983d42b35d3d37739aa474df1fb51b2/third_party/WebKit/LayoutTests/typedcssom/stylevalue-subclasses/cssUrlImageValue.html [modify] https://crrev.com/b3f24f517983d42b35d3d37739aa474df1fb51b2/third_party/WebKit/Source/core/css/cssom/CSSURLImageValue.h [modify] https://crrev.com/b3f24f517983d42b35d3d37739aa474df1fb51b2/third_party/WebKit/Source/core/css/cssom/CSSURLImageValue.idl
Comment 1 by shanmug...@samsung.com
, Dec 4 2017Status: Assigned (was: Available)