Add metrics to track Drive pinning success

Bug: 1067346
Change-Id: Ie7c5503754d1e2a1808e11230b9f6c6916c97269
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2426063
Commit-Queue: Austin Tankiang <austinct@chromium.org>
Reviewed-by: Sergei Datsenko <dats@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#811664}

tools/metrics/histograms/histograms_xml/file/histograms.xml

@@ -129,6 +129,20 @@ reviews. Googlers can read more about this at go/gwsq-gerrit.
   </summary>
 </histogram>
 
+<histogram name="FileBrowser.DriveHostedFilePinSuccess" enum="BooleanSuccess"
+    expires_after="M92">
+  <owner>austinct@chromium.org</owner>
+  <owner>dats@chromium.org</owner>
+  <summary>Tracks success rate of pinning hosted files in Drive.</summary>
+</histogram>
+
+<histogram name="FileBrowser.DrivePinSuccess" enum="BooleanSuccess"
+    expires_after="M92">
+  <owner>austinct@chromium.org</owner>
+  <owner>dats@chromium.org</owner>
+  <summary>Tracks success rate of pinning files in Drive.</summary>
+</histogram>
+
 <histogram name="FileBrowser.FileSystemProviderMounted"
     enum="FileSystemProviderMountType" expires_after="M89">
   <owner>slangley@chromium.org</owner>

ui/file_manager/file_manager/foreground/js/actions_model.js

@@ -254,6 +254,11 @@ class DriveToggleOfflineAction {
       // Check the result of pinning.
       entryPinned: () => {
         error = !!chrome.runtime.lastError;
+        metrics.recordBoolean('DrivePinSuccess', !error);
+        if (this.metadataModel_.getCache([currentEntry], ['hosted'])[0]
+                .hosted) {
+          metrics.recordBoolean('DriveHostedFilePinSuccess', !error);
+        }
         if (error && this.value_) {
           this.metadataModel_.get([currentEntry], ['size']).then(results => {
             steps.showError(results[0].size);

ui/file_manager/file_manager/foreground/js/actions_model_unittest.js

@@ -109,6 +109,17 @@ function setUp() {
   installMockChrome(mockChrome);
   new MockCommandLinePrivate();
 
+  // Mock metrics.
+  window.metrics = {
+    calls: {
+      DrivePinSuccess: 0,
+      DriveHostedFilePinSuccess: 0,
+    },
+    recordBoolean: function(name) {
+      window.metrics.calls[name]++;
+    },
+  };
+
   // Setup Drive file system.
   volumeManager = new MockVolumeManager();
   let type = VolumeManagerCommon.VolumeType.DRIVE;
@@ -265,6 +276,9 @@ function testDriveFileEntry(callback) {
             assertTrue(metadataModel.properties.pinned);
             assertEquals(1, invalidated);
 
+            assertEquals(1, window.metrics.calls['DrivePinSuccess']);
+            assertEquals(0, window.metrics.calls['DriveHostedFilePinSuccess']);
+
             // The model is invalidated, as list of actions have changed.
             // Recreated the model and check that the actions are updated.
             model = new ActionsModel(
@@ -382,6 +396,9 @@ function testDriveHostedFileEntry(callback) {
             assertTrue(!!metadataModel.getCache([testDocument])[0].pinned);
             assertTrue(!!metadataModel.getCache([testFile])[0].pinned);
 
+            assertEquals(2, window.metrics.calls['DrivePinSuccess']);
+            assertEquals(1, window.metrics.calls['DriveHostedFilePinSuccess']);
+
             model = new ActionsModel(
                 volumeManager, metadataModel, shortcutsModel, driveSyncHandler,
                 ui, [testDocument, testFile]);
@@ -489,6 +506,9 @@ function testDriveHostedFileEntryWithoutExtension(callback) {
             assertFalse(!!metadataModel.getCache([testDocument])[0].pinned);
             assertTrue(!!metadataModel.getCache([testFile])[0].pinned);
 
+            assertEquals(1, window.metrics.calls['DrivePinSuccess']);
+            assertEquals(0, window.metrics.calls['DriveHostedFilePinSuccess']);
+
             model = new ActionsModel(
                 volumeManager, metadataModel, shortcutsModel, driveSyncHandler,
                 ui, [testDocument, testFile]);