Fix time display by using scrubbing as a function

[nickygerritsen]

this.player_.scrubbing is a function and thus this call should have () added. Currently this will always return the cached current time

[heff]

Oh man, thanks for catching that.

I think we should be fixing that in remainingTime too, though it looks like we have more updating to do in that component. Would you mind fixing that comment either way?

video.js/src/js/control-bar/time-controls/remaining-time-display.js

Line 58 in 57152ae

// var time = (this.player_.scrubbing) ? this.player_.getCache().currentTime : this.player_.currentTime();

[nickygerritsen]

Found another spot where scrubbing was used without calling the function in comments, so modified that one too

[heff]

Awesome, thanks! Looks good to me.

@videojs/core-committers can one other person confirm and pull in?

Sent from mobile