Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1050)

Unified Diff: chrome/browser/extensions/default_apps.h

Issue 3522015: Implement new strategy for default apps (Closed)
Patch Set: all done Created 10 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/extensions/crx_installer.cc ('k') | chrome/browser/extensions/default_apps.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/default_apps.h
diff --git a/chrome/browser/extensions/default_apps.h b/chrome/browser/extensions/default_apps.h
new file mode 100644
index 0000000000000000000000000000000000000000..f61539ffd50fff8da428a86fbb2d0d5e6526da44
--- /dev/null
+++ b/chrome/browser/extensions/default_apps.h
@@ -0,0 +1,83 @@
+// Copyright (c) 2010 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_H_
+#define CHROME_BROWSER_EXTENSIONS_DEFAULT_APPS_H_
+#pragma once
+
+#include <set>
+#include <string>
+#include "chrome/common/extensions/extension.h"
+#include "base/gtest_prod_util.h"
+
+class PrefService;
+
+// Manages the installation of the set of default apps into Chrome, and the
+// promotion of those apps in the launcher.
+//
+// It implements the following rules:
+//
+// - Only install default apps once per-profile.
+// - Don't install default apps if any apps are already installed.
+// - Do not start showing the promo until all default apps have been installed.
+// - Do not show the promo if it has been hidden by the user.
+// - Do not show promo after one app has been manually installed or uninstalled.
+// - Do not show promo if the set of installed apps is different than the set of
+// default apps.
+// - Only show promo a certain amount of times.
+//
+// The promo can also be forced on with --force-apps-promo-visible.
+class DefaultApps {
+ public:
+ // The maximum number of times to show the apps promo.
+ static const int kAppsPromoCounterMax;
+
+ // Register our preferences.
+ static void RegisterUserPrefs(PrefService* prefs);
+
+ explicit DefaultApps(PrefService* prefs);
+
+ // Gets the list of default apps that should be installed. Can return NULL if
+ // no apps need to be installed.
+ const ExtensionIdSet* GetAppsToInstall() const;
+
+ // Should be called after each app is installed. Once installed_ids contains
+ // all the default apps, GetAppsToInstall() will start returning NULL.
+ void DidInstallApp(const ExtensionIdSet& installed_ids);
+
+ // Returns true if the apps promo should be displayed in the launcher. This
+ // starts returning true once the default apps have all been installed and
+ // stops after the promo expires.
+ bool ShouldShowPromo(const ExtensionIdSet& installed_ids);
+
+ // Should be called after each time the promo is installed.
+ void DidShowPromo();
+
+ // Force the promo to be hidden.
+ void SetPromoHidden();
+
+ private:
+ FRIEND_TEST_ALL_PREFIXES(DefaultApps, Basics);
+ FRIEND_TEST_ALL_PREFIXES(DefaultApps, HidePromo);
+ FRIEND_TEST_ALL_PREFIXES(DefaultApps, InstallingAnAppHidesPromo);
+ FRIEND_TEST_ALL_PREFIXES(DefaultApps,
+ ManualAppInstalledWhileInstallingDefaultApps);
+
+ bool GetDefaultAppsInstalled() const;
+ void SetDefaultAppsInstalled(bool val);
+
+ int GetPromoCounter() const;
+ void SetPromoCounter(int val);
+
+ // Our permanent state is stored in this PrefService instance.
+ PrefService* prefs_;
+
+ // The set of default extensions. Initialized to a static list in the
+ // constructor.
+ ExtensionIdSet ids_;
+
+ DISALLOW_COPY_AND_ASSIGN(DefaultApps);
+};
+
+#endif // CHROME_BROWSER_EXTENSIONS_DEFAULT_APPS_H_
« no previous file with comments | « chrome/browser/extensions/crx_installer.cc ('k') | chrome/browser/extensions/default_apps.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698