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

Unified Diff: chrome/browser/resources/file_manager/playlist.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: Created 9 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/file_manager/playlist.html
diff --git a/chrome/browser/resources/file_manager/playlist.html b/chrome/browser/resources/file_manager/playlist.html
index 5c7ff3118436844d067367231bdd84e4f9a98226..66a191e69541c2191c3653fb9a9583fde2275cb2 100644
--- a/chrome/browser/resources/file_manager/playlist.html
+++ b/chrome/browser/resources/file_manager/playlist.html
@@ -105,7 +105,9 @@ function setPlaylistOffset(offset) {
function updateUI() {
var main = $('main');
if (currentPlaylist) {
- main.innerHTML = '';
+ while (main.firstChild) {
+ main.removeChild(main.firstChild);
+ }
var main = $('main');
for (var x = 0; x < currentPlaylist.length; x++) {
var rowdiv = document.createElement('div');

Powered by Google App Engine
This is Rietveld 408576698