Remove the "Bookmark sync" menu item

This also cleans up the states.

BUG=28419
TEST=The menu item should be gone but we should still see errors from bookmark
sync on the NTP

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

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

2 files changed, 1 insertions, 14 deletions

diff --git a/chrome/browser/resources/new_new_tab.html b/chrome/browser/resources/new_new_tab.html
index 831d3b1..79d0c0d 100644
--- a/chrome/browser/resources/new_new_tab.html
+++ b/chrome/browser/resources/new_new_tab.html
@@ -185,8 +185,6 @@ function updateSimpleSection(id, section) {
     <div command="hide" section="THUMB" i18n-content="mostvisited"></div>
     <div command="hide" section="RECENT" i18n-content="recentlyclosed"></div>
     <div command="hide" section="TIPS" i18n-content="tips"></div>
-    <div command="hide" section="SYNC" i18n-content="sync"
-         id="sync-menu-item"></div>
     <hr>
     <div command="clear-all-blacklisted"
          i18n-content="restorethumbnails"></div>
@@ -195,8 +193,6 @@ function updateSimpleSection(id, section) {
   <script>
     $('thumb-checkbox').checked = shownSections & Section.THUMB;
     $('list-checkbox').checked = shownSections & Section.LIST;
-    $('sync-menu-item').style.display =
-        templateData['syncispresent'] == 'true' ? '' : 'none';
   </script>
 
   <div id="notification">
@@ -252,7 +248,6 @@ function updateSimpleSection(id, section) {
     <h2></h2>
     <span></span>
   </div>
-  <script>updateSimpleSection('sync-status', Section.SYNC);</script>
 
   <div id="tip-line"></div>
   <script>updateSimpleSection('tip-line', Section.TIPS);</script>
diff --git a/chrome/browser/resources/new_new_tab.js b/chrome/browser/resources/new_new_tab.js
index 570e3ea..093e36a 100644
--- a/chrome/browser/resources/new_new_tab.js
+++ b/chrome/browser/resources/new_new_tab.js
@@ -335,9 +335,6 @@ function showSection(section) {
       case Section.TIPS:
         $('tip-line').style.display = '';
         break;
-      case Section.SYNC:
-        $('sync-status').style.display = '';
-        break;
     }
   }
 }
@@ -359,9 +356,6 @@ function hideSection(section) {
       case Section.TIPS:
         $('tip-line').style.display = 'none';
         break;
-      case Section.SYNC:
-        $('sync-status').style.display = 'none';
-        break;
     }
   }
 }
@@ -620,7 +614,6 @@ function layoutRecentlyClosed() {
  *
  * syncsectionisvisible: true if the sync section needs to show up on the new
  *                       tab page and false otherwise.
- * msgtype: represents the states - "error", "presynced" or "synced".
  * title: the header for the sync status section.
  * msg: the actual message (e.g. "Synced to foo@gmail.com").
  * linkisvisible: true if the link element should be visible within the sync
@@ -637,10 +630,9 @@ function layoutRecentlyClosed() {
 function syncMessageChanged(newMessage) {
   var syncStatusElement = $('sync-status');
   var style = syncStatusElement.style;
-  $('sync-menu-item').style.display = 'block';
 
   // Hide the section if the message is emtpy.
-  if (!newMessage['syncsectionisvisible'] || !(shownSections & Section.SYNC)) {
+  if (!newMessage['syncsectionisvisible']) {
     style.display = 'none';
     return;
   }