ntp4: record app launch source

chrome/browser/resources/ntp4/apps_page.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.
 
 cr.define('ntp4', function() {
   'use strict';
 
   var localStrings = new LocalStrings;
 
   var APP_LAUNCH = {
@@ skipping 176 matching lines @@
       // image when it's touched.
       appImg.style.WebkitMaskImage = url(this.appData_.icon_big);
       // We put a click handler just on the app image - so clicking on the
       // margins between apps doesn't do anything.
       appImg.addEventListener('click', this.onClick_.bind(this));
       appContents.appendChild(appImg);
       this.appImg_ = appImg;
 
       var appSpan = this.ownerDocument.createElement('span');
       appSpan.textContent = this.appData_.name;
+      appSpan.addEventListener('click', this.onClick_.bind(this));

[Rick Byers, 2011/08/03 14:41:11]

Why add a handler to both the img and span.  Would it be equivalent to put just
a single handler on the appContents?

[Evan Stade, 2011/08/03 22:11:22]

the clickable contour is a little different

       appContents.appendChild(appSpan);
       this.appendChild(appContents);
 
       this.addEventListener('keydown', cr.ui.contextMenuHandler);
       this.addEventListener('keyup', cr.ui.contextMenuHandler);
 
       // This hack is here so that appContents.contextMenu will be the same as
       // this.contextMenu.
       var self = this;
       appContents.__defineGetter__('contextMenu', function() {
@@ skipping 222 matching lines @@
     },
   };
 
   AppsPage.setPromo = function(data) {
     var store = document.querySelector('.webstore');
     if (store)
       store.setAppsPromoData(data);
   };
 
   return {
+    APP_LAUNCH: APP_LAUNCH,
     AppsPage: AppsPage,
   };
 });