Adding a destination platform histogram for UMA.

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

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_ARC_INTENT_HELPER_ARC_NAVIGATION_THROTTLE_H_
 #define CHROME_BROWSER_CHROMEOS_ARC_INTENT_HELPER_ARC_NAVIGATION_THROTTLE_H_
 
 #include <memory>
 #include <string>
 #include <utility>
@@ skipping 25 matching lines @@
   enum class CloseReason : int {
     ERROR = 0,
     DIALOG_DEACTIVATED = 1,
     ALWAYS_PRESSED = 2,
     JUST_ONCE_PRESSED = 3,
     PREFERRED_ACTIVITY_FOUND = 4,
     SIZE,
     INVALID = SIZE,
   };
 
+  // As for CloseReason, these define the buckets for an UMA histogram, so this
+  // must be treated in an append-only fashion. This helps especify where a
+  // navigation will continue.
+  enum class Platform : int {
+    ARC = 0,
+    CHROME = 1,
+    SIZE,
+  };
+
   // Restricts the amount of apps displayed to the user without the need of a
   // ScrollView.
   enum { kMaxAppResults = 3 };
 
   struct AppInfo {
     AppInfo(gfx::Image img, std::string package, std::string activity)
         : icon(img), package_name(package), activity_name(activity) {}
     gfx::Image icon;
     std::string package_name;
     std::string activity_name;
   };
 
   using ShowIntentPickerCallback = base::Callback<void(
       content::WebContents* web_contents,
       const std::vector<AppInfo>& app_info,
       const base::Callback<void(const std::string&, CloseReason)>& cb)>;
   ArcNavigationThrottle(content::NavigationHandle* navigation_handle,
                         const ShowIntentPickerCallback& show_intent_picker_cb);
   ~ArcNavigationThrottle() override;
 
   static bool ShouldOverrideUrlLoadingForTesting(const GURL& previous_url,
                                                  const GURL& current_url);
 
   // Finds |selected_app_package| from the |handlers| array and returns the
   // index. If the app is not found, returns |handlers.size()|.
   static size_t GetAppIndex(
       const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers,
       const std::string& selected_app_package);
+  // Determines the destination of the current navigation. We know that if the
+  // |close_reason| is either ERROR or DIALOG_DEACTIVATED the navigation MUST
+  // stay in Chrome, otherwise we can assume the navigation goes to ARC with the
+  // exception of the |selected_app_package| being Chrome.
+  static Platform GetDestinationPlatform(
+      const std::string& selected_app_package,
+      CloseReason close_reason);
+  // Records intent picker usage statistics as well as whether navigations are
+  // continued or redirected to Chrome or ARC respectively, via UMA histograms.
+  static void RecordUma(CloseReason close_reason, Platform platform);
 
   static bool IsAppAvailableForTesting(
       const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers);
   static size_t FindPreferredAppForTesting(
       const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers);
   static bool IsSwapElementsNeededForTesting(
       const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers,
       std::pair<size_t, size_t>* out_indices);
 
  private:
@@ skipping 27 matching lines @@
 
   // This has to be the last member of the class.
   base::WeakPtrFactory<ArcNavigationThrottle> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ArcNavigationThrottle);
 };
 
 }  // namespace arc
 
 #endif  // CHROME_BROWSER_CHROMEOS_ARC_INTENT_HELPER_ARC_NAVIGATION_THROTTLE_H_