Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4490)

Unified Diff: chrome/browser/resources/settings/people_page/password_prompt_dialog.js

Issue 2313103002: Added uma for pin unlock set up. (Closed)
Patch Set: Closure compiler fix. Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/settings/people_page/password_prompt_dialog.js
diff --git a/chrome/browser/resources/settings/people_page/password_prompt_dialog.js b/chrome/browser/resources/settings/people_page/password_prompt_dialog.js
index 7dae95142d97ae14d155c4d897468b608f94a32a..0ff12bbb62bfcd541e3a35df50a34ad42d94609b 100644
--- a/chrome/browser/resources/settings/people_page/password_prompt_dialog.js
+++ b/chrome/browser/resources/settings/people_page/password_prompt_dialog.js
@@ -72,6 +72,18 @@ Polymer({
},
/**
+ * writeUma_ is a function that handles writing uma stats. It may be
+ * overridden for tests.
+ *
+ * @type {Function}
+ * @private
+ */
+ writeUma_: {
+ type: Object,
+ value: function() { return settings.recordLockScreenProgress; }
+ },
+
+ /**
* PASSWORD_ACTIVE_DURATION_MS value. May be overridden by tests.
* @private
*/
@@ -96,6 +108,7 @@ Polymer({
if (this.$.dialog.open)
return;
+ this.writeUma_(LockScreenProgress.START_SCREEN_LOCK);
this.$.dialog.showModal();
},
@@ -163,6 +176,8 @@ Polymer({
this.password_ = '';
if (this.$.dialog.open)
this.$.dialog.close();
+
+ this.writeUma_(LockScreenProgress.ENTER_PASSWORD_CORRECTLY);
}
}

Powered by Google App Engine
This is Rietveld 408576698