Bug 1526847 Comment 13 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 Boris Chiou [:boris] from comment #12)

> (In reply to Hiroyuki Ikezoe (:hiro) from comment #11)
> 
> > I think the test case would be a reftest which has a scaling animation box containing some text.  We need to add reftest-no-flush.  The reftest might not be failed without the fix though.
> 
> Just tried a scaling animation box containing some text, but unfortunately, this may not reflect the change I did because the reftest was not failed without the fix. Maybe need a trickier way. ;(

And I found a gfx? issue. If we use `reftest-no-flush`, the edges of the animation box look blur when taking the screenshot. This happens on both `transform: scale(..)` and `scale: ..`. And this blur makes the result of this test intermittent.
(In reply to Boris Chiou [:boris] from comment #12)

> (In reply to Hiroyuki Ikezoe (:hiro) from comment #11)
> 
> > I think the test case would be a reftest which has a scaling animation box containing some text.  We need to add reftest-no-flush.  The reftest might not be failed without the fix though.
> 
> Just tried a scaling animation box containing some text, but unfortunately, this may not reflect the change I did because the reftest was not failed without the fix. Maybe need a trickier way. ;(

And I found a gfx? issue. If we use `reftest-no-flush`, the edges of the animation box look blurred when taking the screenshot. This happens on both `transform: scale(..)` and `scale: ..`. And this blurred edge makes the result of this test intermittent.
(In reply to Boris Chiou [:boris] from comment #12)

> (In reply to Hiroyuki Ikezoe (:hiro) from comment #11)
> 
> > I think the test case would be a reftest which has a scaling animation box containing some text.  We need to add reftest-no-flush.  The reftest might not be failed without the fix though.
> 
> Just tried a scaling animation box containing some text, but unfortunately, this may not reflect the change I did because the reftest was not failed without the fix. Maybe need a trickier way. ;(

And I found a gfx? issue. If we use `reftest-no-flush`, the edges of the animation box look blurred when taking the screenshot. This happens on both `transform: scale(..)` and `scale: ..`. And this blur makes the result of this test intermittent.
(In reply to Boris Chiou [:boris] from comment #12)

> (In reply to Hiroyuki Ikezoe (:hiro) from comment #11)
> 
> > I think the test case would be a reftest which has a scaling animation box containing some text.  We need to add reftest-no-flush.  The reftest might not be failed without the fix though.
> 
> Just tried a scaling animation box containing some text, but unfortunately, this may not reflect the change I did because the reftest was not failed without the fix. Maybe need a trickier way. ;(

And I found a gfx fuzzy issue. If we use `reftest-no-flush`, the edges of the animation box look blurred when taking the screenshot (AA issue, I guess). This happens on both `transform: scale(..)` and `scale: ..`. And this blur makes the result of this test intermittent.
(In reply to Boris Chiou [:boris] from comment #12)

> (In reply to Hiroyuki Ikezoe (:hiro) from comment #11)
> 
> > I think the test case would be a reftest which has a scaling animation box containing some text.  We need to add reftest-no-flush.  The reftest might not be failed without the fix though.
> 
> Just tried a scaling animation box containing some text, but unfortunately, this may not reflect the change I did because the reftest was not failed without the fix. Maybe need a trickier way. ;(

And I found a gfx issue. If we use `reftest-no-flush`, the edges of the animation box look blurred when taking the screenshot (AA issue, I guess). This happens on both `transform: scale(..)` and `scale: ..`. And this blur makes the result of this test intermittent.
(In reply to Boris Chiou [:boris] from comment #12)

> (In reply to Hiroyuki Ikezoe (:hiro) from comment #11)
> 
> > I think the test case would be a reftest which has a scaling animation box containing some text.  We need to add reftest-no-flush.  The reftest might not be failed without the fix though.
> 
> Just tried a scaling animation box containing some text, but unfortunately, this may not reflect the change I did because the reftest was not failed without the fix. Maybe need a trickier way. ;(

And I found a gfx issue. If we use `reftest-no-flush`, the edges of the animation box look blurred (AA issue, I guess) when taking the screenshot. This happens on both `transform: scale(..)` and `scale: ..`. And this blur makes the result of this test intermittent.

Back to Bug 1526847 Comment 13