diff options
author | smain@google.com <smain@google.com> | 2015-02-05 13:27:16 -0800 |
---|---|---|
committer | smain@google.com <smain@google.com> | 2015-02-05 17:49:49 -0800 |
commit | d162be5b5f65ba0311374242d48303f120398d48 (patch) | |
tree | 18b2463b1feaadf3f972da9ae34d6f6b670a7466 /tools/droiddoc/templates-sdk | |
parent | 529a107f9de5051b734fad4cecf1f971998aeade (diff) | |
download | build-d162be5b5f65ba0311374242d48303f120398d48.zip build-d162be5b5f65ba0311374242d48303f120398d48.tar.gz build-d162be5b5f65ba0311374242d48303f120398d48.tar.bz2 |
change video analytics to distinguish between start and resume events.
also scrub out any additional parameters from the videoId before passing to analytics
Change-Id: Idf3351028e62d0d03c0f680a904bb83ef8353797
Diffstat (limited to 'tools/droiddoc/templates-sdk')
-rw-r--r-- | tools/droiddoc/templates-sdk/assets/js/docs.js | 30 |
1 files changed, 21 insertions, 9 deletions
diff --git a/tools/droiddoc/templates-sdk/assets/js/docs.js b/tools/droiddoc/templates-sdk/assets/js/docs.js index b0ee279..7f4be4e 100644 --- a/tools/droiddoc/templates-sdk/assets/js/docs.js +++ b/tools/droiddoc/templates-sdk/assets/js/docs.js @@ -603,9 +603,12 @@ function getVideoHeight() { return frameHeight - (marginTop * 2); } +var mPlayerPaused = false; + function startYouTubePlayer(videoId) { $("#video-container").show(); $("#video-frame").show(); + mPlayerPaused = false; // compute the size of the player so it's centered in window var maxWidth = 940; // the width of the web site content @@ -645,7 +648,7 @@ function startYouTubePlayer(videoId) { // reset the size in case the user adjusted the window since last play youTubePlayer.setSize(videoWidth, videoHeight); // if a video different from the one already playing was requested, cue it up - if (videoId != youTubePlayer.getVideoUrl().split('?v=')[1]) { + if (videoId != youTubePlayer.getVideoUrl().split('?v=')[1].split('&')[0].split('%')[0]) { youTubePlayer.cueVideoById(videoId); } youTubePlayer.playVideo(); @@ -654,16 +657,13 @@ function startYouTubePlayer(videoId) { function onPlayerReady(event) { event.target.playVideo(); - // track the start playing event so we know from which page the video was selected - ga('send', 'event', 'Videos', 'Start: ' + - youTubePlayer.getVideoUrl().split('?v=')[1], 'on: ' + document.location.href); + mPlayerPaused = false; } function closeVideo() { try { youTubePlayer.pauseVideo(); } catch(e) { - console.log('Video not available'); } $("#video-container").fadeOut(200); } @@ -672,18 +672,30 @@ function closeVideo() { function onPlayerStateChange(event) { // Video starts, send the video ID if (event.data == YT.PlayerState.PLAYING) { - ga('send', 'event', 'Videos', 'Play', - youTubePlayer.getVideoUrl().split('?v=')[1]); + if (mPlayerPaused) { + ga('send', 'event', 'Videos', 'Resume', + youTubePlayer.getVideoUrl().split('?v=')[1].split('&')[0].split('%')[0]); + } else { + // track the start playing event so we know from which page the video was selected + ga('send', 'event', 'Videos', 'Start: ' + + youTubePlayer.getVideoUrl().split('?v=')[1].split('&')[0].split('%')[0], + 'on: ' + document.location.href); + } + mPlayerPaused = false; } // Video paused, send video ID and video elapsed time if (event.data == YT.PlayerState.PAUSED) { ga('send', 'event', 'Videos', 'Paused', - youTubePlayer.getVideoUrl().split('?v=')[1], youTubePlayer.getCurrentTime()); + youTubePlayer.getVideoUrl().split('?v=')[1].split('&')[0].split('%')[0], + youTubePlayer.getCurrentTime()); + mPlayerPaused = true; } // Video finished, send video ID and video elapsed time if (event.data == YT.PlayerState.ENDED) { ga('send', 'event', 'Videos', 'Finished', - youTubePlayer.getVideoUrl().split('?v=')[1], youTubePlayer.getCurrentTime()); + youTubePlayer.getVideoUrl().split('?v=')[1].split('&')[0].split('%')[0], + youTubePlayer.getCurrentTime()); + mPlayerPaused = true; } } |