[Offline Pages] Add new download namespace

components/offline_pages/client_policy_controller.cc

 // Copyright 2016 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 "components/offline_pages/client_policy_controller.h"
 
 #include <utility>
 
 #include "base/time/time.h"
 #include "components/offline_pages/client_namespace_constants.h"
@@ skipping 15 matching lines @@
                                   kUnlimitedPages)));
   policies_.insert(std::make_pair(
       kAsyncNamespace, MakePolicy(kAsyncNamespace, LifetimeType::PERSISTENT,
                                   base::TimeDelta::FromDays(2), kUnlimitedPages,
                                   kUnlimitedPages)));
   policies_.insert(std::make_pair(
       kCCTNamespace,
       MakePolicy(kCCTNamespace, LifetimeType::TEMPORARY,
                  base::TimeDelta::FromDays(2), kUnlimitedPages, 1)));
 
+  policies_.insert(std::make_pair(
+      kDownloadNamespace, MakePolicy(kDownloadNamespace,
+                                     LifetimeType::PERSISTENT,
+                                     base::TimeDelta::FromDays(2),

[fgorski, 2016/08/04 06:05:41]

this value is/should be irrelevant for PERSISTENT namespaces, could you try
setting it o 0?

[chili, 2016/08/04 06:53:28]

Done.

+                                     kUnlimitedPages,
+                                     kUnlimitedPages)));
+
   // Fallback policy.
   policies_.insert(std::make_pair(
       kDefaultNamespace, MakePolicy(kDefaultNamespace, LifetimeType::TEMPORARY,
                                     base::TimeDelta::FromDays(1), 10, 1)));
 }
 
 ClientPolicyController::~ClientPolicyController() {}
 
 // static
 const OfflinePageClientPolicy ClientPolicyController::MakePolicy(
@@ skipping 11 matching lines @@
 const OfflinePageClientPolicy& ClientPolicyController::GetPolicy(
     const std::string& name_space) const {
   const auto& iter = policies_.find(name_space);
   if (iter != policies_.end())
     return iter->second;
   // Fallback when the namespace isn't defined.
   return policies_.at(kDefaultNamespace);
 }
 
 }  // namespace offline_pages