ResourceLoadScheduler: Enable the feature by default

This patch enables the ResourceLoadScheduler feature by default.
The feature has been enabled via a field trial flag on beta, dev,
and canary channels, and now it is going to be enabled on stable.

Bug: 729954
Change-Id: I6ae63dffe2fc0d89b036a4ed9c8affe4c7ed12c8
Reviewed-on: https://chromium-review.googlesource.com/1195265Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Takashi Toyoshima <toyoshim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#588818}

content/public/common/content_features.cc

@@ -367,7 +367,7 @@ const base::Feature kResamplingInputEvents{"ResamplingInputEvents",
 
 // Loading Dispatcher v0 support with ResourceLoadScheduler (crbug.com/729954).
 const base::Feature kResourceLoadScheduler{"ResourceLoadScheduler",
-                                           base::FEATURE_DISABLED_BY_DEFAULT};
+                                           base::FEATURE_ENABLED_BY_DEFAULT};
 
 // Run video capture service in the Browser process as opposed to a dedicated
 // utility process

third_party/blink/renderer/platform/loader/fetch/resource_load_scheduler.cc

@@ -29,7 +29,8 @@ const char kOutstandingLimitForBackgroundMainFrameName[] = "bg_limit";
 const char kOutstandingLimitForBackgroundSubFrameName[] = "bg_sub_limit";
 
 // Field trial default parameters.
-constexpr size_t kOutstandingLimitForBackgroundFrameDefault = 16u;
+constexpr size_t kOutstandingLimitForBackgroundMainFrameDefault = 3u;
+constexpr size_t kOutstandingLimitForBackgroundSubFrameDefault = 2u;
 
 // Maximum request count that request count metrics assume.
 constexpr base::HistogramBase::Sample kMaximumReportSize10K = 10000;
@@ -94,18 +95,14 @@ size_t GetOutstandingThrottledLimit(FetchContext* context) {
   if (!RuntimeEnabledFeatures::ResourceLoadSchedulerEnabled())
     return ResourceLoadScheduler::kOutstandingUnlimited;
 
-  uint32_t main_frame_limit = GetFieldTrialUint32Param(
+  static size_t main_frame_limit = GetFieldTrialUint32Param(
       kResourceLoadThrottlingTrial, kOutstandingLimitForBackgroundMainFrameName,
-      kOutstandingLimitForBackgroundFrameDefault);
-  if (context->IsMainFrame())
-    return main_frame_limit;
-
-  // We do not have a fixed default limit for sub-frames, but use the limit for
-  // the main frame so that it works as how previous versions that haven't
-  // consider sub-frames' specific limit work.
-  return GetFieldTrialUint32Param(kResourceLoadThrottlingTrial,
-                                  kOutstandingLimitForBackgroundSubFrameName,
-                                  main_frame_limit);
+      kOutstandingLimitForBackgroundMainFrameDefault);
+  static size_t sub_frame_limit = GetFieldTrialUint32Param(
+      kResourceLoadThrottlingTrial, kOutstandingLimitForBackgroundSubFrameName,
+      kOutstandingLimitForBackgroundSubFrameDefault);
+
+  return context->IsMainFrame() ? main_frame_limit : sub_frame_limit;
 }
 
 int TakeWholeKilobytes(int64_t& bytes) {