[NTP4] Fix theme and bookmarks bar notifications.

chrome/browser/resources/incognito_tab.html

Index: chrome/browser/resources/incognito_tab.html
diff --git a/chrome/browser/resources/incognito_tab.html b/chrome/browser/resources/incognito_tab.html
index d2230e0f020b7ac1f78ba8152c482b394931c4e7..2fb757bcca92cd4b4303eb89316f0922c48c18b1 100644
--- a/chrome/browser/resources/incognito_tab.html
+++ b/chrome/browser/resources/incognito_tab.html
@@ -20,18 +20,27 @@ document.write('<link id="incognitothemecss" rel="stylesheet" ' +
   </div>
 </div>
 </body>
+<script src="chrome://resources/js/cr.js"></script>

[Dan Beam, 2012/03/27 10:30:13]

I don't really have to do this, but I figured it wasn't too expensive: I can
change to vanilla namespace as well if you don't want to depend on cr.js here.

[Evan Stade, 2012/03/27 18:25:09]

it's fine

 <script>
-function themeChanged() {
-  document.getElementById('incognitothemecss').href =
-      'chrome://theme/css/incognito_new_tab_theme.css?' + Date.now();
-}
+cr.define('ntp', function() {
+  'use strict';
+  /**
+   * Set whether the bookmarks bar is attached or not.
+   * @param {boolean} attached Whether the bar is attached or not.
+   */
+  function setBookmarkBarAttached(attached) {
+    document.documentElement.setAttribute('bookmarkbarattached', !!attached);
+  }
 
-function bookmarkBarAttached() {
-  document.documentElement.setAttribute("bookmarkbarattached", "true");
-}
+  function themeChanged() {
+    document.getElementById('incognitothemecss').href =
+        'chrome://theme/css/incognito_new_tab_theme.css?' + Date.now();
+  }
 
-function bookmarkBarDetached() {
-  document.documentElement.setAttribute("bookmarkbarattached", "false");
-}
+  return {
+    setBookmarkBarAttached: setBookmarkBarAttached,
+    themeChanged: themeChanged,
+  };
+});
 </script>
 </html>