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

Unified Diff: chrome/browser/resources/local_discovery/local_discovery.js

Issue 23508003: Added UMA actions to devices page (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 months 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/local_discovery/local_discovery.js
diff --git a/chrome/browser/resources/local_discovery/local_discovery.js b/chrome/browser/resources/local_discovery/local_discovery.js
index e9491d620ab688b394c37e2e3b18f3f407f2b034..9c62618d6aa1bbad59706c9538e36562d1254f06 100644
--- a/chrome/browser/resources/local_discovery/local_discovery.js
+++ b/chrome/browser/resources/local_discovery/local_discovery.js
@@ -83,6 +83,7 @@ cr.define('local_discovery', function() {
* Register the device.
*/
register: function() {
+ recordUmaAction('DevicesPage_RegisterClicked');
chrome.send('registerDevice', [this.info.service_name]);
setRegisterPage('register-page-adding1');
}
@@ -139,9 +140,10 @@ cr.define('local_discovery', function() {
}
/**
- * Hide the register overlay.
+ * Show the register overlay.
*/
function showRegisterOverlay() {
+ recordUmaAction('DevicesPage_AddPrintersClicked');
$('register-overlay').classList.add('showing');
$('overlay').hidden = false;
uber.invokeMethodOnParent('beginInterceptingEvents');
@@ -149,13 +151,12 @@ cr.define('local_discovery', function() {
}
/**
- * Show the register overlay.
+ * Hide the register overlay.
*/
function hideRegisterOverlay() {
$('register-overlay').classList.remove('showing');
$('overlay').hidden = true;
uber.invokeMethodOnParent('stopInterceptingEvents');
- chrome.send('cancelRegistration');
}
/**
@@ -173,6 +174,7 @@ cr.define('local_discovery', function() {
*/
function onRegistrationFailed() {
setRegisterPage('register-page-error');
+ recordUmaAction('DevicesPage_RegisterFailure');
}
/**
@@ -265,6 +267,7 @@ cr.define('local_discovery', function() {
function onRegistrationSuccess() {
hideRegisterOverlay();
requestPrinterList();
+ recordUmaAction('DevicesPage_RegisterSuccess');
}
/**
@@ -302,25 +305,42 @@ cr.define('local_discovery', function() {
* @param {string} device_id ID of device.
*/
function manageCloudDevice(device_id) {
+ recordUmaAction('DevicesPage_ManageClicked');
chrome.send('openCloudPrintURL',
[PRINTER_MANAGEMENT_PAGE_PREFIX + device_id]);
}
+ /**
+ * Record an action in UMA.
+ * @param {string} actionDesc The name of the action to be logged.
+ */
+ function recordUmaAction(actionDesc) {
+ chrome.send('metricsHandler:recordAction', [actionDesc]);
+ }
+
+ /**
+ * Cancel the registration.
+ */
+ function cancelRegistration() {
+ hideRegisterOverlay();
+ chrome.send('cancelRegistration');
+ recordUmaAction('DevicesPage_RegisterCancel');
+ }
document.addEventListener('DOMContentLoaded', function() {
uber.onContentFrameLoaded();
cr.ui.overlay.setupOverlay($('overlay'));
cr.ui.overlay.globalInitialization();
- $('overlay').addEventListener('cancelOverlay', hideRegisterOverlay);
+ $('overlay').addEventListener('cancelOverlay', cancelRegistration);
var cancelButtons = document.querySelectorAll('.register-cancel');
var cancelButtonsLength = cancelButtons.length;
for (var i = 0; i < cancelButtonsLength; i++) {
- cancelButtons[i].addEventListener('click', hideRegisterOverlay);
+ cancelButtons[i].addEventListener('click', cancelRegistration);
}
- $('register-error-exit').addEventListener('click', hideRegisterOverlay);
+ $('register-error-exit').addEventListener('click', cancelRegistration);
$('add-printers-button').addEventListener('click',
showRegisterOverlay);
@@ -333,6 +353,7 @@ cr.define('local_discovery', function() {
uber.invokeMethodOnParent('setTitle', {title: title});
chrome.send('start');
+ recordUmaAction('DevicesPage_Opened');
requestPrinterList();
});

Powered by Google App Engine
This is Rietveld 408576698