Commit 4aa87e1c authored by gogerald's avatar gogerald Committed by Commit bot

Add Signin_Impression_FromSettings user action for Desktops

BUG=639860
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2339723002
Cr-Commit-Position: refs/heads/master@{#419270}
parent cca1bac5
......@@ -214,6 +214,10 @@ cr.define('options', function() {
// Sync (Sign in) section.
this.updateSyncState_(/** @type {options.SyncStatus} */(
loadTimeData.getValue('syncData')));
if (!$('sync-overview').hidden) {
chrome.send('metricsHandler:recordAction',
['Signin_Impression_FromSettings']);
}
$('start-stop-sync').onclick = function(event) {
if (self.signedIn_) {
......
......@@ -184,6 +184,9 @@ Polymer({
* @private
*/
handleSyncStatus_: function(syncStatus) {
if (!this.syncStatus && syncStatus && !syncStatus.signedIn) {
chrome.metricsPrivate.recordUserAction('Signin_Impression_FromSettings');
}
this.syncStatus = syncStatus;
},
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment