Enable pnacl by default

native_client_sdk/src/build_tools/test_projects.py

 #!/usr/bin/env python
 # Copyright (c) 2013 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 optparse
 import os
 import subprocess
 import sys
 import time
@@ skipping 85 matching lines @@
 
   if toolchain == platform:
     ppapi_plugin = os.path.join(exe_dir, desc['NAME'])
     if platform == 'win':
       ppapi_plugin += '.dll'
     else:
       ppapi_plugin += '.so'
     args.extend(['--ppapi_plugin', ppapi_plugin])
 
   if toolchain == 'pnacl':
-    args.extend(['--browser_flag', '--enable-pnacl'])
+    args.extend(['--browser_flag'])

[jvoung (off chromium), 2013/07/24 15:46:58]

I don't think you need --browser_flag either.  My ssh is bricked, so I can't
look at the code, but I think it's just a way to pass through browser flags from
the "browser_tester_py" to the actual browser.

[sehr, 2013/07/24 16:57:02]

Yes, what I had would never have worked.  Sorry.

 
   url = 'index.html'
   url += '?tc=%s&config=%s&test=true' % (toolchain, config)
   args.extend(['--url', url])
   return args
 
 
 def GetBrowserTesterEnv():
   # browser_tester imports tools/valgrind/memcheck_analyze, which imports
   # tools/valgrind/common. Well, it tries to, anyway, but instead imports
@@ skipping 198 matching lines @@
 
 
 if __name__ == '__main__':
   script_name = os.path.basename(sys.argv[0])
   try:
     sys.exit(main(sys.argv))
   except parse_dsc.ValidationError as e:
     buildbot_common.ErrorExit('%s: %s' % (script_name, e))
   except KeyboardInterrupt:
     buildbot_common.ErrorExit('%s: interrupted' % script_name)