Prevented default apps from being installed in ChromeOS tests

chrome/test/pyautolib/pyauto.py

 #!/usr/bin/env python
 # Copyright (c) 2012 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.
 
 """PyAuto: Python Interface to Chromium's Automation Proxy.
 
 PyAuto uses swig to expose Automation Proxy interfaces to Python.
 For complete documentation on the functionality available,
 run pydoc on this file.
@@ skipping 265 matching lines @@
     use a custom set of Chrome flags.
     """
     auth_ext_path = ('/usr/local/autotest/deps/pyauto_dep/' +
         'test_src/chrome/browser/resources/gaia_auth')
     if self.IsChromeOS():
       return [
         '--homepage=about:blank',
         '--allow-file-access',
         '--allow-file-access-from-files',
         '--enable-file-cookies',
+        '--disable-default-apps',

[dpolukhin, 2012/11/08 02:25:34]

Unfortunately it looks like it won't help because on Chrome OS default apps as
external extensions. I suspect that this flag will only affect default apps for
desktop  (will check it tomorrow). See chrome/browser/extensions/default_apps.cc
Provider::ShouldInstallInProfile.

[zel, 2012/11/08 04:37:46]

Can we change chromeos-specific install to start obeying this flag too? Where
does this code reside now?

[Dmitry Polukhin, 2012/11/08 13:34:43]

The code is in ExternalProviderImpl::CreateExternalProviders. Basically we need
to skip external providers registration if this flag set (it should be good
thing to do anyway). I'll try to make this change, I hope it won't break
extensions tests.

[Dmitry Polukhin, 2012/11/08 16:28:14]

CL is here https://codereview.chromium.org/11359105/ but I need to fix tests
before committing it.

         '--dom-automation',
         '--skip-oauth-login',
         # Enables injection of test content script for webui login automation
         '--auth-ext-path=%s' % auth_ext_path,
         # Enable automation provider and chromeos net logs
         '--vmodule=*/browser/automation/*=2,*/chromeos/net/*=2',
       ]
     else:
       return []
 
@@ skipping 6247 matching lines @@
     successful = result.wasSuccessful()
     if not successful:
       pyauto_tests_file = os.path.join(self.TestsDir(), self._tests_filename)
       print >>sys.stderr, 'Tests can be disabled by editing %s. ' \
                           'Ref: %s' % (pyauto_tests_file, _PYAUTO_DOC_URL)
     sys.exit(not successful)
 
 
 if __name__ == '__main__':
   Main()