Add functions to clean URLs from the activity log

chrome/browser/extensions/activity_log/activity_log.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/activity_log/activity_log.h"
 
 #include <set>
 #include <vector>
 
 #include "base/command_line.h"
@@ skipping 409 matching lines @@
 
 // static
 void ActivityLog::RegisterProfilePrefs(
     user_prefs::PrefRegistrySyncable* registry) {
   registry->RegisterBooleanPref(
       prefs::kWatchdogExtensionActive,
       false,
       user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
 }
 
+void ActivityLog::RemoveURLs(const std::vector<GURL>& restrict_urls) {
+  if (!IsLogEnabled()) {

[felt, 2013/08/27 19:59:03]

I don't think the DLOG is needed -- I think just returning would suffice.

You also should check if(policy_)

[karenlees, 2013/08/27 21:13:25]

Done.

+    DLOG(INFO) << "Log not enabled for this profile.";
+    return;
+  }
+  policy_->RemoveURLs(restrict_urls);
+}
+
+void ActivityLog::RemoveURLs(const std::set<GURL>& restrict_urls) {
+  if (!IsLogEnabled()) {
+    DLOG(INFO) << "Log not enabled for this profile.";
+    return;
+  }
+
+  std::vector<GURL> urls;
+  for (std::set<GURL>::iterator it = restrict_urls.begin();
+       it != restrict_urls.end(); ++it) {
+    urls.push_back(*it);
+  }
+  policy_->RemoveURLs(urls);
+}
+
+void ActivityLog::RemoveURL(const GURL& url) {
+  if (url.is_empty())
+    return;
+  std::vector<GURL> urls;
+  urls.push_back(url);
+  RemoveURLs(urls);
+}
+
 }  // namespace extensions