Commit b37b3b81 authored by Drew Stonebraker's avatar Drew Stonebraker Committed by Commit Bot

Remove kWebViewUmaLogAppPackageName conditions

The feature has been fully launched, so remove the feature flag and the
conditions.

Bug: 987712
Test: none
Change-Id: I87de2af8976a70d44790b34ad97fa7fefba49b64
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1719465Reviewed-by: default avatarTao Bai <michaelbai@chromium.org>
Commit-Queue: Drew Stonebraker <stonebraker@chromium.org>
Cr-Commit-Position: refs/heads/master@{#681084}
parent 0292bdcb
...@@ -57,10 +57,6 @@ const base::Feature kWebViewConnectionlessSafeBrowsing{ ...@@ -57,10 +57,6 @@ const base::Feature kWebViewConnectionlessSafeBrowsing{
const base::Feature kWebViewPageStartedOnCommit{ const base::Feature kWebViewPageStartedOnCommit{
"WebViewPageStartedOnCommit", base::FEATURE_ENABLED_BY_DEFAULT}; "WebViewPageStartedOnCommit", base::FEATURE_ENABLED_BY_DEFAULT};
// Whether the application package name is logged in UMA.
const base::Feature kWebViewUmaLogAppPackageName{
"WebViewUmaLogAppPackageName", base::FEATURE_ENABLED_BY_DEFAULT};
// Enable raster in wide color gamut for apps that use webview in a wide color // Enable raster in wide color gamut for apps that use webview in a wide color
// gamut activity. // gamut activity.
const base::Feature kWebViewWideColorGamutSupport{ const base::Feature kWebViewWideColorGamutSupport{
......
...@@ -17,7 +17,6 @@ namespace features { ...@@ -17,7 +17,6 @@ namespace features {
extern const base::Feature kWebViewBrotliSupport; extern const base::Feature kWebViewBrotliSupport;
extern const base::Feature kWebViewConnectionlessSafeBrowsing; extern const base::Feature kWebViewConnectionlessSafeBrowsing;
extern const base::Feature kWebViewPageStartedOnCommit; extern const base::Feature kWebViewPageStartedOnCommit;
extern const base::Feature kWebViewUmaLogAppPackageName;
extern const base::Feature kWebViewWideColorGamutSupport; extern const base::Feature kWebViewWideColorGamutSupport;
} // namespace features } // namespace features
......
...@@ -218,9 +218,6 @@ base::TimeDelta AwMetricsServiceClient::GetStandardUploadInterval() { ...@@ -218,9 +218,6 @@ base::TimeDelta AwMetricsServiceClient::GetStandardUploadInterval() {
} }
std::string AwMetricsServiceClient::GetAppPackageName() { std::string AwMetricsServiceClient::GetAppPackageName() {
if (!base::FeatureList::IsEnabled(features::kWebViewUmaLogAppPackageName))
return std::string();
JNIEnv* env = base::android::AttachCurrentThread(); JNIEnv* env = base::android::AttachCurrentThread();
base::android::ScopedJavaLocalRef<jstring> j_app_name = base::android::ScopedJavaLocalRef<jstring> j_app_name =
Java_AwMetricsServiceClient_getAppPackageName(env); Java_AwMetricsServiceClient_getAppPackageName(env);
......
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