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

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

Issue 175263003: Add chrome.webstore API methods to allow sites to see progress of installation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Latest master 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
Index: chrome/test/data/extensions/api_test/webstore_inline_install/download_progress_listener.html
diff --git a/chrome/test/data/extensions/api_test/webstore_inline_install/install.html b/chrome/test/data/extensions/api_test/webstore_inline_install/download_progress_listener.html
similarity index 63%
copy from chrome/test/data/extensions/api_test/webstore_inline_install/install.html
copy to chrome/test/data/extensions/api_test/webstore_inline_install/download_progress_listener.html
index 6cf0f673749d7fd7afbddf9852bacfba61350fcc..b2da0b0d8153c56bc3ccfe0b5d485ef92bd3d6dc 100644
--- a/chrome/test/data/extensions/api_test/webstore_inline_install/install.html
+++ b/chrome/test/data/extensions/api_test/webstore_inline_install/download_progress_listener.html
@@ -5,10 +5,8 @@
</head>
<body>
<script>
- console.log('Page loaded');
-
function runTest(galleryUrl) {
- console.log('In runTest');
+ var receivedProgressUpdate = false;
// Link URL has to be generated dynamically in order to include the right
// port number. The ID corresponds to the data in the "extension" directory.
@@ -16,12 +14,22 @@
galleryUrl + '/detail/ecglahbcnmdpdciemllbhojghbkagdje';
try {
+ chrome.webstore.onDownloadProgress.addListener(function(progress) {
+ if (progress < 0 || progress > 1.0) {
+ console.log('Received invalid download progress: ' + progress);
+ window.domAutomationController.send(false);
+ }
+ receivedProgressUpdate = true;
+ });
chrome.webstore.install(
undefined,
function() {
- console.log('Ran test, sending response');
+ if (!receivedProgressUpdate) {
+ console.log(
+ 'Test Failed: Did not receive download progress update.');
+ window.domAutomationController.send(false);
+ }
window.domAutomationController.send(true);
- console.log('Test complete');
},
function(error) {
console.log('Unexpected error: ' + error);
@@ -32,6 +40,7 @@
window.gc();
} catch (e) {
console.log('Unexpected exception: ' + e);
+ console.log(e.stack);
window.domAutomationController.send(false);
throw e;
}

Powered by Google App Engine
This is Rietveld 408576698