Move topSites API out of experimental

chrome/browser/history/history_extension_api.cc

 // Copyright (c) 2012 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/history/history_extension_api.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/json/json_writer.h"
 #include "base/message_loop.h"
 #include "base/string_number_conversions.h"
 #include "base/values.h"
+#include "chrome/browser/cancelable_request.h"
 #include "chrome/browser/extensions/extension_event_router.h"
 #include "chrome/browser/history/history.h"
 #include "chrome/browser/history/history_types.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_source.h"
 
 namespace {
 
 const char kAllHistoryKey[] = "allHistory";
 const char kEndTimeKey[] = "endTime";
 const char kFaviconUrlKey[] = "favIconUrl";
 const char kIdKey[] = "id";
 const char kLastVisitdKey[] = "lastVisitTime";
 const char kMaxResultsKey[] = "maxResults";
 const char kNewKey[] = "new";
 const char kReferringVisitId[] = "referringVisitId";
 const char kRemovedKey[] = "removed";
 const char kStartTimeKey[] = "startTime";
 const char kTextKey[] = "text";
 const char kTitleKey[] = "title";
 const char kTypedCountKey[] = "typedCount";
 const char kVisitCountKey[] = "visitCount";
 const char kTransition[] = "transition";
 const char kUrlKey[] = "url";
 const char kUrlsKey[] = "urls";
 const char kVisitId[] = "visitId";
 const char kVisitTime[] = "visitTime";
+const char kRedirectsKey[] = "redirects";
 
 const char kOnVisited[] = "history.onVisited";
 const char kOnVisitRemoved[] = "history.onVisitRemoved";
 
 const char kInvalidIdError[] = "History item id is invalid.";
 const char kInvalidUrlError[] = "Url is invalid.";
 
+const int kDaysOfHistory = 90;
+
 double MilliSecondsFromTime(const base::Time& time) {
   return 1000 * time.ToDoubleT();
 }
 
 void GetHistoryItemDictionary(const history::URLRow& row,
                               DictionaryValue* value) {
   value->SetString(kIdKey, base::Int64ToString(row.id()));
   value->SetString(kUrlKey, row.url().spec());
   value->SetString(kTitleKey, row.title());
   value->SetDouble(kLastVisitdKey,
@@ skipping 21 matching lines @@
   DCHECK(trans) << "Invalid transition.";
   value->SetString(kTransition, trans);
 }
 
 void AddVisitNode(const history::VisitRow& row, ListValue* list) {
   DictionaryValue* dict = new DictionaryValue();
   GetVisitInfoDictionary(row, dict);
   list->Append(dict);
 }
 
+void AddMostVisitedNode(const history::MostVisitedURL& item, ListValue* list) {
+  DictionaryValue* dict = new DictionaryValue();
+  ListValue* redirect_list = new ListValue();
+  dict->SetString(kUrlKey, item.url.spec());
+  dict->SetString(kTitleKey, item.title);
+  for (history::RedirectList::const_iterator iterator = item.redirects.begin();
+       iterator != item.redirects.end(); ++iterator) {
+    StringValue* redirect = new StringValue((*iterator).spec());
+    redirect_list->Append(redirect);
+  }
+  dict->Set(kRedirectsKey, redirect_list);
+  list->Append(dict);
+}
+
 }  // namespace
 
 HistoryExtensionEventRouter::HistoryExtensionEventRouter() {}
 
 HistoryExtensionEventRouter::~HistoryExtensionEventRouter() {}
 
 void HistoryExtensionEventRouter::ObserveProfile(Profile* profile) {
   CHECK(registrar_.IsEmpty());
   const content::Source<Profile> source = content::Source<Profile>(profile);
   registrar_.Add(this,
@@ skipping 292 matching lines @@
       &cancelable_consumer_,
       base::Bind(&DeleteAllHistoryFunction::DeleteComplete,
                  base::Unretained(this)));
 
   return true;
 }
 
 void DeleteAllHistoryFunction::DeleteComplete() {
   SendAsyncResponse();
 }
+
+bool GetMostVisitedHistoryFunction::RunAsyncImpl() {
+  DictionaryValue* json = NULL;
+  EXTENSION_FUNCTION_VALIDATE(args_->GetDictionary(0, &json));
+
+  Value* value = NULL;
+  int how_many = 0;
+  EXTENSION_FUNCTION_VALIDATE(json->Get(kMaxResultsKey, &value));
+  value->GetAsInteger(&how_many);
+
+  HistoryService* hs = profile()->GetHistoryService(Profile::EXPLICIT_ACCESS);
+  hs->QueryMostVisitedURLs(
+      how_many,
+      kDaysOfHistory,

[Aaron Boodman, 2012/03/20 02:24:24]

Why not allow developers to specify this too. You could default it to 90 if they
don't specify.

+      &history_consumer_,
+      base::Bind(&GetMostVisitedHistoryFunction::QueryComplete,
+                 base::Unretained(this)));
+  return true;
+}
+
+void GetMostVisitedHistoryFunction::QueryComplete(
+    CancelableRequestProvider::Handle handle,
+    history::MostVisitedURLList pages) {
+  ListValue* list = new ListValue();
+  for (history::MostVisitedURLList::const_iterator iterator = pages.begin();
+       iterator != pages.end();
+       ++iterator) {

[Aaron Boodman, 2012/03/20 02:24:24]

This can go on previous line.

+    AddMostVisitedNode(*iterator, list);
+  }
+  result_.reset(list);
+  SendAsyncResponse();
+}