Closed Bug 939612 Opened 11 years ago Closed 11 years ago

Improve readability of MediaSource::SetReadyState

Categories

(Core :: Audio/Video, defect)

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

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla28

People

(Reporter: kinetik, Assigned: kinetik)

References

(Blocks 1 open bug)

Details

(Whiteboard: [qa-])

Attachments

(1 file)

bug939612_v0.patch
(deleted), patch
cajbir
: review+
Details | Diff | Splinter Review 
Set mReadyState outside of the if tree and move the check of the new state before the old state to make the if tree layout clearer.
Attached patch bug939612_v0.patch (deleted) — — Splinter Review 

        Attachment #8333599 -
        Flags: review?(chris.double)

    
  

        Attachment #8333599 -
        Flags: review?(chris.double) → review+

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/b65f7a6ecc35
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28

    
  
Whiteboard: [qa-]

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: