Fix broken thumbnails for sites with redirects.

chrome/browser/dom_ui/most_visited_handler.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/dom_ui/most_visited_handler.h"
 
 #include <set>
 
 #include "app/l10n_util.h"
 #include "base/callback.h"
@@ skipping 109 matching lines @@
 void MostVisitedHandler::HandleGetMostVisited(const ListValue* args) {
   if (!got_first_most_visited_request_) {
     // If our intial data is already here, return it.
     SendPagesValue();
     got_first_most_visited_request_ = true;
   } else {
     StartQueryForMostVisited();
   }
 }
 
-// Set a DictionaryValue |dict| from a MostVisitedURL.
-void SetDictionaryValue(const history::MostVisitedURL& url,
-                        DictionaryValue& dict) {
-  NewTabUI::SetURLTitleAndDirection(&dict, url.title, url.url);
-  dict.SetString("url", url.url.spec());
-  dict.SetString("faviconUrl", url.favicon_url.spec());
-  // TODO(Nik): Need thumbnailUrl?
-  // TODO(Nik): Add pinned and blacklisted URLs.
-  dict.SetBoolean("pinned", false);
-}
-
 void MostVisitedHandler::SendPagesValue() {
   if (pages_value_.get()) {
     FundamentalValue first_run(IsFirstRun());
     dom_ui_->CallJavascriptFunction(L"mostVisitedPages",
                                     *(pages_value_.get()), first_run);
     pages_value_.reset();
   }
 }
 
 void MostVisitedHandler::StartQueryForMostVisited() {
@@ skipping 381 matching lines @@
 
 std::string MostVisitedHandler::GetDictionaryKeyForURL(const std::string& url) {
   return MD5String(url);
 }
 
 // static
 void MostVisitedHandler::RegisterUserPrefs(PrefService* prefs) {
   prefs->RegisterDictionaryPref(prefs::kNTPMostVisitedURLsBlacklist);
   prefs->RegisterDictionaryPref(prefs::kNTPMostVisitedPinnedURLs);
 }