Commit 0c7b4da1 authored by mariakhomenko's avatar mariakhomenko Committed by Commit bot

Log a rappor metric for external app ids.

BUG=640734

Review-Url: https://codereview.chromium.org/2272863004
Cr-Commit-Position: refs/heads/master@{#414318}
parent 7b88754c
...@@ -307,6 +307,12 @@ public class IntentHandler { ...@@ -307,6 +307,12 @@ public class IntentHandler {
ExternalAppId externalId = determineExternalIntentSource(mPackageName, intent); ExternalAppId externalId = determineExternalIntentSource(mPackageName, intent);
RecordHistogram.recordEnumeratedHistogram("MobileIntent.PageLoadDueToExternalApp", RecordHistogram.recordEnumeratedHistogram("MobileIntent.PageLoadDueToExternalApp",
externalId.ordinal(), ExternalAppId.INDEX_BOUNDARY.ordinal()); externalId.ordinal(), ExternalAppId.INDEX_BOUNDARY.ordinal());
if (externalId == ExternalAppId.OTHER) {
String appId = IntentUtils.safeGetStringExtra(intent, Browser.EXTRA_APPLICATION_ID);
if (!TextUtils.isEmpty(appId)) {
RapporServiceBridge.sampleString("Android.PageLoadDueToExternalApp", appId);
}
}
} }
/** /**
......
...@@ -146,6 +146,15 @@ components/rappor/rappor_parameters.h. ...@@ -146,6 +146,15 @@ components/rappor/rappor_parameters.h.
</summary> </summary>
</rappor-metric> </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"> <rappor-metric name="AppBanner.NativeApp.Dismissed" type="ETLD_PLUS_ONE">
<owner>dfalcantara@chromium.org</owner> <owner>dfalcantara@chromium.org</owner>
<summary> <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