Chromium Code Reviews| Index: content/browser/renderer_host/render_view_host_impl.cc |
| diff --git a/content/browser/renderer_host/render_view_host_impl.cc b/content/browser/renderer_host/render_view_host_impl.cc |
| index f09be50cc455cfccd7b1edc9764228c44469c2f2..5b5d7e55a5de7826f281462872e4b8e4ee57f778 100644 |
| --- a/content/browser/renderer_host/render_view_host_impl.cc |
| +++ b/content/browser/renderer_host/render_view_host_impl.cc |
| @@ -473,9 +473,24 @@ WebPreferences RenderViewHostImpl::ComputeWebkitPrefs(const GURL& url) { |
| } |
| } |
| + std::string streaming_experiment_group = |
| + base::FieldTrialList::FindFullName("V8ScriptStreaming"); |
| prefs.v8_script_streaming_enabled = |
| - command_line.HasSwitch(switches::kEnableV8ScriptStreaming) || |
| - base::FieldTrialList::FindFullName("V8ScriptStreaming") == "Enabled"; |
| + command_line.HasSwitch(switches::kEnableV8ScriptStreaming); |
| + if (streaming_experiment_group == "Enabled") { |
| + prefs.v8_script_streaming_enabled = true; |
|
Mike West
2014/10/31 14:42:43
This means that you're going to override the comma
marja
2014/10/31 14:47:48
Yes. People should get this enabled if 1) they got
|
| + prefs.v8_script_streaming_mode = V8_SCRIPT_STREAMING_MODE_ALL; |
| + } |
|
jochen (gone - plz use gerrit)
2014/11/03 09:09:29
else if {
marja
2014/11/03 09:13:33
Done.
|
| + if (streaming_experiment_group == "OnlyAsyncAndDefer") { |
| + prefs.v8_script_streaming_enabled = true; |
| + prefs.v8_script_streaming_mode = |
| + V8_SCRIPT_STREAMING_MODE_ONLY_ASYNC_AND_DEFER; |
| + } |
| + if (streaming_experiment_group == "AllPlusBlockParserBlocking") { |
| + prefs.v8_script_streaming_enabled = true; |
| + prefs.v8_script_streaming_mode = |
| + V8_SCRIPT_STREAMING_MODE_ALL_PLUS_BLOCK_PARSER_BLOCKING; |
| + } |
| GetContentClient()->browser()->OverrideWebkitPrefs(this, url, &prefs); |
| return prefs; |