diff options
| author | aa@chromium.org <aa@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-12-14 20:14:12 +0000 |
|---|---|---|
| committer | aa@chromium.org <aa@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-12-14 20:14:12 +0000 |
| commit | cefe5c46afeb6e48e15a396435ca13274406004d (patch) | |
| tree | d851d2b162561286fb86308415648ee6eee44984 /chrome/browser/extensions/default_apps.cc | |
| parent | 9ef20a191ca73d574ac9f9d5208e8de814622aa7 (diff) | |
| download | chromium_src-cefe5c46afeb6e48e15a396435ca13274406004d.zip chromium_src-cefe5c46afeb6e48e15a396435ca13274406004d.tar.gz chromium_src-cefe5c46afeb6e48e15a396435ca13274406004d.tar.bz2 | |
Default apps promo was getting expired early if NTP open.
This was a timing issue: we had a pref change observer that was
triggering refresh of NTP. Inside this refresh, we checked the
promo status. But setting up the pref tree of an extension
during installation was also triggering this listener.
BUG=64736
TEST=See bug
Review URL: http://codereview.chromium.org/5750005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69164 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/default_apps.cc')
| -rw-r--r-- | chrome/browser/extensions/default_apps.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/extensions/default_apps.cc b/chrome/browser/extensions/default_apps.cc index 43db8ba..a9e305f 100644 --- a/chrome/browser/extensions/default_apps.cc +++ b/chrome/browser/extensions/default_apps.cc @@ -52,7 +52,7 @@ void DefaultApps::DidInstallApp(const ExtensionIdSet& installed_ids) { } } -bool DefaultApps::ShouldShowPromo(const ExtensionIdSet& installed_ids) { +bool DefaultApps::CheckShouldShowPromo(const ExtensionIdSet& installed_ids) { #if defined(OS_CHROMEOS) // Don't show the promo at all on Chrome OS. return false; |