Makes new tab page update properly when its told bookmarks are

changed.

BUG=1861
TEST=see bug

Review URL: http://codereview.chromium.org/1618

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@1845 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 10 insertions, 5 deletions

diff --git a/chrome/browser/resources/new_tab.html b/chrome/browser/resources/new_tab.html
index a49901b..2b90255 100644
--- a/chrome/browser/resources/new_tab.html
+++ b/chrome/browser/resources/new_tab.html
@@ -373,6 +373,7 @@ document.addEventListener('DOMContentLoaded', handleDOMContentLoaded);
 
     <div id="recentlyBookmarked" class="sidebar" style="display:none">
       <span class="section-title" jscontent="bookmarks"></span>
+      <div id="recentlyBookmarkedContainer"></div>
     </div>
 
     <div id="recentlyClosedTabs" class="sidebar" style="display:none">
@@ -537,10 +538,16 @@ function renderSearchURLs(urls) {
    attributes. */
 function renderRecentlyBookmarked(entries) {
   logEvent('renderRecentlyBookmarked called: ' + entries.length);
-  var container = document.getElementById('recentlyBookmarked');
+  var section   = document.getElementById('recentlyBookmarked');
+  var container = document.getElementById('recentlyBookmarkedContainer');
+
+  /* recentlyBookmarked is called any time the bookmarks change. Remove existing
+     entries before adding new ones. */
+  section.style.display = 'none';
+  container.innerHTML = '';
 
   if (entries.length > 0) {
-    container.style.display = 'block';
+    section.style.display = 'block';
     for (var i = 0, entry = entries[0]; entry = entries[i]; ++i) {
       var link = DOM('a', {href: entry.url, className:'recent-bookmark', title:entry.title});
       link.style.backgroundImage =
@@ -564,9 +571,7 @@ function renderRecentlyClosedTabs(entries) {
      date. Therefore, reset the recentlyClosedTabs state on every
      call. */
   section.style.display = 'none';
-  while(container.hasChildNodes()) {
-    container.removeChild(container.firstChild);
-  }
+  container.innerHTML = '';
 
   if (entries.length > 0) {
     section.style.display = 'block';