Closed Bug 566498 Opened 14 years ago Closed 14 years ago

Move generic video duration computing code into Ogg decoder

Categories

(Core :: Audio/Video, defect)

Product:
Core
Component:
Audio/Video
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: cajbir, Assigned: cajbir)

References

Details

Attachments

(1 file)

fix
8.42 KB, patch
cpearce
: review+
Details | Diff | Splinter Review 
As part of the refactoring of the Ogg decoder I moved the code that seeks to the end to find the duration into the generic base class. This isn't needed for all backends (eg. Wave) and should be moved back into the Ogg backend.
Attached patch fix — — Splinter Review 

        Attachment #445872 -
        Flags: review?(chris)

    
    

    
  
Comment on attachment 445872 [details] [diff] [review]
fix

Looks good.

        Attachment #445872 -
        Flags: review?(chris) → review+

    
  
Keywords: checkin-needed

    
    

    
  
http://hg.mozilla.org/mozilla-central/rev/26052d858595
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: