Closed Bug 743817 Opened 12 years ago Closed 12 years ago

make font inflation reftests live in reftest harness rather than mochitest

Categories

(Core :: Layout, defect, P4)

Product:
Core
Component:
Layout
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla14

People

(Reporter: dbaron, Assigned: dbaron)

References

Details

Attachments

(1 file)

patch
41.45 KB, patch
jwir3
: review+
Details | Diff | Splinter Review 
To make testing bug 706193 easier, I want to convert the font inflation reftests to use the reftest harness rather than the mochitest harness.  Bug 743105 allows doing this.

This makes it easy to make the existing tests, written on the assumption of not having bug 706193, run with that fix disabled, but to make new tests that test that fix and run with it enabled.

Note also that I ran into bug 743805 while doing this, and had to modify a few of the tests (for which we must have been cutting off the scrollbars when running in the mochitest harness, perhaps?).
Also noticed bug 706889 comment 24 while checking that I did this correctly.
Attached patch patch — — Splinter Review 
Assignee: nobody → dbaron
Status: NEW → ASSIGNED

        Attachment #613462 -
        Flags: review?(sjohnson)

        Attachment #613462 -
        Flags: review?(sjohnson) → review+

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/ce4cfbc091dc
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED

    
  
Depends on: 780581

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: