With the recent NNTP changes, we change the way themes work - they're now closer to NTP1 theming - ntp_section_* is used to control the display of the light blue sections, and we've added ntp_header that controls the dark blue hover color, ntp_header falls back to ntp_section if provided.

Also fix a minor bug where the default theme was saving theme data - we were generating colors before saving the provided colors.

BUG=18720,18722
TEST=Install a theme and verify that the recently closed section is themed correctly.

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

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

1 files changed, 13 insertions, 9 deletions

diff --git a/chrome/browser/resources/new_tab_theme.css b/chrome/browser/resources/new_tab_theme.css
index 12ac86d..b8459aa 100644
--- a/chrome/browser/resources/new_tab_theme.css
+++ b/chrome/browser/resources/new_tab_theme.css
@@ -16,11 +16,14 @@ body {
 #main {
   background: url(chrome://theme/product_logo?$1) no-repeat 0 8px;
 }
+.thumbnail {
+  border-color: $$1; /* COLOR_NTP_SECTION */
+}
 .thumbnail-container,
 .list > .thumbnail-container > .title,
 .list .title > div,
 .thumbnail-container > .title {
-  color: $8;
+  color: $9;
 }
 .thumbnail-container:focus .thumbnail,
 .thumbnail-container:hover .thumbnail {
@@ -45,14 +48,15 @@ body {
   background-color: $7;
   color: white;
 }
-.tips-title {
-  color: $$1;
-}
 
-/* TODO(arv): The theme does not have the thumbnail outline color in non hover
-   mode
 #recently-closed {
-  border-color:
-  background-color:
+  border-color: $$2; /* color_section_border */
+  background-color: $$1; /* COLOR_NTP_SECTION */
+  color: $$3; /* COLOR_NTP_SECTION_TEXT */
+}
+#recently-closed span,
+#recently-closed a,
+#recently-closed a:link {
+  color: $$4; /* COLOR_NTP_SECTION_LINK */
+  text-decoration:underline;
 }
-*/