Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(759)

Side by Side Diff: chrome/browser/extensions/api/top_sites/top_sites_api.cc

Issue 815983002: Topsites become keyedService based. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove extra inclusion from testing_profile.h Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/extensions/api/top_sites/top_sites_api.h" 5 #include "chrome/browser/extensions/api/top_sites/top_sites_api.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/values.h" 8 #include "base/values.h"
9 #include "chrome/browser/history/top_sites.h" 9 #include "chrome/browser/history/top_sites.h"
10 #include "chrome/browser/history/top_sites_factory.h"
10 #include "chrome/browser/profiles/profile.h" 11 #include "chrome/browser/profiles/profile.h"
11 #include "chrome/browser/ui/webui/ntp/new_tab_ui.h" 12 #include "chrome/browser/ui/webui/ntp/new_tab_ui.h"
12 13
13 namespace extensions { 14 namespace extensions {
14 15
15 TopSitesGetFunction::TopSitesGetFunction() 16 TopSitesGetFunction::TopSitesGetFunction()
16 : weak_ptr_factory_(this) {} 17 : weak_ptr_factory_(this) {}
17 18
18 TopSitesGetFunction::~TopSitesGetFunction() {} 19 TopSitesGetFunction::~TopSitesGetFunction() {}
19 20
20 bool TopSitesGetFunction::RunAsync() { 21 bool TopSitesGetFunction::RunAsync() {
21 history::TopSites* ts = GetProfile()->GetTopSites(); 22 scoped_refptr<history::TopSites> ts =
23 TopSitesFactory::GetForProfile(GetProfile());
22 if (!ts) 24 if (!ts)
23 return false; 25 return false;
24 26
25 ts->GetMostVisitedURLs( 27 ts->GetMostVisitedURLs(
26 base::Bind(&TopSitesGetFunction::OnMostVisitedURLsAvailable, 28 base::Bind(&TopSitesGetFunction::OnMostVisitedURLsAvailable,
27 weak_ptr_factory_.GetWeakPtr()), false); 29 weak_ptr_factory_.GetWeakPtr()), false);
28 return true; 30 return true;
29 } 31 }
30 32
31 void TopSitesGetFunction::OnMostVisitedURLsAvailable( 33 void TopSitesGetFunction::OnMostVisitedURLsAvailable(
(...skipping 10 matching lines...) Expand all
42 page_value->SetString("title", url.title); 44 page_value->SetString("title", url.title);
43 pages_value->Append(page_value); 45 pages_value->Append(page_value);
44 } 46 }
45 } 47 }
46 48
47 SetResult(pages_value.release()); 49 SetResult(pages_value.release());
48 SendResponse(true); 50 SendResponse(true);
49 } 51 }
50 52
51 } // namespace extensions 53 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698