Bug 1478472 Comment 19 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

(In reply to Alastor Wu [:alwu] from comment #18)
> I've added the `fuzzy` keyword on some wpts, but it seems not work at all [1]. Is there any thing I miss?

One guess: it looks like in your fuzzy annotations, you're repeating the test name, like here in your try commit:
> --- /dev/null
> +++ b/testing/web-platform/meta/webvtt/rendering/cues-with-video/processing-model/align_center.html.ini
> @@ -0,0 +1,2 @@
> +[align_center.html]
> +  fuzzy: align_center.html:maxDifference=2;totalPixels=5000

I think you're supposed to use the **reference name** in the fuzzy annotation.  (The test name is already present in the brackets, so it'd be redundant to list it again.)

I think (?) the idea is that tests can have multiple references (though they don't often), so to handle that correctly, you have to clarify which reference you're referring to in the fuzzy annotation. Or something like that.  You don't have to clarify which testcase you're referring to, because the testcase has already been mentioned.
(In reply to Alastor Wu [:alwu] from comment #18)
> I've added the `fuzzy` keyword on some wpts, but it seems not work at all [1]. Is there any thing I miss?

One guess: it looks like in your fuzzy annotations, you're repeating the test name, like here in your try commit:
```
 --- /dev/null
 +++ b/testing/web-platform/meta/webvtt/rendering/cues-with-video/processing-model/align_center.html.ini
 @@ -0,0 +1,2 @@
 +[align_center.html]
 +  fuzzy: align_center.html:maxDifference=2;totalPixels=5000
           ^^^^^^^^^^^^^^^^^----(I think this is the problem)
```

I think you're supposed to use the **reference name** in the fuzzy annotation.  (The test name is already present in the brackets, so it'd be redundant to list it again.)

I think (?) the idea is that tests can have multiple references (though they don't often), so to handle that correctly, you have to clarify which reference you're referring to in the fuzzy annotation. Or something like that.  You don't have to clarify which testcase you're referring to, because the testcase has already been mentioned.

Back to Bug 1478472 Comment 19