Chromium Code Reviews| Index: content/browser/ppapi_plugin_process_host.cc |
| =================================================================== |
| --- content/browser/ppapi_plugin_process_host.cc (revision 246096) |
| +++ content/browser/ppapi_plugin_process_host.cc (working copy) |
| @@ -307,11 +307,15 @@ |
| // TODO(vtl): Stop passing flash args in the command line, or windows is |
| // going to explode. |
| std::string field_trial = |
| - base::FieldTrialList::FindFullName(kLowLatencyFlashAudioFieldTrialName); |
| + base::FieldTrialList::FindFullName(kFlashHwVideoDecodeFieldTrialName); |
| std::string existing_args = |
| browser_command_line.GetSwitchValueASCII(switches::kPpapiFlashArgs); |
| - if (field_trial == kLowLatencyFlashAudioFieldTrialEnabledName) |
| - existing_args.append(" enable_low_latency_audio=1"); |
| + if (field_trial == kFlashHwVideoDecodeFieldTrialEnabledName) { |
| + // Arguments passed to Flash are comma delimited. |
| + if (existing_args.size()) |
|
Alexei Svitkine (slow)
2014/01/22 16:27:24
Nit: !existing_args.empty()
Alexei Svitkine (slow)
2014/01/22 20:42:20
On 2014/01/22 16:27:24, Alexei Svitkine wrote:
> N
|
| + existing_args.append(","); |
| + existing_args.append("enable_hw_video_decode=1"); |
| + } |
| cmd_line->AppendSwitchASCII(switches::kPpapiFlashArgs, existing_args); |
| } |