Adding an experimental app notification API.

chrome/browser/resources/ntp/apps.js

 // 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.
 
 var MAX_APPS_PER_ROW = [];
 MAX_APPS_PER_ROW[LayoutMode.SMALL] = 4;
 MAX_APPS_PER_ROW[LayoutMode.NORMAL] = 6;
 
 function getAppsCallback(data) {
   logEvent('received apps');
@@ skipping 131 matching lines @@
 
 function appsPrefChangeCallback(data) {
   // Currently the only pref that is watched is the launch type.
   data.apps.forEach(function(app) {
     var appLink = document.querySelector('.app a[app-id=' + app['id'] + ']');
     if (appLink)
       appLink.setAttribute('launch-type', app['launch_type']);
   });
 }
 
+function appsNotificationChangeCallback(id, last_notification) {

[Evan Stade, 2011/06/24 17:29:20]

nits:

lastNotification instead of last_notification

also, this fn should be called appNotificationChanged (I think the plural is
wrong, and the Callback is unnecessary even though appsPrefChangeCallbackoddly 
uses it)

[asargent_no_longer_on_chrome, 2011/06/24 20:02:54]

Done.

+  // TODO(asargent/finnur) use this when we hook up notifications into the NTP.
+}
+
 // Launches the specified app using the APP_LAUNCH_NTP_APP_RE_ENABLE histogram.
 // This should only be invoked from the AppLauncherHandler.
 function launchAppAfterEnable(appId) {
   chrome.send('launchApp', [appId, APP_LAUNCH.NTP_APP_RE_ENABLE]);
 }
 
 var apps = (function() {
 
   function createElement(app) {
     var div = document.createElement('div');
@@ skipping 619 matching lines @@
       a.href = localStrings.getString('web_store_url');
       a.style.backgroundImage = url('chrome://theme/IDR_PRODUCT_LOGO_16');
       a.className = 'item';
       return a;
     }
   };
 })();
 
 // Enable drag and drop reordering of the app launcher.
 var appDragAndDrop = new DragAndDropController(apps);