Skip to content
  • jer.noble@apple.com's avatar
    REGRESSION(r156546): media/video-no-audio.html broken · 76755699
    jer.noble@apple.com authored
    https://bugs.webkit.org/show_bug.cgi?id=122049
    
    Reviewed by Eric Carlson.
    
    Source/WebCore:
    
    Now that our video elements have audio tracks, use their presence or
    absense to show or hide the volume controls.
    
    * Modules/mediacontrols/mediaControlsApple.js:
    (Controller):
    (Controller.prototype.addVideoListeners): Listen for audioTrack changes.
    (Controller.prototype.removeVideoListeners): Stop listening to same.
    (Controller.prototype.handleAudioTrackChange): Call this.updateHasAudio()
    (Controller.prototype.handleAudioTrackAdd): Ditto.
    (Controller.prototype.handleAudioTrackRemove): Ditto.
    (Controller.prototype.updateHasAudio): Hide the volume controls if no
        audioTracks are present, and vice versa.
    
    LayoutTests:
    
    * platform/mac/TestExpectations:
    * platform/mac/media/video-no-audio-expected.txt:
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@156656 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    76755699