Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(210)

Side by Side Diff: chrome/browser/resources/file_manager/mediaplayer.html

Issue 8727028: Eliminate use of assignments to innerHTML in File Browser (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Deleted another redundant file Created 9 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 <!DOCTYPE HTML> 1 <!DOCTYPE HTML>
2 <html i18n-values="dir:textdirection;"> 2 <html i18n-values="dir:textdirection;">
3 <head> 3 <head>
4 <meta charset="utf-8"> 4 <meta charset="utf-8">
5 <title>Media Player</title> 5 <title>Media Player</title>
6 <style type="text/css"> 6 <style type="text/css">
7 7
8 body { 8 body {
9 overflow: hidden; 9 overflow: hidden;
10 background: black; 10 background: black;
(...skipping 585 matching lines...) Expand 10 before | Expand all | Expand 10 after
596 checkInactivity: function() { 596 checkInactivity: function() {
597 if ((Date.now() - this.recentActivity) > INACTIVITY_TIMEOUT && 597 if ((Date.now() - this.recentActivity) > INACTIVITY_TIMEOUT &&
598 !userChangingProgressStatus) { 598 !userChangingProgressStatus) {
599 $('playercontrols').style.top = '100%'; 599 $('playercontrols').style.top = '100%';
600 } 600 }
601 } 601 }
602 }; 602 };
603 603
604 function setupPlaybackControls() { 604 function setupPlaybackControls() {
605 var element = $('playercontrols'); 605 var element = $('playercontrols');
606 playercontrols.innerHTML = ''; // clear out other 606 element.textContent = '';
607 var controlsclass = ''; 607 var controlsclass = '';
608 if (hidingControlsAnimation) { 608 if (hidingControlsAnimation) {
609 hidingControlsAnimation.cleanup(); 609 hidingControlsAnimation.cleanup();
610 hidingControlsAnimation = null; 610 hidingControlsAnimation = null;
611 } 611 }
612 if (videoPlaybackElement != null) { 612 if (videoPlaybackElement != null) {
613 controlsclass = 'video'; 613 controlsclass = 'video';
614 element.className = 'videocontrols'; 614 element.className = 'videocontrols';
615 hidingControlsAnimation = new HidingControlsAnimation(); 615 hidingControlsAnimation = new HidingControlsAnimation();
616 } else if (audioPlaybackElement != null) { 616 } else if (audioPlaybackElement != null) {
(...skipping 249 matching lines...) Expand 10 before | Expand all | Expand 10 after
866 <body onload='load();' onselectstart='return false'> 866 <body onload='load();' onselectstart='return false'>
867 <div id='error' class='error'></div> 867 <div id='error' class='error'></div>
868 <div id='title' class='audiotitle'></div> 868 <div id='title' class='audiotitle'></div>
869 <div id='glow' class='glow'></div> 869 <div id='glow' class='glow'></div>
870 <div class='playercontrolsbox'> 870 <div class='playercontrolsbox'>
871 <div id='playercontrols'> 871 <div id='playercontrols'>
872 </div> 872 </div>
873 </div> 873 </div>
874 </body> 874 </body>
875 </html> 875 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698