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

Side by Side Diff: chrome/browser/history/top_sites_impl.h

Issue 59903010: Ensure using chrome://thumb2/... adds the URL as a forced URL in TopSites so that a thumbnail is ca… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Take 2. Created 7 years, 1 month 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 | Annotate | Revision Log
« no previous file with comments | « chrome/browser/history/top_sites.h ('k') | chrome/browser/history/top_sites_impl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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 #ifndef CHROME_BROWSER_HISTORY_TOP_SITES_IMPL_H_ 5 #ifndef CHROME_BROWSER_HISTORY_TOP_SITES_IMPL_H_
6 #define CHROME_BROWSER_HISTORY_TOP_SITES_IMPL_H_ 6 #define CHROME_BROWSER_HISTORY_TOP_SITES_IMPL_H_
7 7
8 #include <list> 8 #include <list>
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
80 virtual void ClearBlacklistedURLs() OVERRIDE; 80 virtual void ClearBlacklistedURLs() OVERRIDE;
81 virtual void Shutdown() OVERRIDE; 81 virtual void Shutdown() OVERRIDE;
82 virtual CancelableRequestProvider::Handle StartQueryForMostVisited() OVERRIDE; 82 virtual CancelableRequestProvider::Handle StartQueryForMostVisited() OVERRIDE;
83 virtual bool IsKnownURL(const GURL& url) OVERRIDE; 83 virtual bool IsKnownURL(const GURL& url) OVERRIDE;
84 virtual const std::string& GetCanonicalURLString( 84 virtual const std::string& GetCanonicalURLString(
85 const GURL& url) const OVERRIDE; 85 const GURL& url) const OVERRIDE;
86 virtual bool IsNonForcedFull() OVERRIDE; 86 virtual bool IsNonForcedFull() OVERRIDE;
87 virtual bool IsForcedFull() OVERRIDE; 87 virtual bool IsForcedFull() OVERRIDE;
88 virtual MostVisitedURLList GetPrepopulatePages() OVERRIDE; 88 virtual MostVisitedURLList GetPrepopulatePages() OVERRIDE;
89 virtual bool loaded() const OVERRIDE; 89 virtual bool loaded() const OVERRIDE;
90 virtual bool AddForcedURL(const GURL& url, const base::Time& time) OVERRIDE;
90 91
91 protected: 92 protected:
92 virtual ~TopSitesImpl(); 93 virtual ~TopSitesImpl();
93 94
94 private: 95 private:
95 friend class TopSitesImplTest; 96 friend class TopSitesImplTest;
96 FRIEND_TEST_ALL_PREFIXES(TopSitesImplTest, DiffMostVisited); 97 FRIEND_TEST_ALL_PREFIXES(TopSitesImplTest, DiffMostVisited);
97 FRIEND_TEST_ALL_PREFIXES(TopSitesImplTest, DiffMostVisitedWithForced); 98 FRIEND_TEST_ALL_PREFIXES(TopSitesImplTest, DiffMostVisitedWithForced);
98 99
99 typedef base::Callback<void(const MostVisitedURLList&, 100 typedef base::Callback<void(const MostVisitedURLList&,
(...skipping 166 matching lines...) Expand 10 before | Expand all | Expand 10 after
266 267
267 // Are we loaded? 268 // Are we loaded?
268 bool loaded_; 269 bool loaded_;
269 270
270 DISALLOW_COPY_AND_ASSIGN(TopSitesImpl); 271 DISALLOW_COPY_AND_ASSIGN(TopSitesImpl);
271 }; 272 };
272 273
273 } // namespace history 274 } // namespace history
274 275
275 #endif // CHROME_BROWSER_HISTORY_TOP_SITES_IMPL_H_ 276 #endif // CHROME_BROWSER_HISTORY_TOP_SITES_IMPL_H_
OLDNEW
« no previous file with comments | « chrome/browser/history/top_sites.h ('k') | chrome/browser/history/top_sites_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698