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

components/offline_pages/client_policy_controller.h

Index: components/offline_pages/client_policy_controller.h
diff --git a/components/offline_pages/client_policy_controller.h b/components/offline_pages/client_policy_controller.h
index 77f116b7fa865041c21e9322dbf882c762b3e520..231d028d4a6e3bf0e79ce41e35efee983c47bce5 100644
--- a/components/offline_pages/client_policy_controller.h
+++ b/components/offline_pages/client_policy_controller.h
@@ -27,7 +27,8 @@ class ClientPolicyController {
       const std::string& name_space,
       LifetimePolicy::LifetimeType lifetime_type,
       const base::TimeDelta& expiration_period,
-      size_t page_limit);
+      size_t page_limit,
+      size_t pages_allowed_per_url);

[chili, 2016/06/20 18:17:47]

just curious, and for my information... do we see a case in the future where we
*want* to allow multiple pages per url?  When/how would it happen, other than
last_n & bookmarks?

Also, how do we define url?  As the full url with query + fragment? (like
http://www.google.com?query=foo#header) or do we cut off the fragment?

 
   // Get the client policy for |name_space|.
   const OfflinePageClientPolicy& GetPolicy(const std::string& name_space) const;