aw: Remove features::kWebViewPageStartedOnCommit

Clean up. It's been enabled by default for awhile now.

Change-Id: Ice4ffc2d69aa8f17564293d9befe3dc3021244b5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725536
Auto-Submit: Bo <boliu@chromium.org>
Reviewed-by: Richard Coles <torne@chromium.org>
Commit-Queue: Bo <boliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#682300}

android_webview/browser/aw_feature_list.cc

@@ -24,7 +24,6 @@ namespace {
 // in other locations in the code base (e.g. content/, components/, etc).
 const base::Feature* kFeaturesExposedToJava[] = {
     &features::kWebViewConnectionlessSafeBrowsing,
-    &features::kWebViewPageStartedOnCommit,
 };
 
 const base::Feature* FindFeatureExposedToJava(const std::string& feature_name) {
@@ -52,11 +51,6 @@ const base::Feature kWebViewBrotliSupport{"WebViewBrotliSupport",
 const base::Feature kWebViewConnectionlessSafeBrowsing{
     "WebViewConnectionlessSafeBrowsing", base::FEATURE_DISABLED_BY_DEFAULT};
 
-// Kill switch for feature to call onPageFinished for browser-initiated
-// navigations when the navigation commits.
-const base::Feature kWebViewPageStartedOnCommit{
-    "WebViewPageStartedOnCommit", base::FEATURE_ENABLED_BY_DEFAULT};
-
 // Enable raster in wide color gamut for apps that use webview in a wide color
 // gamut activity.
 const base::Feature kWebViewWideColorGamutSupport{

android_webview/browser/aw_feature_list.h

@@ -16,7 +16,6 @@ namespace features {
 // Alphabetical:
 extern const base::Feature kWebViewBrotliSupport;
 extern const base::Feature kWebViewConnectionlessSafeBrowsing;
-extern const base::Feature kWebViewPageStartedOnCommit;
 extern const base::Feature kWebViewWideColorGamutSupport;
 
 }  // namespace features

android_webview/java/src/org/chromium/android_webview/AwFeatureList.java

@@ -25,7 +25,6 @@ final public class AwFeatureList {
     private static Boolean sPageStartedOnCommitForBrowserNavigations;
 
     private static boolean computePageStartedOnCommitForBrowserNavigations() {
-        if (!nativeIsEnabled(WEBVIEW_PAGE_STARTED_ON_COMMIT)) return false;
         if (GMS_PACKAGE.equals(ContextUtils.getApplicationContext().getPackageName())) {
             try {
                 PackageInfo gmsPackage =
@@ -66,7 +65,6 @@ final public class AwFeatureList {
     // Alphabetical:
     public static final String WEBVIEW_CONNECTIONLESS_SAFE_BROWSING =
             "WebViewConnectionlessSafeBrowsing";
-    public static final String WEBVIEW_PAGE_STARTED_ON_COMMIT = "WebViewPageStartedOnCommit";
 
     private static native boolean nativeIsEnabled(String featureName);
 }