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

Unified Diff: chrome/test/data/webui/ntp4.js

Issue 10141005: switch ntp to jstemplate v2 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix test once and for all Created 8 years, 8 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
« no previous file with comments | « chrome/common/jstemplate_builder.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webui/ntp4.js
diff --git a/chrome/test/data/webui/ntp4.js b/chrome/test/data/webui/ntp4.js
index a4a020802584e0f437a281a5dad65bf64fadedf7..378d91f376e4bbd9752a81cbfa89f7bb6e56e88b 100644
--- a/chrome/test/data/webui/ntp4.js
+++ b/chrome/test/data/webui/ntp4.js
@@ -36,7 +36,7 @@ TEST_F('NTP4WebUITest', 'FLAKY_NTPHasThumbnails', function() {
assertEquals(8, mostVisited.length, 'There should be 8 most visited tiles.');
var apps = document.querySelectorAll('.app');
- if (window.templateData.showApps)
+ if (loadTimeData.getBoolean('showApps'))
assertGE(apps.length, 1, 'There should be at least one app.');
else
assertEquals(0, apps.length, 'There should be no apps.');
@@ -44,7 +44,7 @@ TEST_F('NTP4WebUITest', 'FLAKY_NTPHasThumbnails', function() {
TEST_F('NTP4WebUITest', 'NTPHasNavDots', function() {
var navDots = document.querySelectorAll('.dot');
- if (window.templateData.showApps)
+ if (loadTimeData.getBoolean('showApps'))
assertGE(navDots.length, 2, 'There should be at least two navdots.');
else
assertEquals(1, navDots.length, 'There should be exactly one navdot.');
« no previous file with comments | « chrome/common/jstemplate_builder.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698