Gets the initial sync service status. In the next patch I'll make the syncfs-internals dashboard a …

chrome/browser/resources/sync_file_system_internals/sync_service.js

+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+/**
+ * WebUI to monitor the Sync File System Service.
+ */
+var syncService = (function() {
+'use strict';
+
+/**
+ * Request Sync Service Status.
+ * Asynchronous, result via getServiceStatusResult.
+ */
+function getServiceStatus() {
+  chrome.send('getServiceStatus');
+}
+
+/**
+ * Handles callback from getServiceStatus.
+ * @param {string} Service status enum as a string.
+ */
+function getServiceStatusResult(status_string) {

[arv (Not doing code reviews), 2013/05/02 19:45:06]

no underscores in js

[calvinlo, 2013/05/07 06:14:10]

Done. Changed to camel case.

+  console.log('getServiceStatusResult', status_string);
+  $('service-status').innerText = status_string;

[arv (Not doing code reviews), 2013/05/02 19:45:06]

textContent

[calvinlo, 2013/05/07 06:14:10]

Done.

+}
+
+/**
+ * Get initial sync service values and set listeners to get updated values.
+ */
+function main() {
+  cr.ui.decorate('tabbox', cr.ui.TabBox);
+  getServiceStatus();
+}
+
+return {
+  main: main,
+  getServiceStatusResult: getServiceStatusResult
+};
+
+})();
+
+document.addEventListener('DOMContentLoaded', syncService.main);

[arv (Not doing code reviews), 2013/05/02 19:45:06]

move into IIFE and skip syncService

document.addEventListener('DOMContentLoaded', main);

Then you probably do not need to export main

[calvinlo, 2013/05/07 06:14:10]

Done.