dont call change handlers if player is not defined

Author: neuling

examples/simpleExample.js

@@ -9,7 +9,6 @@ export default class SimpleExample extends React.Component {
           videoId="bQQVybyIk5E"
           width="560"
           height="315"
-          onVolumeChange={(v) => console.log(v)}
         ></YouTubeVideo>
       </div>
     )

src/YouTubeVideo.js

@@ -31,7 +31,7 @@ export default class YouTubeVideo extends Component {
     return this.player.getCurrentTime() * 1000;
   }
 
-  startVideoStateObserver = (t) => {
+  callChangeHandlers() {
     const currentPosition = this.getCurrentTimeInMs();
     const currentVolume = this.player.isMuted() ? 0 : this.player.getVolume();
     if (this.lastSentVolume != currentVolume) {
@@ -42,6 +42,10 @@ export default class YouTubeVideo extends Component {
       this.lastSentPosition = currentPosition;
       this.props.onProgress(currentPosition);
     }
+  }
+
+  startVideoStateObserver = (t) => {
+    if (this.player) this.callChangeHandlers();
     window.requestAnimationFrame(this.startVideoStateObserver);
   }
 
@@ -152,7 +156,7 @@ export default class YouTubeVideo extends Component {
       }
     }
 
-    if (!isNaN(this.props.volume) && prevProps.volume !== this.props.volume) {
+    if (prevProps.volume !== this.props.volume) {
       this.setPlayerVolume(this.props.volume);
     }