Add a triggered profile reset mechanism.

chrome/browser/profile_resetter/triggered_profile_resetter_factory.cc

+// Copyright 2013 The Chromium Authors. All rights reserved.

[grt (UTC plus 2), 2015/09/05 12:44:14]

2015

[robertshield, 2015/09/13 06:04:01]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/profile_resetter/triggered_profile_resetter_factory.h"
+
+#include "base/memory/singleton.h"
+#include "base/prefs/pref_registry_simple.h"

[grt (UTC plus 2), 2015/09/05 12:44:14]

is this needed given that you include pref_registry_syncable?

[robertshield, 2015/09/13 06:04:01]

Done.

+#include "chrome/browser/profile_resetter/triggered_profile_resetter.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/search_engines/template_url_service_factory.h"

[grt (UTC plus 2), 2015/09/05 12:44:14]

unused

[robertshield, 2015/09/13 06:04:01]

Done.

+#include "chrome/browser/ui/global_error/global_error_service_factory.h"

[grt (UTC plus 2), 2015/09/05 12:44:14]

unused

[robertshield, 2015/09/13 06:04:01]

Done.

+#include "chrome/common/pref_names.h"
+#include "components/keyed_service/content/browser_context_dependency_manager.h"
+#include "components/pref_registry/pref_registry_syncable.h"
+#include "content/public/browser/browser_context.h"
+
+// static
+TriggeredProfileResetter* TriggeredProfileResetterFactory::GetForBrowserContext(
+    content::BrowserContext* context) {
+  return static_cast<TriggeredProfileResetter*>(
+      GetInstance()->GetServiceForBrowserContext(context, true));
+}
+
+// static
+TriggeredProfileResetterFactory*
+TriggeredProfileResetterFactory::GetInstance() {
+  return Singleton<TriggeredProfileResetterFactory>::get();
+}
+
+// static
+void TriggeredProfileResetterFactory::SetGlobalTestingFactory(
+    TestingFactoryFunction testing_factory) {
+  GetInstance()->testing_factory_function_ = testing_factory;
+}
+
+TriggeredProfileResetterFactory::TriggeredProfileResetterFactory()
+    : BrowserContextKeyedServiceFactory(
+          "TriggeredProfileResetter",
+          BrowserContextDependencyManager::GetInstance()) {}
+
+TriggeredProfileResetterFactory::~TriggeredProfileResetterFactory() {}
+
+KeyedService* TriggeredProfileResetterFactory::BuildServiceInstanceFor(
+    content::BrowserContext* context) const {
+  if (testing_factory_function_)
+    return testing_factory_function_(context).release();

[grt (UTC plus 2), 2015/09/05 12:44:14]

to fulfull the contract of this function even in tests, why not call Activate()
on the test object?

[robertshield, 2015/09/13 06:04:01]

Done.

+
+  Profile* profile = Profile::FromBrowserContext(context);
+  TriggeredProfileResetter* service = new TriggeredProfileResetter(profile);
+  service->Activate();
+  return service;
+}
+
+void TriggeredProfileResetterFactory::RegisterProfilePrefs(
+    user_prefs::PrefRegistrySyncable* registry) {
+#if defined(OS_WIN)
+  registry->RegisterInt64Pref(prefs::kLastProfileResetTimestamp, 0L);
+#endif
+}
+
+bool TriggeredProfileResetterFactory::ServiceIsCreatedWithBrowserContext()
+    const {
+  return true;
+}