Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5248)

Unified Diff: chrome/browser/resources/ntp4/trash.js

Issue 7592001: ntp4: most visited dragging onto apps page (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: most visited polish Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/ntp4/trash.js
diff --git a/chrome/browser/resources/ntp4/trash.js b/chrome/browser/resources/ntp4/trash.js
index e554e2eceb0e09a86a23b363e2d5a4e586e998b6..4db841db50b3b226c406ed7eed5ec8ab8cf7d27b 100644
--- a/chrome/browser/resources/ntp4/trash.js
+++ b/chrome/browser/resources/ntp4/trash.js
@@ -33,11 +33,7 @@ cr.define('ntp4', function() {
if (!tile)
return false;
- var app = tile.querySelector('.app');
- if (!app)
- return false;
-
- return app.appData.can_uninstall;
+ return tile.firstChild.canBeRemoved();
},
/**
@@ -67,15 +63,8 @@ cr.define('ntp4', function() {
e.preventDefault();
var tile = ntp4.getCurrentlyDraggingTile();
- var app = tile.querySelector('.app');
- if (!app)
- return;
-
- chrome.send('uninstallApp', [app.appData.id, true]);
-
- var page = tile.tilePage;
- tile.parentNode.removeChild(tile);
- page.cleanupDrag();
+ tile.firstChild.removeFromChrome();
+ tile.landedOnTrash = true;
},
/**
« chrome/browser/resources/ntp4/tile_page.js ('K') | « chrome/browser/resources/ntp4/tile_page.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698