Attaches OfflineContentAggregator to Profile

components/offline_items_collection/content/offline_content_aggregator_factory.cc

+// Copyright 2017 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_items_collection/content/offline_content_aggregator_factory.h"
+
+#include "base/memory/singleton.h"
+#include "components/keyed_service/content/browser_context_dependency_manager.h"
+#include "components/offline_items_collection/core/offline_content_aggregator.h"
+#include "content/public/browser/browser_context.h"
+
+namespace offline_items_collection {
+
+// static
+OfflineContentAggregatorFactory*
+OfflineContentAggregatorFactory::GetInstance() {
+  return base::Singleton<OfflineContentAggregatorFactory>::get();
+}
+
+// static
+OfflineContentAggregator* OfflineContentAggregatorFactory::GetForBrowserContext(
+    content::BrowserContext* context) {
+  DCHECK(!context->IsOffTheRecord());

[Dmitry Titov, 2017/03/08 00:46:00]

It is unclear if you want the service to be created for Incognito or not. The
override of GetBrowserContextToUse seems to skip the default implementation
which returns nullptr for Incognito profile. So it looks one part of the code
wants to specifically enable Incognito profile while another is DCHECK'ing
against it.

[David Trainor- moved to gerrit, 2017/03/09 18:44:40]

Ah nice catch sorry!  I was messing around with the ordering and mixed it up. 
Thanks :).  Looks like I'll have to move this class out of the component and
into chrome/.  I think the only way to grab the original BrowserContext is by
using GetBrowserContextRedirectedInIncognito().

+  return static_cast<offline_items_collection::OfflineContentAggregator*>(
+      GetInstance()->GetServiceForBrowserContext(context, true));
+}
+
+OfflineContentAggregatorFactory::OfflineContentAggregatorFactory()
+    : BrowserContextKeyedServiceFactory(
+          "offline_items_collection::OfflineContentAggregator",
+          BrowserContextDependencyManager::GetInstance()) {}
+
+OfflineContentAggregatorFactory::~OfflineContentAggregatorFactory() = default;
+
+KeyedService* OfflineContentAggregatorFactory::BuildServiceInstanceFor(
+    content::BrowserContext* context) const {
+  return new offline_items_collection::OfflineContentAggregator();
+}
+
+content::BrowserContext*
+OfflineContentAggregatorFactory::GetBrowserContextToUse(
+    content::BrowserContext* context) const {
+  return context;
+}
+
+}  // namespace offline_items_collection