[Offline Pages] Duplicate pages when save/delete bookmarks.

components/offline_pages/client_policy_controller.cc

Index: components/offline_pages/client_policy_controller.cc
diff --git a/components/offline_pages/client_policy_controller.cc b/components/offline_pages/client_policy_controller.cc
index b8ccb0f679bb9cdf05d057f5f46ed163e28b7c63..c3c454cfa0305a1ccbbf7eec653462699d80a33c 100644
--- a/components/offline_pages/client_policy_controller.cc
+++ b/components/offline_pages/client_policy_controller.cc
@@ -19,14 +19,14 @@ ClientPolicyController::ClientPolicyController() {
   policies_.insert(std::make_pair(
       kBookmarkNamespace,
       MakePolicy(kBookmarkNamespace, LifetimeType::TEMPORARY,
-                 base::TimeDelta::FromDays(7), kUnlimitedPages)));
+                 base::TimeDelta::FromDays(7), kUnlimitedPages, 1)));
   policies_.insert(std::make_pair(
       kLastNNamespace, MakePolicy(kLastNNamespace, LifetimeType::TEMPORARY,
-                                  base::TimeDelta::FromDays(2), 20)));
+                                  base::TimeDelta::FromDays(2), 20, 1)));
   // Fallback policy.
   policies_.insert(std::make_pair(
       kDefaultNamespace, MakePolicy(kDefaultNamespace, LifetimeType::TEMPORARY,
-                                    base::TimeDelta::FromDays(1), 10)));
+                                    base::TimeDelta::FromDays(1), 10, 1)));
 }
 
 ClientPolicyController::~ClientPolicyController() {}
@@ -36,9 +36,11 @@ const OfflinePageClientPolicy ClientPolicyController::MakePolicy(
     const std::string& name_space,
     LifetimeType lifetime_type,
     const base::TimeDelta& expire_period,
-    size_t page_limit) {
-  OfflinePageClientPolicy policy(
-      {name_space, {lifetime_type, expire_period, page_limit}});
+    size_t page_limit,
+    size_t pages_allowed_per_url) {
+  OfflinePageClientPolicy policy({name_space,
+                                  {lifetime_type, expire_period, page_limit},

[fgorski, 2016/06/21 19:49:51]

nit: please let me know if you did git cl format.
I am simply curious if that is default formatting for such case. If so I am
perfectly happy

[romax, 2016/06/21 23:04:45]

yes that's what git cl format gives me.

+                                  pages_allowed_per_url});
   return policy;
 }