ntp4: drag from bookmark page to apps page

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 624 matching lines @@
     /** @inheritDoc */
     addDragData: function(dataTransfer, index) {
       var currentlyDraggingTile = ntp4.getCurrentlyDraggingTile();
       if (currentlyDraggingTile) {
         var tileContents = currentlyDraggingTile.firstChild;
         if (tileContents.classList.contains('app')) {
           this.tileGrid_.insertBefore(
               currentlyDraggingTile,
               this.tileElements_[index]);
           this.tileMoved(currentlyDraggingTile);
-        } else if (currentlyDraggingTile.querySelector('.most-visited')) {
+        } else if (currentlyDraggingTile.querySelector('.most-visited') ||
+                   currentlyDraggingTile.querySelector('.bookmark')) {

[Dan Beam, 2011/09/22 22:36:01]

like we talked about on chat, might as well add a class in 1 .querySelector()
string instead of 2 calls (not huge diff, but doesn't hurt).

           this.generateAppForLink(tileContents.data);
         }
       } else {
         this.addOutsideData_(dataTransfer, index);
       }
     },
 
     /**
      * Adds drag data that has been dropped from a source that is not a tile.
      * @param {Object} dataTransfer The data transfer object that holds drop
@@ skipping 93 matching lines @@
     appNotificationChanged: appNotificationChanged,
     AppsPage: AppsPage,
     launchAppAfterEnable: launchAppAfterEnable,
   };
 });
 
 // TODO(estade): update the content handlers to use ntp namespace instead of
 // making these global.
 var appNotificationChanged = ntp4.appNotificationChanged;
 var launchAppAfterEnable = ntp4.launchAppAfterEnable;