From 398d1a99e87fb3f57f6330616f5aa1fd494faced Mon Sep 17 00:00:00 2001 From: "aa@chromium.org" Date: Mon, 20 Dec 2010 21:47:24 +0000 Subject: Do not show apps section until default apps are installed. The referenced bugs were side-effects of this underlying issue. BUG=67073,67075 TEST=Run chrome on new profile. Apps section should not show up initially. Go to chrome://extensions/ -> developer mode, and click 'update now'. Apps should show up along with promo. Uninstall all apps. Apps section should keep showing up. Review URL: http://codereview.chromium.org/5958002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69758 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/resources/ntp/apps.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'chrome/browser/resources/ntp/apps.js') diff --git a/chrome/browser/resources/ntp/apps.js b/chrome/browser/resources/ntp/apps.js index eef4fa2..32b8a4a 100644 --- a/chrome/browser/resources/ntp/apps.js +++ b/chrome/browser/resources/ntp/apps.js @@ -66,7 +66,9 @@ function getAppsCallback(data) { apps.createWebStoreClosedMenuElement()); } - if (!(shownSections & MINIMIZED_APPS)) { + if (!data.showLauncher || (shownSections & MINIMIZED_APPS)) { + appsSection.classList.add('disabled'); + } else { appsSection.classList.remove('disabled'); } addClosedMenuFooter(apps.menu, 'apps', MINIMIZED_APPS, Section.APPS); -- cgit v1.1