have to agree, this makes more sense as a first check inside changeCurrentTime. The "set" is an implicit request to attempt a timechange, changeCurrentTime can then decide not to change it if it's the same as what it is right now. Although, that said, this check might not even be necessary; returning early means the seek events never fire. Technically correct but possibly not in the spirit of explicitly setting a currenttime