fix: nullify srcObject instead of calling removeAttribute (#1145)

* fix: nullify srcObject instead of calling removeAttribute

* Update src/players/FilePlayer.js

Co-authored-by: Pete Cook <[email protected]>

Co-authored-by: Pete Cook <[email protected]>

Author: komachi

src/players/FilePlayer.js

@@ -38,7 +38,7 @@ export default class FilePlayer extends Component {
       this.props.url !== prevProps.url &&
       !isMediaStream(this.props.url)
     ) {
-      this.player.removeAttribute('srcObject')
+      this.player.srcObject = null
     }
   }
 

test/players/FilePlayer.js

@@ -398,8 +398,7 @@ test('clear srcObject on url change', t => {
   const url = new MockMediaStream()
   const wrapper = shallow(<FilePlayer url={url} config={config} />)
   const instance = wrapper.instance()
-  instance.player.removeAttribute = sinon.fake()
   instance.load(url)
   wrapper.setProps({ url: 'file.mpv' })
-  t.true(instance.player.removeAttribute.calledOnceWith('srcObject'))
+  t.is(instance.player.srcObject, null)
 })