Closed Bug 714496 Opened 13 years ago Closed 12 years ago

Intermittent TEST-UNEXPECTED-PASS test_smilCSSInherit.xhtml | deeply-inherited font-size halfway through animation - 75px should equal 75px

Categories

(Core :: SVG, defect)

Product:
Core
Component:
SVG
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla12

People

(Reporter: philor, Assigned: dholbert)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

disable the test
1.54 KB, patch
jwatt
: review+
Details | Diff | Splinter Review 
According to http://mxr.mozilla.org/mozilla-central/source/content/smil/test/test_smilCSSInherit.xhtml#75 there's a 1/11! chance of hitting this, and I'd like my lottery winnings, please:

https://tbpl.mozilla.org/php/getParsedLog.php?id=8260997&tree=Firefox
Rev3 Fedora 12x64 mozilla-central opt test mochitests-1/5 on 2011-12-31 21:39:42 PST for push d77b056ed4bd

86412 ERROR TEST-UNEXPECTED-PASS | /tests/content/smil/test/test_smilCSSInherit.xhtml | deeply-inherited font-size halfway through animation - 75px should equal 75px
(In reply to Phil Ringnalda (:philor) from comment #0)
> According to
> http://mxr.mozilla.org/mozilla-central/source/content/smil/test/
> test_smilCSSInherit.xhtml#75 there's a 1/11! chance of hitting this, and I'd
> like my lottery winnings, please:
> 

Sure, just send the 50$ release fee to the following address...
Might be time to recalculate those odds, you might have partly fixed it :)
Weird!

We can always add a few more levels of nesting to the test - that should make it harder to accidentally pass.
Summary: Intermittent (1/39,916,800 intermittent) TEST-UNEXPECTED-PASS test_smilCSSInherit.xhtml | deeply-inherited font-size halfway through animation - 75px should equal 75px → Intermittent TEST-UNEXPECTED-PASS test_smilCSSInherit.xhtml | deeply-inherited font-size halfway through animation - 75px should equal 75px
This test can just be disabled.

It's really a regression-test for the yet-to-be-fixed bug 501183.  While its outcome is currently random, I'd set it up in such a way that it was vastly more likely to fail than to pass, so we could mark it "failing" and not cause too much randomorange.  Something apparently changed to tip those odds, and while it'd be interesting to know what that was, it's not particularly important.

Patch coming up to disable it. (sorry for not having posted one sooner.)
Attached patch disable the test — — Splinter Review 
Assignee: nobody → dholbert
Status: NEW → ASSIGNED

        Attachment #592430 -
        Flags: review?

    
  

        Attachment #592430 -
        Flags: review? → review?(longsonr)

        Attachment #592430 -
        Flags: review?(longsonr) → review+

    
  
OS: Mac OS X → All
Hardware: x86 → All

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/b0fd43a506f3
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
Whiteboard: [orange]

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: