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

Unified Diff: client/site_tests/power_LoadTest/extension/ct.js

Issue 669230: Update Power Test to be 60/20/10 test. (Closed)
Patch Set: Fix issues pointed out by snanda and petkov Created 10 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: client/site_tests/power_LoadTest/extension/ct.js
diff --git a/client/site_tests/power_LoadTest/extension/ct.js b/client/site_tests/power_LoadTest/extension/ct.js
index b27f246096b611ec1161dfc502906102367c8d50..64eb1056bf85352e54c13e5c2ebef1b15f17a0a5 100755
--- a/client/site_tests/power_LoadTest/extension/ct.js
+++ b/client/site_tests/power_LoadTest/extension/ct.js
@@ -6,31 +6,28 @@ request = {action: "should_scroll"}
chrome.extension.sendRequest(request, function(response) {
if (response.should_scroll) {
+ window.focus();
lastOffset = window.pageYOffset;
- function smoothScrollDown()
- {
+ var start_interval = Math.max(10000, response.scroll_interval);
+ function smoothScrollDown() {
window.scrollBy(0, response.scroll_by);
if (window.pageYOffset != lastOffset) {
lastOffset = window.pageYOffset;
setTimeout(smoothScrollDown, response.scroll_interval);
- }
- else {
- if (response.should_scroll_up) {
- setTimeout(smoothScrollUp, 5000);
- }
+ } else if (response.should_scroll_up) {
+ setTimeout(smoothScrollUp, start_interval);
}
}
- function smoothScrollUp()
- {
+ function smoothScrollUp() {
window.scrollBy(0, -1 * response.scroll_by);
if (window.pageYOffset != lastOffset) {
lastOffset = window.pageYOffset;
- if (response.scroll_loop) {
- setTimeout(smoothScrollUp, response.scroll_interval);
- }
+ setTimeout(smoothScrollUp, response.scroll_interval);
+ } else if (response.scroll_loop) {
+ setTimeout(smoothScrollDown, start_interval);
}
}
- setTimeout(smoothScrollDown, 10000);
+ setTimeout(smoothScrollDown, start_interval);
}
});
« no previous file with comments | « client/site_tests/power_LoadTest/extension/background.html ('k') | client/site_tests/power_LoadTest/extension/manifest.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698