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

Unified Diff: chrome/browser/resources/settings/people_page/lock_screen.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/lock_screen.js
diff --git a/chrome/browser/resources/settings/people_page/lock_screen.js b/chrome/browser/resources/settings/people_page/lock_screen.js
index 762bd2cb8a426c762e429159415f752e82664d04..73fcdb622621f0f294b54891c40a5d3a0a885fd0 100644
--- a/chrome/browser/resources/settings/people_page/lock_screen.js
+++ b/chrome/browser/resources/settings/people_page/lock_screen.js
@@ -37,6 +37,18 @@ Polymer({
type: Object,
observer: 'onSetModesChanged_'
},
+
+ /**
+ * 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; }
+ }
},
/** selectedUnlockType is defined in LockStateBehavior. */
@@ -125,5 +137,6 @@ Polymer({
onConfigurePin_: function(e) {
e.preventDefault();
this.$.setupPin.open();
+ this.writeUma_(LockScreenProgress.CHOOSE_PIN_OR_PASSWORD);
},
});

Powered by Google App Engine
This is Rietveld 408576698