CSS filter invert() with no argument uses the wrong value: 1 instead of 0
Reported by
woxxom@gmail.com,
Jan 3 2018
|
||||||
Issue descriptionChrome version: all versions since as early as v24 and up to v65 ================================= 1. open the attached test.html Expected: "SUCCESS" is shown Observed: empty page ================================= As per CSS specification, the initial value for invert() is 0 so "invert()" must be parsed as "invert(0)", but Chrome incorrectly parses it as "invert(1)". Quoting https://drafts.fxtf.org/filter-effects/#supported-filter-functions > omitted values default to the initial value for interpolation > invert() = invert( <number-percentage> ) > The initial value for interpolation is 0. Quoting https://www.w3.org/TR/filter-effects-1/#FilterProperty > invert() = invert( [ <number> | <percentage> ] ) > The lacuna value for interpolation is 0. ================================= Firefox correctly shows "SUCCESS" but only because it refuses to parse the empty "invert()", but at least the end result is correct.
,
Jan 4 2018
Able to reproduce this issue on latest stable 63.0.3239.108 and on latest canary 65.0.3310.0 using Windows 10, Mac 10.13.1 and Ubuntu 14.04 with HTML file given in comment#0. i.e; Blank page is seen instead of SUCCESS. Issue is seen from M50.[50.0.2661.0]. Hence considering this issue as Non-Regression and marking as Untriaged. Issue is not seen in Firefox. Thanks!
,
Jan 4 2018
,
Jan 4 2018
,
Jan 8 2018
,
Jan 8 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/68455a18e94e8ea744ea9ff248b2d645e853b53d commit 68455a18e94e8ea744ea9ff248b2d645e853b53d Author: Shanmuga Pandi M <shanmuga.m@samsung.com> Date: Mon Jan 08 11:34:55 2018 Set CSS filter invert() default argument as 0. As per spec, the initial value for invert() is 0 so "invert()" must be parsed as "invert(0)" spec: https://drafts.fxtf.org/filter-effects/#funcdef-filter-invert https://www.w3.org/TR/filter-effects-1/#funcdef-invert Bug: 798683 Change-Id: Ia0086db1d3079d7a8fffd9c88d5e1a19dcbd83de Reviewed-on: https://chromium-review.googlesource.com/853798 Reviewed-by: nainar <nainar@chromium.org> Commit-Queue: Shanmuga Pandi <shanmuga.m@samsung.com> Cr-Commit-Position: refs/heads/master@{#527608} [modify] https://crrev.com/68455a18e94e8ea744ea9ff248b2d645e853b53d/third_party/WebKit/LayoutTests/css3/filters/filter-property-computed-style.html [modify] https://crrev.com/68455a18e94e8ea744ea9ff248b2d645e853b53d/third_party/WebKit/LayoutTests/css3/filters/nested-filters.html [modify] https://crrev.com/68455a18e94e8ea744ea9ff248b2d645e853b53d/third_party/WebKit/LayoutTests/css3/filters/unprefixed.html [modify] https://crrev.com/68455a18e94e8ea744ea9ff248b2d645e853b53d/third_party/WebKit/Source/core/css/resolver/StyleBuilderConverter.cpp
,
Jan 8 2018
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by krajshree@chromium.org
, Jan 3 2018