Chromium Code Reviews| 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 0fa946c3c41bec271769be9ca83af09b10e5b438..2a6c63bc4ed55e958f4d526b4b40df80bfe9f47f 100644 |
| --- a/chrome/browser/resources/ntp4/new_tab.js |
| +++ b/chrome/browser/resources/ntp4/new_tab.js |
| @@ -110,6 +110,14 @@ cr.define('ntp', function() { |
| } |
| }; |
| + function gotShouldShowApps(shouldShowApps) { |
| + if (shouldShowApps != loadTimeData.getBoolean('showApps')) { |
| + // TODO(jeremya): update the UI in-place instead of reloading. |
| + window.location.reload(); |
| + return; |
| + } |
| + } |
| + |
| /** |
| * Invoked at startup once the DOM is available to initialize the app. |
| */ |
| @@ -628,6 +636,7 @@ cr.define('ntp', function() { |
| getAppsPageIndex: getAppsPageIndex, |
| getCardSlider: getCardSlider, |
| onLoad: onLoad, |
| + gotShouldShowApps: gotShouldShowApps, |
| leaveRearrangeMode: leaveRearrangeMode, |
| logTimeToClick: logTimeToClick, |
| NtpFollowAction: NtpFollowAction, |
| @@ -645,6 +654,8 @@ cr.define('ntp', function() { |
| }; |
| }); |
| +// This will end up calling ntp.gotShouldShowApps. |
| +chrome.send('getShouldShowApps'); |
|
Evan Stade
2013/01/23 23:34:24
this should probably be a part of onLoad
jeremya
2013/01/24 02:40:04
Why's that?
Evan Stade
2013/01/24 18:40:16
well, my thinking was that if gotShouldShowApps we
jeremya
2013/01/25 00:06:19
Right, that makes sense -- if we get the gotShould
|
| document.addEventListener('DOMContentLoaded', ntp.onLoad); |
| var toCssPx = cr.ui.toCssPx; |