MD Settings: disable cursor: pointer when sync is disabled

chrome/browser/resources/settings/people_page/people_page.js

Index: chrome/browser/resources/settings/people_page/people_page.js
diff --git a/chrome/browser/resources/settings/people_page/people_page.js b/chrome/browser/resources/settings/people_page/people_page.js
index 78452d1b7c6cd308a711b4bc0d1940261ff0cbfe..4a5ac48e2dc20c50bfc9810cfccf4b200bb6dd50 100644
--- a/chrome/browser/resources/settings/people_page/people_page.js
+++ b/chrome/browser/resources/settings/people_page/people_page.js
@@ -310,6 +310,15 @@ Polymer({
   },
 
   /**
+   * @param {?settings.SyncStatus} syncStatus
+   * @return {boolean}
+   * @private
+   */
+  isSyncEnabled_: function(syncStatus) {
+    return !(syncStatus && syncStatus.managed);

[tommycli, 2016/07/26 23:51:13]

It seems that this should be:

!syncStatus || syncStatus.managed

It seems that if we haven't retrieved the sync status it should also be not
actionable yet right?

[tommycli, 2016/07/26 23:51:41]

Err.. i guess actually:

syncStatus && !syncStatus.managed.

+  },
+
+  /**
    * @private
    * @param {?settings.SyncStatus} syncStatus
    * @return {string}