Closed
Bug 1502955
Opened 6 years ago
Closed 6 years ago
[wpt-sync] Sync PR 13763 - [css-properties-values-api] Validate generate_property argument.
Categories
(Core :: CSS Parsing and Computation, enhancement, P4)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla65
Tracking | Status | |
---|---|---|
firefox65 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 13763 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/web-platform-tests/wpt/pull/13763 Details from upstream follow. Anders Hartvoll Ruud <andruud@chromium.org> wrote: > [css-properties-values-api] Validate generate_property argument. > > This prevents silently dropping invalid arguments. For example, the > following currently generates a non-inherited property: > > let name = generate_property(syntax: '\<length>', inherited: true); > > This is because 'inherited' is not an option for CSS.registerProperty (the > actual option is 'inherits'). > > With this change, an exception is thrown if unknown options are provided. > > R=futhark@chromium.org > > Bug: 641877 > Change-Id: I7b639651c331aa705416db95a3de751353648031 > Reviewed-on: https://chromium-review.googlesource.com/1304522 > WPT-Export-Revision: 872d26676c59d1bc8ab871a340922e2c993754ab
Assignee | ||
Updated•6 years ago
|
Component: web-platform-tests → CSS Parsing and Computation
Product: Testing → Core
Assignee | ||
Comment 1•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=f234888b48e66e0f802f4a08b6e4f67b1e687a49
Assignee | ||
Comment 2•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=b3cb67a84f24f3641b3f9e0f6c33396be13f893b
Assignee | ||
Comment 3•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=6fa1d9c0d10762cfdc304c620538c033c4996cdf
Assignee | ||
Updated•6 years ago
|
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Assignee | ||
Updated•6 years ago
|
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Assignee | ||
Comment 4•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=46ade8992f3b737292d47c18c9f21dce03b2cd44
Assignee | ||
Comment 5•6 years ago
|
||
Ran 2 tests and 17 subtests OK : 2 PASS : 1 FAIL : 16 New tests that have failures or other problems: /css/css-properties-values-api/registered-property-computation.html <length-percentage> values are computed correctly for divWithFontSizeInherited: FAIL <length-percentage> values are computed correctly for divWithFontSizeSet: FAIL <length-percentage># values are computed correctly for divWithFontSizeInherited: FAIL <length-percentage># values are computed correctly for divWithFontSizeSet: FAIL <length-percentage>+ values are computed correctly for divWithFontSizeInherited: FAIL <length-percentage>+ values are computed correctly for divWithFontSizeSet: FAIL <length> values are computed correctly for divWithFontSizeInherited: FAIL <length> values are computed correctly for divWithFontSizeSet: FAIL <length># values are computed correctly for divWithFontSizeInherited: FAIL <length># values are computed correctly for divWithFontSizeSet: FAIL <length>+ values are computed correctly for divWithFontSizeInherited: FAIL <length>+ values are computed correctly for divWithFontSizeSet: FAIL <transform-function> values are computed correctly for divWithFontSizeInherited: FAIL <transform-function> values are computed correctly for divWithFontSizeSet: FAIL /css/css-properties-values-api/self-utils.html Default initial values of generated properties are valid (self-test).: FAIL Generated properties respect inherits flag: FAIL
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/60774ca43509 [wpt PR 13763] - [css-properties-values-api] Validate generate_property argument., a=testonly
Comment 7•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/60774ca43509
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox65:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in
before you can comment on or make changes to this bug.
Description
•