disable 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 87 matching lines @@
   exe_dir = GetExecutableDirForProject(desc, toolchain, config)
 
   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':

[binji, 2013/08/12 17:59:35]

I don't think this should be removed, should it?

[sehr, 2013/08/12 18:43:14]

Oops.  Thanks.

-    args.extend(['--browser_flag', '--enable-pnacl'])
-
   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
   # common from PYTHONPATH first (which on the buildbots, is a
@@ skipping 197 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)