Retry r88137:

chrome/browser/ui/webui/ntp/most_visited_handler.cc

Index: chrome/browser/ui/webui/ntp/most_visited_handler.cc
diff --git a/chrome/browser/ui/webui/ntp/most_visited_handler.cc b/chrome/browser/ui/webui/ntp/most_visited_handler.cc
index 8d78c4ac9e24657be5f5fcd4499466353e881813..712823c48d82ddbc3fb9525ba4d2628b3dff06d2 100644
--- a/chrome/browser/ui/webui/ntp/most_visited_handler.cc
+++ b/chrome/browser/ui/webui/ntp/most_visited_handler.cc
@@ -84,8 +84,6 @@ WebUIMessageHandler* MostVisitedHandler::Attach(WebUI* web_ui) {
 }
 
 void MostVisitedHandler::RegisterMessages() {
-  // Register ourselves as the handler for the "getMostSisited" message from
-  // Javascript.
   web_ui_->RegisterMessageCallback("getMostVisited",
       NewCallback(this, &MostVisitedHandler::HandleGetMostVisited));
 
@@ -291,10 +289,12 @@ void MostVisitedHandler::SetPagesValueFromTopSites(
     if (url.url.spec() == l10n_util::GetStringUTF8(IDS_CHROME_WELCOME_URL)) {
       page_value->SetString("thumbnailUrl",
           "chrome://theme/IDR_NEWTAB_CHROME_WELCOME_PAGE_THUMBNAIL");
+      page_value->SetString("faviconDominantColor", "rgb(0, 147, 60)");

[tony, 2011/06/09 20:37:23]

Nit: Might be worth commenting what these colors are.  Or you could do something
like const char* kGreen = "rgb(...)";

[Evan Stade, 2011/06/09 20:51:07]

I did once ask someone in a code review to do this, but have since decided it's
not worth it. The problem is I wouldn't know what to call it (this color is
different from what you get if you were to type just "green" in css). Seems the
best way to get an accurate idea of the color is to use some graphic tool. And
FWIW, we don't usually do this when colors are defined in css files, which is
pretty much the same situation. 

     } else if (url.url.spec() ==
                l10n_util::GetStringUTF8(IDS_THEMES_GALLERY_URL)) {
       page_value->SetString("thumbnailUrl",
           "chrome://theme/IDR_NEWTAB_THEMES_GALLERY_THUMBNAIL");
+      page_value->SetString("faviconDominantColor", "rgb(63, 132, 197)");
     }
 
     history::TopSites* ts = web_ui_->GetProfile()->GetTopSites();