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

Unified Diff: chrome/test/mini_installer/variable_expander.py

Issue 214823006: Move ContentBrowserTest class to content/public, since it's used by components_browsertests for bro… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: fix mac Created 6 years, 9 months 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
« no previous file with comments | « no previous file | components/dom_distiller/content/DEPS » ('j') | content/child/child_thread.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/mini_installer/variable_expander.py
===================================================================
--- chrome/test/mini_installer/variable_expander.py (revision 259922)
+++ chrome/test/mini_installer/variable_expander.py (working copy)
@@ -108,8 +108,8 @@
chrome_update_registry_subkey = 'Software\\Chromium'
chrome_html_prog_id = 'ChromiumHTM'
else:
- raise KeyError("Unknown mini_installer product name '%s'" %
- mini_installer_product_name)
+ raise KeyError("Unknown mini_installer product name '%s' for %s" %
+ (mini_installer_product_name, mini_installer_abspath))
Avi (use Gerrit) 2014/03/27 18:36:50 What's this doing in this CL?
self._variable_mapping = {
'PROGRAM_FILES': program_files_path,
« no previous file with comments | « no previous file | components/dom_distiller/content/DEPS » ('j') | content/child/child_thread.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698