[Offline pages] Use the new policy bits

components/ntp_snippets/offline_pages/offline_page_suggestions_provider.cc

Index: components/ntp_snippets/offline_pages/offline_page_suggestions_provider.cc
diff --git a/components/ntp_snippets/offline_pages/offline_page_suggestions_provider.cc b/components/ntp_snippets/offline_pages/offline_page_suggestions_provider.cc
index 3c1b0e3e58017d788e065c94326f0ccf79613398..4e140eabd60d56c24feff3a48b3fe7b4de842208 100644
--- a/components/ntp_snippets/offline_pages/offline_page_suggestions_provider.cc
+++ b/components/ntp_snippets/offline_pages/offline_page_suggestions_provider.cc
@@ -15,6 +15,7 @@
 #include "base/values.h"
 #include "components/ntp_snippets/pref_names.h"
 #include "components/offline_pages/client_namespace_constants.h"
+#include "components/offline_pages/client_policy_controller.h"
 #include "components/prefs/pref_registry_simple.h"
 #include "components/prefs/pref_service.h"
 #include "grit/components_strings.h"
@@ -43,9 +44,8 @@ bool IsRecentTab(const offline_pages::ClientId& client_id) {
 }
 
 bool IsDownload(const offline_pages::ClientId& client_id) {
-  // TODO(pke): Use kDownloadNamespace once the OfflinePageModel uses that.
-  // The current logic is taken from DownloadUIAdapter::IsVisibleInUI.
-  return client_id.name_space == offline_pages::kAsyncNamespace &&
+  return offline_pages::ClientPolicyController().IsSupportedByDownload(

[Dmitry Titov, 2016/09/20 18:51:55]

This is unorthodox way to use a constructor. A better way would be if either
those methods were static, so you would do CPC::IsFoo(..), or use the existing
OfflinePageModel::getPolicyController(). 
I think calling a ctor w/o 'new' is a dangerous thing...  

+             client_id.name_space) &&
          base::IsValidGUID(client_id.id);
 }