Remove race condition when installing default apps into a new profile.

chrome/browser/extensions/default_apps_provider.h

Index: chrome/browser/extensions/default_apps_provider.h
===================================================================
--- chrome/browser/extensions/default_apps_provider.h	(revision 0)
+++ chrome/browser/extensions/default_apps_provider.h	(revision 0)
@@ -0,0 +1,43 @@
+// Copyright (c) 2011 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.
+
+#ifndef CHROME_BROWSER_EXTENSIONS_DEFAULT_APPS_PROVIDER_H_
+#define CHROME_BROWSER_EXTENSIONS_DEFAULT_APPS_PROVIDER_H_
+#pragma once
+
+#include "base/basictypes.h"
+
+class ExternalExtensionLoader;
+class PrefService;
+class Profile;
+
+namespace base {
+class DictionaryValue;
+}

[Finnur, 2011/10/21 09:59:32]

Are ExternalExtensionLoader and DictionaryValue needed anymore?

[Roger Tawa OOO till Jul 10th, 2011/10/24 14:23:14]

Done.

+
+// A provider of apps that are installed by default into all new profiles.
+class DefaultAppsProvider {
+ public:
+  // These values are persisted in the user preferences so values should not

[Finnur, 2011/10/21 09:59:32]

Wording nit: These values ... so values ...
Maybe talk about enums and values instead?

[Roger Tawa OOO till Jul 10th, 2011/10/24 14:23:14]

Done.

+  // be changed.
+  enum InstallState {
+    kUnknown,
+    kAlwaysProvideDefaultApps,
+    kNeverProvideDefaultApps
+  };
+
+  // Register preference properties used by DefaultAppsProvider to maintain
+  // install state.
+  static void RegisterUserPrefs(PrefService* prefs);
+
+  // Determines whether default apps should be installed into the specified
+  // profile.  If true, then an instance of DefaultAppsProvider should be
+  // added to the external providers list.
+  static bool ShouldRegister(Profile* profile);
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(DefaultAppsProvider);
+};
+
+#endif  // CHROME_BROWSER_EXTENSIONS_DEFAULT_APPS_PROVIDER_H_
Property changes on: chrome\browser\extensions\default_apps_provider.h
___________________________________________________________________
Added: svn:eol-style
   + LF