Commit 09bcdef1 authored by Robert Kaplow's avatar Robert Kaplow Committed by Commit Bot

Remove not-used Android RAPPOR metrics.

Bug: 1016906
Change-Id: Ibf0bce1a5528b6e9495413342b84128687d5ba7d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1872477Reviewed-by: default avatarYaron Friedman <yfriedman@chromium.org>
Commit-Queue: Robert Kaplow <rkaplow@chromium.org>
Cr-Commit-Position: refs/heads/master@{#708603}
parent 87441e91
......@@ -40,7 +40,6 @@ import org.chromium.chrome.browser.externalnav.ExternalNavigationDelegateImpl;
import org.chromium.chrome.browser.externalnav.IntentWithGesturesHandler;
import org.chromium.chrome.browser.offlinepages.OfflinePageUtils;
import org.chromium.chrome.browser.omnibox.suggestions.AutocompleteCoordinatorFactory;
import org.chromium.chrome.browser.rappor.RapporServiceBridge;
import org.chromium.chrome.browser.search_engines.TemplateUrlServiceFactory;
import org.chromium.chrome.browser.tab.Tab;
import org.chromium.chrome.browser.tabmodel.TabLaunchType;
......@@ -411,12 +410,6 @@ public class IntentHandler {
int externalId = determineExternalIntentSource(intent);
RecordHistogram.recordEnumeratedHistogram(
"MobileIntent.PageLoadDueToExternalApp", externalId, ExternalAppId.NUM_ENTRIES);
if (externalId == ExternalAppId.OTHER) {
String appId = IntentUtils.safeGetStringExtra(intent, Browser.EXTRA_APPLICATION_ID);
if (!TextUtils.isEmpty(appId)) {
RapporServiceBridge.sampleString("Android.PageLoadDueToExternalApp", appId);
}
}
}
/**
......@@ -429,9 +422,6 @@ public class IntentHandler {
IntentHandler.EXTRA_EXTERNAL_NAV_PACKAGES);
if (packages != null && packages.size() > 0) {
RecordUserAction.record("MobileExternalNavigationReceived");
for (String name : packages) {
RapporServiceBridge.sampleString("Android.ExternalNavigationNotChosen", name);
}
}
}
......
......@@ -101,24 +101,6 @@ components/rappor/rappor_parameters.h.
<!-- Rappor metric definitions -->
<rappor-metrics>
<rappor-metric name="Android.ExternalNavigationNotChosen"
type="UMA_RAPPOR_TYPE">
<owner>mariakhomenko@chromium.org</owner>
<summary>
The package name of an application that was given as an alternative choice
to handling a URL in Chrome, but wasn't chosen.
</summary>
</rappor-metric>
<rappor-metric name="Android.PageLoadDueToExternalApp" type="UMA_RAPPOR_TYPE">
<owner>mariakhomenko@chromium.org</owner>
<summary>
The package name of an application that sent an intent to Chrome. The name
is taken from EXTRA_APPLICATION_ID passed to Chrome in the intent by the
application.
</summary>
</rappor-metric>
<rappor-metric name="AppBanner.NativeApp.Dismissed" type="ETLD_PLUS_ONE">
<owner>dfalcantara@chromium.org</owner>
<summary>
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment