Remove UMA Webapp.AmbientBadge.*

Remove Webapp.AmbientBadge.Display/Dismiss/Click
Keep Webapp.AmbientBadge.Terminate


OBSOLETE_HISTOGRAMS=no longer used

Bug: 354712669
Change-Id: Ib0313a3c7e6d940b8b8956989e22902b706cef0f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5730837
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Auto-Submit: Ella Ge <eirage@chromium.org>
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1331418}

components/webapps/browser/BUILD.gn

@@ -115,8 +115,6 @@ source_set("browser") {
       "android/add_to_homescreen_params.h",
       "android/ambient_badge_manager.cc",
       "android/ambient_badge_manager.h",
-      "android/ambient_badge_metrics.cc",
-      "android/ambient_badge_metrics.h",
       "android/app_banner_manager_android.cc",
       "android/app_banner_manager_android.h",
       "android/bottomsheet/pwa_bottom_sheet_controller.cc",

components/webapps/browser/android/ambient_badge_manager.cc

@@ -9,14 +9,14 @@
 #include <string>
 
 #include "base/feature_list.h"
+#include "base/metrics/histogram_functions.h"
 #include "base/metrics/histogram_macros.h"
 #include "components/prefs/pref_service.h"
 #include "components/segmentation_platform/public/constants.h"
 #include "components/segmentation_platform/public/input_context.h"
 #include "components/segmentation_platform/public/result.h"
 #include "components/segmentation_platform/public/segmentation_platform_service.h"
 #include "components/webapps/browser/android/add_to_homescreen_params.h"
-#include "components/webapps/browser/android/ambient_badge_metrics.h"
 #include "components/webapps/browser/android/app_banner_manager_android.h"
 #include "components/webapps/browser/android/install_prompt_prefs.h"
 #include "components/webapps/browser/android/shortcut_info.h"
@@ -32,6 +32,8 @@ namespace {
 
 constexpr char kSegmentationResultHistogramName[] =
     "WebApk.InstallPrompt.SegmentationResult";
+constexpr char kAmbientBadgeTerminateHistogram[] =
+    "Webapp.AmbientBadge.Terminate";
 
 // This enum is used to back UMA histograms, Entries should not be renumbered
 // and numeric values should never be reused.
@@ -57,7 +59,7 @@ AmbientBadgeManager::AmbientBadgeManager(
       pref_service_(prefs) {}
 
 AmbientBadgeManager::~AmbientBadgeManager() {
-  RecordAmbientBadgeTeminateState(state_);
+  base::UmaHistogramEnumeration(kAmbientBadgeTerminateHistogram, state_);
 }
 
 void AmbientBadgeManager::MaybeShow(
@@ -83,7 +85,6 @@ void AmbientBadgeManager::MaybeShow(
 
 void AmbientBadgeManager::AddToHomescreenFromBadge() {
   CHECK(a2hs_params_);
-  RecordAmbientBadgeClickEvent(a2hs_params_->app_type);
   InstallPromptPrefs::RecordInstallPromptClicked(pref_service());
   std::move(show_banner_callback_).Run();
 }
@@ -97,7 +98,6 @@ void AmbientBadgeManager::BadgeDismissed() {
 
   InstallPromptPrefs::RecordInstallPromptDismissed(
       pref_service(), AppBannerManager::GetCurrentTime());
-  RecordAmbientBadgeDismissEvent(a2hs_params_->app_type);
   UpdateState(State::kDismissed);
 }
 
@@ -110,7 +110,6 @@ void AmbientBadgeManager::BadgeIgnored() {
 
   InstallPromptPrefs::RecordInstallPromptIgnored(
       pref_service(), AppBannerManager::GetCurrentTime());
-  RecordAmbientBadgeDismissEvent(a2hs_params_->app_type);
   UpdateState(State::kDismissed);
 }
 
@@ -228,7 +227,6 @@ void AmbientBadgeManager::ShowAmbientBadge() {
     return;
   }
 
-  RecordAmbientBadgeDisplayEvent(a2hs_params_->app_type);
   UpdateState(State::kShowing);
 
   WebappInstallSource install_source = InstallableMetrics::GetInstallSource(

components/webapps/browser/android/app_banner_manager_android.h

@@ -15,7 +15,6 @@
 #include "base/memory/weak_ptr.h"
 #include "base/types/expected.h"
 #include "components/webapps/browser/android/add_to_homescreen_installer.h"
-#include "components/webapps/browser/android/ambient_badge_metrics.h"
 #include "components/webapps/browser/android/installable/installable_ambient_badge_client.h"
 #include "components/webapps/browser/android/installable/installable_ambient_badge_message_controller.h"
 #include "components/webapps/browser/banners/app_banner_manager.h"

tools/metrics/histograms/metadata/webapps/histograms.xml

@@ -232,42 +232,6 @@ chromium-metrics-reviews@google.com.
   </summary>
 </histogram>
 
-<histogram name="Webapp.AmbientBadge.Click" enum="InstallAmbientBadgeAppType"
-    expires_after="2024-09-01">
-  <owner>eirage@chromium.org</owner>
-  <owner>src/chrome/android/webapk/OWNERS</owner>
-  <summary>
-    On mobile, an install ambient badge may automatically shows to promote an
-    application related to the current website when the site meets all install
-    criteria. When the ambient badge was clicked by the user and displaying the
-    app banner UI, record the type of the app (native app or web app).
-  </summary>
-</histogram>
-
-<histogram name="Webapp.AmbientBadge.Dismiss" enum="InstallAmbientBadgeAppType"
-    expires_after="2024-09-01">
-  <owner>eirage@chromium.org</owner>
-  <owner>src/chrome/android/webapk/OWNERS</owner>
-  <summary>
-    On mobile, an install ambient badge may automatically shows to promote an
-    application related to the current website when the site meets all install
-    criteria. When the ambient badge was explictly dismissed by the user, record
-    the type of the app (native app or web app).
-  </summary>
-</histogram>
-
-<histogram name="Webapp.AmbientBadge.Display" enum="InstallAmbientBadgeAppType"
-    expires_after="2024-09-01">
-  <owner>eirage@chromium.org</owner>
-  <owner>src/chrome/android/webapk/OWNERS</owner>
-  <summary>
-    On mobile, an install ambient badge may automatically shows to promote an
-    application related to the current website when the site meets all install
-    criteria. When an ambient badge was shown, record the type of the app
-    (native app or web app).
-  </summary>
-</histogram>
-
 <histogram name="Webapp.AmbientBadge.Terminate"
     enum="InstallableAmbientBadgeState" expires_after="2025-01-05">
   <owner>eirage@chromium.org</owner>