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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/history/history_notifications.h

Issue 10031028: Single URL Expires Were Not Being Deleted. (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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/history/history_extension_api.cc ('k') | chrome/browser/history/history_types.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/history/history_notifications.h
===================================================================
--- chrome/browser/history/history_notifications.h (revision 132078)
+++ chrome/browser/history/history_notifications.h (working copy)
@@ -57,15 +57,9 @@
// Set when all history was deleted. False means just a subset was deleted.
bool all_history;
- // The URLRows which have been deleted.
+ // The URLRows of URLs deleted. This is valid only when all_history is false
+ // indicating that a subset of history has been deleted.
URLRows rows;
-
- // The list of unique URLs affected. This is valid only when a subset of
- // history is deleted. When all of it is deleted, this will be empty, since
- // we do not bother to list all URLs. (This information can be gleaned from
- // |rows| but, since there are several clients who need the set, we pre-build
- // it so that the clients don't have to.)
- std::set<GURL> urls;
};
// Details for NOTIFY_URLS_STARRED.
« no previous file with comments | « chrome/browser/history/history_extension_api.cc ('k') | chrome/browser/history/history_types.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698