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

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

Issue 10019004: Changes TopSites to access blacklist from prefs when necessary. I'm (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | chrome/browser/history/top_sites.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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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_H_ 5 #ifndef CHROME_BROWSER_HISTORY_TOP_SITES_H_
6 #define CHROME_BROWSER_HISTORY_TOP_SITES_H_ 6 #define CHROME_BROWSER_HISTORY_TOP_SITES_H_
7 #pragma once 7 #pragma once
8 8
9 #include <list> 9 #include <list>
10 #include <set> 10 #include <set>
(...skipping 356 matching lines...) Expand 10 before | Expand all | Expand 10 after
367 // non-empty at startup. After we read the top sites from the DB, we'll 367 // non-empty at startup. After we read the top sites from the DB, we'll
368 // always have a cached list. 368 // always have a cached list.
369 PendingCallbackSet pending_callbacks_; 369 PendingCallbackSet pending_callbacks_;
370 370
371 // Stores thumbnails for unknown pages. When SetPageThumbnail is 371 // Stores thumbnails for unknown pages. When SetPageThumbnail is
372 // called, if we don't know about that URL yet and we don't have 372 // called, if we don't know about that URL yet and we don't have
373 // enough Top Sites (new profile), we store it until the next 373 // enough Top Sites (new profile), we store it until the next
374 // SetTopSites call. 374 // SetTopSites call.
375 TempImages temp_images_; 375 TempImages temp_images_;
376 376
377 // Blacklisted and pinned URLs are stored in Preferences.
378
379 // Blacklisted URLs. They are filtered out from the list of Top
380 // Sites when GetMostVisitedURLs is called. Note that we are still
381 // storing all URLs, but filtering on access. It is a dictionary,
382 // key is the URL, value is a dummy value. This is owned by the
383 // PrefService.
384 const base::DictionaryValue* blacklist_;
385
386 // This is a dictionary for the pinned URLs for the the most visited part of 377 // This is a dictionary for the pinned URLs for the the most visited part of
387 // the new tab page. Key is the URL, value is index where it is pinned at (may 378 // the new tab page. Key is the URL, value is index where it is pinned at (may
388 // be the same as key). This is owned by the PrefService. 379 // be the same as key). This is owned by the PrefService.
389 const base::DictionaryValue* pinned_urls_; 380 const base::DictionaryValue* pinned_urls_;
390 381
391 // See description above HistoryLoadState. 382 // See description above HistoryLoadState.
392 HistoryLoadState history_state_; 383 HistoryLoadState history_state_;
393 384
394 // See description above TopSitesLoadState. 385 // See description above TopSitesLoadState.
395 TopSitesLoadState top_sites_state_; 386 TopSitesLoadState top_sites_state_;
396 387
397 // Are we loaded? 388 // Are we loaded?
398 bool loaded_; 389 bool loaded_;
399 390
400 DISALLOW_COPY_AND_ASSIGN(TopSites); 391 DISALLOW_COPY_AND_ASSIGN(TopSites);
401 }; 392 };
402 393
403 extern const TopSites::PrepopulatedPage kPrepopulatedPages[2]; 394 extern const TopSites::PrepopulatedPage kPrepopulatedPages[2];
404 395
405 } // namespace history 396 } // namespace history
406 397
407 #endif // CHROME_BROWSER_HISTORY_TOP_SITES_H_ 398 #endif // CHROME_BROWSER_HISTORY_TOP_SITES_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/history/top_sites.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698