MD Settings: disable cursor: pointer when sync is disabled

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

 // Copyright 2015 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.
 
 /**
  * @fileoverview
  * 'settings-people-page' is the settings page containing sign-in settings.
  */
 Polymer({
   is: 'settings-people-page',
@@ skipping 292 matching lines @@
    * @private
    * @param {?settings.SyncStatus} syncStatus
    * @return {boolean}
    */
   isAdvancedSyncSettingsVisible_: function(syncStatus) {
     return !!syncStatus && !!syncStatus.signedIn &&
         !!syncStatus.syncSystemEnabled;
   },
 
   /**
+   * @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}
    */
   getSyncIcon_: function(syncStatus) {
     if (!syncStatus)
       return '';
     if (syncStatus.hasError)
       return 'settings:sync-problem';
     if (syncStatus.managed)
       return 'settings:sync-disabled';
 
     return 'settings:done';
   },
 });