Fix perfbotnot feature param setting from trial testing config.

tools/variations/fieldtrial_util_unittest.py

 # Copyright 2015 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.
 
 import unittest
 
 import fieldtrial_util
 import os
 import tempfile
 
@@ skipping 48 matching lines @@
           ]
         }
       ]
     }'''
     result = self.runGenerateArgs(config, 'win')
     self.assertEqual(['--force-fieldtrials='
         'BrowserBlackList/Enabled/SimpleParams/Default/c/d.',
         '--force-fieldtrial-params='
         'SimpleParams.Default:id/abc,'
         'c.d%2E:url/http%3A%2F%2Fwww%2Egoogle%2Ecom',
-        '--enable-features=a,b,x',
-        '--disable-features=y'], result)
+        '--enable-features="a<SimpleParams,b<SimpleParams,x<c"',
+        '--disable-features="y<c"'], result)

[perezju, 2017/03/30 08:16:06]

Are we sure that those quotes are needed there?

The command line flags are not passed through the shell, so there should be no
need of any special quoting.

[Alexei Svitkine (slow), 2017/03/30 14:28:30]

You're right that it might not be necessary, though I think it should work
either way, no?

 
   def test_DuplicateEnableFeatures(self):
     config = '''{
       "X": [
         {
           "platforms": ["win"],
           "experiments": [
             {
               "name": "x",
               "enable_features": ["x"]
@@ skipping 74 matching lines @@
         }
       ]
     }'''
     with self.assertRaises(Exception) as raised:
       self.runGenerateArgs(config, 'win')
     self.assertEqual('Conflicting features set as both enabled and disabled: x',
                      str(raised.exception))
 
 if __name__ == '__main__':
   unittest.main()