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

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

Issue 270153004: Introduce ActiveScriptController; track active extension scripts (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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
Index: chrome/browser/extensions/activity_log/uma_policy_unittest.cc
diff --git a/chrome/browser/extensions/activity_log/uma_policy_unittest.cc b/chrome/browser/extensions/activity_log/uma_policy_unittest.cc
index 019feeb05dc41c5de6dd19cb9cb62fd788324438..86f57f839a0c7723d732bd0fc654b48e469a8765 100644
--- a/chrome/browser/extensions/activity_log/uma_policy_unittest.cc
+++ b/chrome/browser/extensions/activity_log/uma_policy_unittest.cc
@@ -89,8 +89,8 @@ TEST_F(UmaPolicyTest, MatchActionToStatusTest) {
TEST_F(UmaPolicyTest, SiteUrlTest) {
UmaPolicy* policy = new UmaPolicy(profile_.get());
- const std::string site0 = "http://www.zzz.com";
- const std::string site1 = "http://www.foo.com";
+ const std::string site0 = "http://www.zzz.com/";
+ const std::string site1 = "http://www.foo.com/";
const std::string site2 = "http://www.google.com#a";
const std::string site3 = "http://www.google.com#bb";
@@ -109,10 +109,10 @@ TEST_F(UmaPolicyTest, SiteUrlTest) {
ASSERT_EQ(3, policy->url_status()[site3][UmaPolicy::kNumberOfTabs]);
// Remove some sites.
- policy->CleanupClosedPage(site0);
- policy->CleanupClosedPage(site2);
- policy->CleanupClosedPage(site2);
- policy->CleanupClosedPage(site3);
+ policy->CleanupClosedPage(site0, NULL);
+ policy->CleanupClosedPage(site2, NULL);
+ policy->CleanupClosedPage(site2, NULL);
+ policy->CleanupClosedPage(site3, NULL);
// Check that the removal worked.
ASSERT_EQ(2u, policy->url_status().size());

Powered by Google App Engine
This is Rietveld 408576698