Fix Resource Priorities and Scheduling (Chrome Side)

chrome/browser/chrome_content_browser_client.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chrome_content_browser_client.h"
 
 #include <map>
 #include <set>
 #include <utility>
 #include <vector>
@@ skipping 1200 matching lines @@
     // GetVariationParams has the side-effect of assigning the client to
     // a field trial group.
     std::map<std::string, std::string> params;
     if (variations::GetVariationParams(field_trial_name, &params)) {
       for (const auto& param : params) {
         blink_settings.push_back(base::StringPrintf(
             "%s=%s", param.first.c_str(), param.second.c_str()));
       }
     }
   }
+
+  // Flags for the ResourcePriorities field trial. The settings are
+  // encoded in the field trial group name instead of as variations
+  // because the variations code is not accessible from the loader.
+  //
+  // The group name encoding looks like this:
+  //  <descriptiveName>_ABCDE_E2_F_G
+  //    A - fetchDeferLateScripts (1 for true, 0 for false)
+  //    B - fetchIncreaseFontPriority (1 for true, 0 for false)
+  //    C - fetchIncreaseAsyncScriptPriority (1 for true, 0 for false)
+  //    D - fetchIncreasePriorities (1 for true, 0 for false)
+  //    E - fetchEnableLayoutBlockingThreshold (1 for true, 0 for false)
+  //    E2 - fetchLayoutBlockingThreshold (Numeric)
+  //    F - fetchMaxNumDelayableWhileLayoutBlocking (Numeric)
+  //    G - fetchMaxNumDelayableRequests (Numeric)
+  //
+  // Only A-D are relevant to blink and exposed as settings
+  // Any group names (Control, Default, etc) will not match the pattern or
+  // flags and will get the default settings which is the expected behavior.
+  std::string resource_priorities_trial_group =
+      base::FieldTrialList::FindFullName("ResourcePriorities");
+  std::vector<std::string> split_group(
+      base::SplitString(resource_priorities_trial_group, "_",
+                        base::KEEP_WHITESPACE, base::SPLIT_WANT_ALL));
+  if (split_group.size() == 5 && split_group[1].length() == 5) {
+    if (split_group[1].at(0) == '1')
+      blink_settings.push_back("fetchDeferLateScripts=true");
+    if (split_group[1].at(1) == '1')
+      blink_settings.push_back("fetchIncreaseFontPriority=true");
+    if (split_group[1].at(2) == '1')
+      blink_settings.push_back("fetchIncreaseAsyncScriptPriority=true");
+    if (split_group[1].at(3) == '1')
+      blink_settings.push_back("fetchIncreasePriorities=true");
+  }
+
   if (blink_settings.empty()) {
     return;
   }
 
   if (browser_command_line.HasSwitch(switches::kBlinkSettings) ||
       command_line->HasSwitch(switches::kBlinkSettings)) {
     // The field trial is configured to force users that specify the

[Bryan McQuade, 2015/08/11 13:25:07]

can you update this comment to start with "The field trials should be configured
to"

[Pat Meenan, 2015/08/11 14:37:36]

Done.

     // blink-settings flag into a group with no params, and we return
     // above if no params were specified, so it's an error if we reach
     // this point.
     LOG(WARNING) << "Received field trial params, "
                     "but blink-settings switch already specified.";
     return;
   }
 
   command_line->AppendSwitchASCII(switches::kBlinkSettings,
                                   base::JoinString(blink_settings, ","));
@@ skipping 1326 matching lines @@
       switches::kDisableWebRtcEncryption,
     };
     to_command_line->CopySwitchesFrom(from_command_line,
                                       kWebRtcDevSwitchNames,
                                       arraysize(kWebRtcDevSwitchNames));
   }
 }
 #endif  // defined(ENABLE_WEBRTC)
 
 }  // namespace chrome