Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5384)

Unified Diff: chrome/test/functional/chromeos_security.py

Issue 8649004: Allow chromedriver to install an extension and get all installed extension IDs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: ... Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/functional/chromeos_security.py
diff --git a/chrome/test/functional/chromeos_security.py b/chrome/test/functional/chromeos_security.py
index 40be9f6e9d5be65365886aafd0790ec77d2b9238..5ecd2ac54c6591d0a5729e2a1bdf71b6b92451d0 100644
--- a/chrome/test/functional/chromeos_security.py
+++ b/chrome/test/functional/chromeos_security.py
@@ -169,8 +169,7 @@ class ChromeosSecurity(pyauto.PyUITest):
file_name in [x['crx_file'] for x in self._bundled_crx_baseline],
msg='Unexpected CRX file: ' + file_name)
crx_file = os.path.join(self._bundled_crx_directory, file_name)
- self.assertTrue(self.InstallExtension(crx_file, False),
- msg='Extension install failed: %s' % crx_file)
+ self.InstallExtension(crx_file)
# Verify that the permissions information in the baseline matches the
# permissions associated with the installed bundled CRX extensions.
@@ -182,8 +181,7 @@ class ChromeosSecurity(pyauto.PyUITest):
for file_name in os.listdir(self._bundled_crx_directory):
if file_name.endswith('.crx'):
crx_file = os.path.join(self._bundled_crx_directory, file_name)
- self.assertTrue(self.InstallExtension(crx_file, False),
- msg='Extension install failed: %s' % crx_file)
+ self.InstallExtension(crx_file)
# Ensure that the set of installed extension names precisely matches the
# baseline.

Powered by Google App Engine
This is Rietveld 408576698