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

Unified Diff: chrome/test/data/extensions/api_test/webstore_inline_install/install_unpack_failure.html

Issue 384823002: Introduce a stable set of errors for inline install (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Replace printf with vlog(1) in unrelated code so it passes presubmit Created 6 years, 5 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/data/extensions/api_test/webstore_inline_install/install_unpack_failure.html
diff --git a/chrome/test/data/extensions/api_test/webstore_inline_install/install_unpack_failure.html b/chrome/test/data/extensions/api_test/webstore_inline_install/install_unpack_failure.html
index d638582da1aed79ead8f540600d85fcbf6823ef5..16826f8188cbdf9b66592628c1659f0c20c7226c 100644
--- a/chrome/test/data/extensions/api_test/webstore_inline_install/install_unpack_failure.html
+++ b/chrome/test/data/extensions/api_test/webstore_inline_install/install_unpack_failure.html
@@ -18,8 +18,9 @@
console.log('Unexpected install success');
window.domAutomationController.send(false);
},
- function(error) {
- if (error.indexOf('Package is invalid') != -1) {
+ function(error, errorCode) {
+ if (error.indexOf('Package is invalid') != -1 &&
+ errorCode == 'otherError') {
window.domAutomationController.send(true);
} else {
console.log('Unexpected install error: ' + error);

Powered by Google App Engine
This is Rietveld 408576698