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

Unified Diff: chrome/browser/resources/sync_internals/notifications.js

Issue 10837111: [Sync] Print out notification state explicitly in about:sync (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix tests Created 8 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/sync_internals/notifications.js
diff --git a/chrome/browser/resources/sync_internals/notifications.js b/chrome/browser/resources/sync_internals/notifications.js
index 35dd94c75487c4af4c6491cee8a19a68efa910a3..af6cb3f59e7053725ea3faadf96104e31e94623e 100644
--- a/chrome/browser/resources/sync_internals/notifications.js
+++ b/chrome/browser/resources/sync_internals/notifications.js
@@ -6,11 +6,11 @@
// TODO(akalin): Use table.js.
-function updateNotificationsEnabledInfo(notificationsEnabled) {
- var notificationsEnabledInfo = $('notificationsEnabledInfo');
+function updateNotificationStateInfo(notificationState) {
+ var notificationStateInfo = $('notificationStateInfo');
jstProcess(
- new JsEvalContext({ 'notificationsEnabled': notificationsEnabled }),
- notificationsEnabledInfo);
+ new JsEvalContext({ 'notificationState': notificationState }),
+ notificationStateInfo);
}
// Contains all notification data. The keys are sync types (as strings) and
@@ -76,10 +76,10 @@ function updateNotificationInfo(notificationInfo) {
}
function onLoad() {
- chrome.sync.getNotificationState(updateNotificationsEnabledInfo);
+ chrome.sync.getNotificationState(updateNotificationStateInfo);
chrome.sync.getNotificationInfo(updateNotificationInfo);
chrome.sync.onNotificationStateChange.addListener(
- function(details) { updateNotificationsEnabledInfo(details.enabled); });
+ function(details) { updateNotificationStateInfo(details.state); });
chrome.sync.onIncomingNotification.addListener(function(details) {
var changedTypes = details.changedTypes;
« no previous file with comments | « chrome/browser/resources/sync_internals/notifications.html ('k') | chrome/browser/sync/profile_sync_service_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698