Move topSites API out of experimental

chrome/browser/history/top_sites_extension_api.cc

Index: chrome/browser/history/top_sites_extension_api.cc
diff --git a/chrome/browser/history/top_sites_extension_api.cc b/chrome/browser/history/top_sites_extension_api.cc
index 6a8c4b2f69deb44f3b4a843a9f4b7d12b42fae7d..9808c763926716f7340518ca168e29ac90207e36 100644
--- a/chrome/browser/history/top_sites_extension_api.cc
+++ b/chrome/browser/history/top_sites_extension_api.cc
@@ -29,24 +29,16 @@ bool GetTopSitesFunction::RunImpl() {
 
 void GetTopSitesFunction::OnMostVisitedURLsAvailable(
     const history::MostVisitedURLList& data) {
-  // Code is a direct rip from most_visited_handler.cc TODO(estade): unfork.
   scoped_ptr<base::ListValue> pages_value(new ListValue);
   for (size_t i = 0; i < data.size(); i++) {
     const history::MostVisitedURL& url = data[i];
     DictionaryValue* page_value = new DictionaryValue();
-    if (url.url.is_empty()) {
-      page_value->SetBoolean("filler", true);
+    if (!url.url.is_empty())

[brettw, 2012/03/21 17:43:22]

You either messed up your indenting or you need {} here.

[cduvall, 2012/03/21 18:00:34]

Oops, Done.

+      NewTabUI::SetURLTitleAndDirection(page_value,
+                                        url.title,
+                                        url.url);
       pages_value->Append(page_value);
-      continue;
-    }
-
-    NewTabUI::SetURLTitleAndDirection(page_value,
-                                      url.title,
-                                      url.url);
-
-    pages_value->Append(page_value);
   }
-  // End copied code. ----------------------------------------------------------
 
   result_.reset(pages_value.release());
   SendResponse(true);