1029636 - [reftest::B2G] TEST-UNEXPECTED-FAIL outer-svg-border-and-padding-01.svg

Status: RESOLVED WONTFIX

(Testing :: Reftest, defect)

Description

[u459114]

https://tbpl.mozilla.org/?rev=f90e81750943&tree=Mozilla-Inbound
bug 1000722 comment 72, the fixed provided in bug 1000722 change this test case from 100% fail to random fail.

File this bug to keep it in trace,

Comment 1

[u459114]

Take it

Comment 2

[u459114]

f90e81750943 Jerry Shih – Bug 998916 - check the visibility change event to defer the webgl context restore. r=jgilbert,smaug

Check whether this random fail caused by this patch.

Comment 3

[u459114]

Details
https://tbpl.mozilla.org/?tree=Mozilla-Inbound&jobname=reftest-18&fromchange=6dbb4388563b&tochange=7b174d47f3cc

Comment 4

[u459114]

Hi emorley,
To fix this bug, I may need to 
1. add some log in rendering path, remove random-if and,
2. associate this fail with an Intermittent bug to receive TPBL fail report. For example, Bug 1019131

Can I do #2 by myself, or I need ask help from sheriff?

Comment 5

[Ed Morley [:emorley]]

To do #2, you just need to 'intermittent-failure' keyword on the bug, so TBPL sees it. However the failure rate in bug 1000722 comment 72 is too high to mark this test as passing and just star it on TBPL as intermittent? Not quite sure what you are asking? :-)

Comment 6

[u459114]

Thanks, Morley, that's all I need!

Comment 7

[u459114]

Content script:
1. MozReftestInvalidate fired
2. Element::setAttribute which generate new pending paint event in nsRefreshDriver of "content" process
3. reftest-wait removed
4. RecordResult fired
5. sendAsyncMessage("reftest:TestDone"..

Chrome Script:
6. RecvTestDone
7. RecordResult
8. DrawWindow - flush all pending event in "chrome" process.

If pending event generated in #2 is not executed on time and update layer buffer before 8, test case fail.

Next step should be change tick rate in nsRefreshDriver from 60 FPS to 1 FPS, if fail rate go up because of this change, root cause confirm

Comment 8

[u459114]

Or do content flush before #3

Comment 9

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

This is WONTFIX (i.e. we're not going to invest in enabling/fixing reftests on B2G), now that we're removing B2G code from the tree via bug 1306391. The reftest annotation got removed in bug 1307332.