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

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

Issue 8637001: [NTP4] Auto-deletion of empty apps panes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: adding back erroneously removed ; Created 9 years 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/new_tab.js
diff --git a/chrome/browser/resources/ntp4/new_tab.js b/chrome/browser/resources/ntp4/new_tab.js
index c4c7766693bc22e7a97e03c91720d08f2db31107..ff7936cdc84d3fe52dafb70f71a314f2cb561e5c 100644
--- a/chrome/browser/resources/ntp4/new_tab.js
+++ b/chrome/browser/resources/ntp4/new_tab.js
@@ -325,40 +325,48 @@ cr.define('ntp4', function() {
/**
* Wrappers to forward the callback to corresponding PageListView member.
*/
- function appAdded(appData, opt_highlight) {
- newTabView.appAdded(appData, opt_highlight);
+ function appAdded() {
+ return newTabView.appAdded.apply(newTabView, arguments);
}
- function appRemoved(appData, isUninstall) {
- newTabView.appRemoved(appData, isUninstall);
+ function appsPrefChangeCallback() {
+ return newTabView.appsPrefChangedCallback.apply(newTabView, arguments);
}
- function appsPrefChangeCallback(data) {
- newTabView.appsPrefChangedCallback(data);
+ function appRemoved() {
+ return newTabView.appRemoved.apply(newTabView, arguments);
+ }
+
+ function appsReordered() {
+ return newTabView.appsReordered.apply(newTabView, arguments);
}
function enterRearrangeMode() {
- newTabView.enterRearrangeMode();
+ return newTabView.enterRearrangeMode.apply(newTabView, arguments);
}
- function getAppsCallback(data) {
- newTabView.getAppsCallback(data);
+ function getAppsCallback() {
+ return newTabView.getAppsCallback.apply(newTabView, arguments);
}
- function getAppsPageIndex(page) {
- return newTabView.getAppsPageIndex(page);
+ function getAppsPageIndex() {
+ return newTabView.getAppsPageIndex.apply(newTabView, arguments);
}
function getCardSlider() {
return newTabView.cardSlider;
}
- function leaveRearrangeMode(e) {
- newTabView.leaveRearrangeMode(e);
+ function leaveRearrangeMode() {
+ return newTabView.leaveRearrangeMode.apply(newTabView, arguments);
+ }
+
+ function removeAppsPageIfEmpty() {
+ return newTabView.removeAppsPageIfEmpty.apply(newTabView, arguments);
}
- function saveAppPageName(appPage, name) {
- newTabView.saveAppPageName(appPage, name);
+ function saveAppsPageName() {
+ return newTabView.saveAppsPageName.apply(newTabView, arguments);
}
function setAppToBeHighlighted(appId) {
@@ -369,6 +377,7 @@ cr.define('ntp4', function() {
return {
appAdded: appAdded,
appRemoved: appRemoved,
+ appsReordered: appsReordered,
appsPrefChangeCallback: appsPrefChangeCallback,
enterRearrangeMode: enterRearrangeMode,
getAppsCallback: getAppsCallback,
@@ -376,7 +385,8 @@ cr.define('ntp4', function() {
getCardSlider: getCardSlider,
onLoad: onLoad,
leaveRearrangeMode: leaveRearrangeMode,
- saveAppPageName: saveAppPageName,
+ removeAppsPageIfEmpty: removeAppsPageIfEmpty,
+ saveAppsPageName: saveAppsPageName,
setAppToBeHighlighted: setAppToBeHighlighted,
setMostVisitedPages: setMostVisitedPages,
setRecentlyClosedTabs: setRecentlyClosedTabs,

Powered by Google App Engine
This is Rietveld 408576698