Adding location events to the histogram.

BUG=164227
TEST=Check that LOCATION_REQUEST and LOCATION_EVENT buckets appeared in the dashboard.

Review URL: https://chromiumcodereview.appspot.com/15306005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@201988 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/resources/google_now/background.js

@@ -132,7 +132,9 @@ var DiagnosticEvent = {
   CARDS_PARSE_SUCCESS: 2,
   DISMISS_REQUEST_TOTAL: 3,
   DISMISS_REQUEST_SUCCESS: 4,
-  EVENTS_TOTAL: 5  // EVENTS_TOTAL is not an event; all new events need to be
+  LOCATION_REQUEST: 5,
+  LOCATION_UPDATE: 6,
+  EVENTS_TOTAL: 7  // EVENTS_TOTAL is not an event; all new events need to be
                    // added before it.
 };
 
@@ -350,6 +352,7 @@ function requestNotificationCards(position, callback) {
  */
 function requestLocation() {
   console.log('requestLocation');
+  recordEvent(DiagnosticEvent.LOCATION_REQUEST);
   // TODO(vadimt): Figure out location request options.
   chrome.location.watchLocation(LOCATION_WATCH_NAME, {});
 }
@@ -589,4 +592,7 @@ chrome.notifications.onButtonClicked.addListener(
 
 chrome.notifications.onClosed.addListener(onNotificationClosed);
 
-chrome.location.onLocationUpdate.addListener(updateNotificationsCards);
+chrome.location.onLocationUpdate.addListener(function(position) {
+  recordEvent(DiagnosticEvent.LOCATION_UPDATE);
+  updateNotificationsCards(position);
+});

tools/metrics/histograms/histograms.xml

@@ -11271,6 +11271,8 @@ other types of suffix sets.
   <int value="2" label="CARDS_PARSE_SUCCESS"/>
   <int value="3" label="DISMISS_REQUEST_TOTAL"/>
   <int value="4" label="DISMISS_REQUEST_SUCCESS"/>
+  <int value="5" label="LOCATION_REQUEST"/>
+  <int value="6" label="LOCATION_UPDATE"/>
 </enum>
 
 <enum name="HttpAuthCount" type="int">