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

Unified Diff: chrome/test/mini_installer/config/chrome_system_installed.prop

Issue 2224843002: Fix broken installer tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@kill_al_win
Patch Set: address comment Created 4 years, 4 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
Index: chrome/test/mini_installer/config/chrome_system_installed.prop
diff --git a/chrome/test/mini_installer/config/chrome_system_installed.prop b/chrome/test/mini_installer/config/chrome_system_installed.prop
index a11cde454690f6188206658bd4badc45ee2bca61..6b358aa3ca49cf26406abfef186f4a8d3e882572 100644
--- a/chrome/test/mini_installer/config/chrome_system_installed.prop
+++ b/chrome/test/mini_installer/config/chrome_system_installed.prop
@@ -28,10 +28,7 @@
},
"HKEY_LOCAL_MACHINE\\$LAUNCHER_UPDATE_REGISTRY_SUBKEY": {
"condition": "'$CHROME_SHORT_NAME' == 'Chrome'",
- "exists": "required",
- "values": {
- "pv": {"type": "SZ", "data": "$MINI_INSTALLER_FILE_VERSION"}
- }
+ "exists": "forbidden"
},
"HKEY_LOCAL_MACHINE\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Uninstall\\$CHROME_LONG_NAME": {
"exists": "required",

Powered by Google App Engine
This is Rietveld 408576698