predictors: Introduce GlowplugPredictor.

chrome/browser/predictors/glowplug_predictor_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 "chrome/browser/predictors/glowplug_predictor_factory.h"
+
+#include "chrome/browser/history/history_service_factory.h"
+#include "chrome/browser/predictors/glowplug_predictor.h"
+#include "chrome/browser/predictors/predictor_database_factory.h"
+#include "chrome/browser/predictors/resource_prefetch_common.h"
+#include "chrome/browser/profiles/profile.h"
+#include "components/keyed_service/content/browser_context_dependency_manager.h"
+#include "components/keyed_service/content/browser_context_keyed_service_factory.h"
+
+namespace predictors {
+
+// static
+GlowplugPredictor* GlowplugPredictorFactory::GetForProfile(

[alexilin, 2017/05/11 16:53:12]

Shouldn't we rename it to GetForBrowserContext?

[Benoit L, 2017/05/12 12:12:53]

It seems that there is no consistent convention in the code base, with
GetForProfile() taking either a profile or a BrowserContext...
And also quite a few GetForBrowserContext().

Changed the parameter to Profile.

+    content::BrowserContext* context) {
+  return static_cast<GlowplugPredictor*>(
+      GetInstance()->GetServiceForBrowserContext(context, true));
+}
+
+// static
+GlowplugPredictorFactory* GlowplugPredictorFactory::GetInstance() {
+  return base::Singleton<GlowplugPredictorFactory>::get();
+}
+
+GlowplugPredictorFactory::GlowplugPredictorFactory()
+    : BrowserContextKeyedServiceFactory(
+          "GlowplugPredictor",
+          BrowserContextDependencyManager::GetInstance()) {
+  DependsOn(HistoryServiceFactory::GetInstance());
+  DependsOn(PredictorDatabaseFactory::GetInstance());
+}
+
+GlowplugPredictorFactory::~GlowplugPredictorFactory() {}
+
+KeyedService* GlowplugPredictorFactory::BuildServiceInstanceFor(
+    content::BrowserContext* context) const {
+  Profile* profile = Profile::FromBrowserContext(context);
+
+  GlowplugPredictorConfig config;
+  if (!IsGlowplugEnabled(profile, &config))
+    return nullptr;
+
+  return new GlowplugPredictor(config, profile);
+}
+
+}  // namespace predictors