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

Unified Diff: chrome/browser/resources/chromeos/chromevox/cvox2/background/command_handler.js

Issue 2495373005: Record ChromeVox metrics (Closed)
Patch Set: Created 4 years, 1 month 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/chromeos/chromevox/cvox2/background/command_handler.js
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/command_handler.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/command_handler.js
index b87ad1267bb4d737f39927558988abe8539f6324..77ce6c8e9fee7e3d172c3a7948a51deda79dc1e2 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/command_handler.js
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/command_handler.js
@@ -227,10 +227,12 @@ CommandHandler.onCommand = function(command) {
break;
case 'forward':
case 'nextLine':
+ chrome.metricsPrivate.recordUserAction('ChromeVox.Navigate');
current = current.move(cursors.Unit.LINE, Dir.FORWARD);
break;
case 'backward':
case 'previousLine':
+ chrome.metricsPrivate.recordUserAction('ChromeVox.Navigate');
current = current.move(cursors.Unit.LINE, Dir.BACKWARD);
break;
case 'nextButton':
@@ -664,6 +666,8 @@ CommandHandler.onCommand = function(command) {
}
if (pred) {
+ chrome.metricsPrivate.recordUserAction('ChromeVox.Jump');
+
var bound = current.getBound(dir).node;
if (bound) {
var node = AutomationUtil.findNextNode(

Powered by Google App Engine
This is Rietveld 408576698