Add Signin_Impression_FromSettings user action for Desktops

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 42622f135709d3beeff493f5414155ac4a7990df..16daf436a3ac8045e6325aa095019d4ec0706a62 100644
--- a/chrome/browser/resources/settings/people_page/people_page.js
+++ b/chrome/browser/resources/settings/people_page/people_page.js
@@ -184,6 +184,9 @@ Polymer({
    * @private
    */
   handleSyncStatus_: function(syncStatus) {
+    if (this.syncStatus == null && !syncStatus.signedIn) {

[Dan Beam, 2016/09/16 18:03:03]

if (!this.syncStatus && syncStatus && !syncStatus.signedIn)
  chrome.metricsPrivate.recordUserAction('Signin_Impression_FromSettings');

[gogerald1, 2016/09/16 20:18:07]

Done.

[Dan Beam, 2016/09/16 21:46:36]

for future reference, I meant not to use curly braces (like the rest of the
code), which is why I wrote it the way I did

+      chrome.metricsPrivate.recordUserAction('Signin_Impression_FromSettings');
+    }
     this.syncStatus = syncStatus;
   },