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

Unified Diff: chrome/browser/extensions/activity_log/activity_log_policy.cc

Issue 23432006: [Activity log] Strip incognito URLs entirely from the database (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 7 years, 3 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 | « no previous file | chrome/browser/extensions/activity_log/activity_log_policy_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/activity_log/activity_log_policy.cc
diff --git a/chrome/browser/extensions/activity_log/activity_log_policy.cc b/chrome/browser/extensions/activity_log/activity_log_policy.cc
index 479e6406b116b74472afd6610a1992b178ca276c..107b49e0a340d3d061907c9be16d45d35e0962e5 100644
--- a/chrome/browser/extensions/activity_log/activity_log_policy.cc
+++ b/chrome/browser/extensions/activity_log/activity_log_policy.cc
@@ -86,9 +86,11 @@ void ActivityLogPolicy::Util::StripPrivacySensitiveFields(
if (action->page_incognito()) {
action->set_page_url(GURL());
action->set_page_title("");
+ action->set_page_incognito(false);
}
if (action->arg_incognito()) {
action->set_arg_url(GURL());
+ action->set_arg_incognito(false);
}
// Strip query parameters, username/password, etc., from URLs.
« no previous file with comments | « no previous file | chrome/browser/extensions/activity_log/activity_log_policy_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698