698967 - test_transitions_per_property says "interpolation of lengths" in non-length-testing functions

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Daniel Holbert [:dholbert]]

While adding to test_transitions_per_property.html, I noticed that it's got "interpolation of lengths" as part of the subtest-description for some non-length properties.

In particular, the hits in test_zindex_transition() and test_pos_integer_or_auto_transition() are incorrect.  (those are testing interpolation of integer values, not lengths)

http://mxr.mozilla.org/mozilla-central/source/layout/style/test/test_transitions_per_property.html?force=1#852
http://mxr.mozilla.org/mozilla-central/source/layout/style/test/test_transitions_per_property.html?force=1#970

Comment 1

[Daniel Holbert [:dholbert]]

Attachment: trivial fix

Comment 2

[Daniel Holbert [:dholbert]]

These chunks are used for testing the properties "z-index" and "-moz-column-count", FWIW.

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 571212 [details] [diff] [review]
trivial fix

r=dbaron, though I'd actually have done "interpolation of z-index" and "interpolation of font-weight" since the functions are specific to those.  Feel free to do it that way.

Comment 4

[Daniel Holbert [:dholbert]]

(In reply to David Baron [:dbaron] from comment #3)
> Comment on attachment 571212 [details] [diff] [review] [diff] [details] [review]
> trivial fix
> 
> r=dbaron, though I'd actually have done "interpolation of z-index" and
> "interpolation of font-weight" since the functions are specific to those. 

I'm going to stick with "integers", if that's all right with you (sounds like it is), because:
 (1) while the first chunk (in "test_zindex_transition") is *currently* specific to z-index, it won't be for long.  I'm co-opting it to also test "flex-order" in the flex-order patch on bug 696253. (attachment 571221 [details] [diff] [review])

 (2) the second chunk ("test_pos_integer_or_auto_transition") is already used for multiple properties -- z-index and -moz-column-count, in particular.  (It's not used for font-weight -- that's only mentioned in the patch header-line because it's the function immediately *before* test_pos_integer_or_auto_transition)

Comment 5

[Daniel Holbert [:dholbert]]

(meant to say -- thanks for the review!)

Comment 6

[Daniel Holbert [:dholbert]]

Pushed:
  https://hg.mozilla.org/integration/mozilla-inbound/rev/ad09520aac4b

(I pushed as DONTBUILD since this is only tweaking warning-strings within a mochitest, which can't affect test-behavior at all.)

Comment 7

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/ad09520aac4b