Adding a destination platform histogram for UMA.

chrome/browser/chromeos/arc/intent_helper/arc_navigation_throttle.cc

 // Copyright 2016 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.
 
 #include "chrome/browser/chromeos/arc/intent_helper/arc_navigation_throttle.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 352 matching lines @@
           handle->GetWebContents()->Close();
       }
       break;
     }
     case CloseReason::INVALID: {
       NOTREACHED();
       return;
     }
   }
 
-  UMA_HISTOGRAM_ENUMERATION("Arc.IntentHandlerAction",
-                            static_cast<int>(close_reason),
-                            static_cast<int>(CloseReason::SIZE));
+  Platform platform =
+      GetDestinationPlatform(selected_app_package, close_reason);
+  RecordUma(close_reason, platform);
 }
 
 // static
 size_t ArcNavigationThrottle::GetAppIndex(
     const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers,
     const std::string& selected_app_package) {
   for (size_t i = 0; i < handlers.size(); ++i) {
     if (handlers[i]->package_name == selected_app_package)
       return i;
   }
   return handlers.size();
 }
 
 // static
+ArcNavigationThrottle::Platform ArcNavigationThrottle::GetDestinationPlatform(
+    const std::string& selected_app_package,
+    CloseReason close_reason) {
+  return (close_reason != CloseReason::ERROR &&
+          close_reason != CloseReason::DIALOG_DEACTIVATED &&
+          !ArcIntentHelperBridge::IsIntentHelperPackage(selected_app_package))
+             ? Platform::ARC
+             : Platform::CHROME;
+}
+
+// static
+void ArcNavigationThrottle::RecordUma(CloseReason close_reason,
+                                      Platform platform) {
+  UMA_HISTOGRAM_ENUMERATION("Arc.IntentHandlerAction",
+                            static_cast<int>(close_reason),
+                            static_cast<int>(CloseReason::SIZE));
+
+  UMA_HISTOGRAM_ENUMERATION("Arc.IntentHandlerDestinationPlatform",
+                            static_cast<int>(platform),
+                            static_cast<int>(Platform::SIZE));
+}
+
+// static
 bool ArcNavigationThrottle::ShouldOverrideUrlLoadingForTesting(
     const GURL& previous_url,
     const GURL& current_url) {
   return ShouldOverrideUrlLoading(previous_url, current_url);
 }
 
 // static
 bool ArcNavigationThrottle::IsAppAvailableForTesting(
     const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers) {
   return IsAppAvailable(handlers);
 }
 
 // static
 size_t ArcNavigationThrottle::FindPreferredAppForTesting(
     const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers) {
   return FindPreferredApp(handlers, GURL());
 }
 
 // static
 bool ArcNavigationThrottle::IsSwapElementsNeededForTesting(
     const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers,
     std::pair<size_t, size_t>* out_indices) {
   return IsSwapElementsNeeded(handlers, out_indices);
 }
 
 }  // namespace arc