Adding metrics for notification clicks and dismissals

chrome/browser/resources/google_now/metrics.js

Index: chrome/browser/resources/google_now/metrics.js
diff --git a/chrome/browser/resources/file_manager/js/metrics.js b/chrome/browser/resources/google_now/metrics.js
similarity index 92%
copy from chrome/browser/resources/file_manager/js/metrics.js
copy to chrome/browser/resources/google_now/metrics.js
index 1fdd63bfdf378b8d9fa6c11982f2a6df81f1a03d..a397edc4e5cedb062673833da5bd7a0617d23097 100644
--- a/chrome/browser/resources/file_manager/js/metrics.js
+++ b/chrome/browser/resources/google_now/metrics.js
@@ -1,4 +1,4 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
@@ -8,6 +8,10 @@
  * To be included as a first script in main.html
  */
 
+// This file was adapted from

[rgustafson, 2013/03/28 21:49:56]

What other metrics will you track with this? If you're just using
recordUserAction, this seems unnecessarily large.

[vadimt, 2013/03/28 22:05:05]

I also plan to record errors (may be, using histograms).
The file has a TODO item to remove dead code before release.

[rgustafson, 2013/03/29 02:17:55]

There's absolutely no benefit to having this file right now, just more
complexity to accomplish what could already be done in one line. I'd much prefer
that you add this when you need it (or add code now that does make this
necessary and lower the number of reviews for owners in the other various
directories).

[skare_, 2013/03/29 18:08:35]

agree - duplicating a sibling file seems error-prone, confusing for people who
search for that API later, and might increase the size of the binary
unnecessarily. You might not even need a wrapper around the API, similar to how
Notifications Galore has one for notifications and you just call the API direct.

[vadimt, 2013/03/29 20:15:28]

Done.

[vadimt, 2013/03/29 20:15:28]

Done.

+// chrome/browser/resources/file_manager/js/metrics.js.
+// TODO(vadimt): Remove unused code.

[skare_, 2013/03/28 21:24:51]

possible to unify between the two extensions?

[vadimt, 2013/03/28 21:36:50]

Ideas?
Copy from file_manager as a custom build step?

[skare_, 2013/03/29 18:08:35]

something like that, but only when you really need it. Or just call the APIs
directly.

[vadimt, 2013/03/29 20:15:28]

Done.

+
 var metrics = {};
 
 /**
@@ -36,11 +40,11 @@ metrics.startInterval('Load.Script');
  * @private
  */
 metrics.convertName_ = function(name) {
-  return 'FileBrowser.' + name;
+  return 'GoogleNow.' + name;
 };
 
 /**
- * Wrapper method for calling chrome.fileBrowserPrivate safely.
+ * Wrapper method for calling chrome.metricsPrivate safely.
  * @param {string} name Method name.
  * @param {Array.<Object>} args Arguments.
  * @private