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

Unified Diff: chrome/test/data/extensions/api_test/settings/simple_test/background.js

Issue 129603002: Re-enable ExtensionSettingsApiTest after removing the "throttling" JS test (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 11 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 | « chrome/browser/extensions/api/storage/settings_apitest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/extensions/api_test/settings/simple_test/background.js
diff --git a/chrome/test/data/extensions/api_test/settings/simple_test/background.js b/chrome/test/data/extensions/api_test/settings/simple_test/background.js
index 02fe31824bc320dfbc0d42a85148af4a0763a538..f641a6033d0d5b2891ed17137346a856578e3d82 100644
--- a/chrome/test/data/extensions/api_test/settings/simple_test/background.js
+++ b/chrome/test/data/extensions/api_test/settings/simple_test/background.js
@@ -362,41 +362,4 @@ chrome.test.runTests([
}
area.clear(stage0);
},
-
- // NOTE: throttling test must come last, since each test runs with a single
- // quota.
- function throttling() {
- // Test script is as so:
- // 1 - storage.local shouldn't be exceeded.
- // 2 - storage.sync should be exceeded.
- // 3 - storage.local still shouldn't be exceeded.
- // 4 - storage.sync should still be exceeded.
- //
- // In general, things should get throttled after 1000 calls (though in
- // reality will be fewer due to previous tests).
-
- function clearNTimes(area, n, whenDone) {
- if (n <= 0) {
- whenDone();
- } else {
- area.clear(function() {
- clearNTimes(area, n - 1, whenDone);
- });
- }
- }
-
- var local = chrome.storage.local;
- var sync = chrome.storage.sync;
- var test = chrome.test;
- var quotaError =
- "This request exceeds the MAX_WRITE_OPERATIONS_PER_HOUR quota.";
-
- clearNTimes(local, 1001, test.callbackPass(function() {
- clearNTimes(sync, 1001, test.callbackFail(quotaError, function() {
- clearNTimes(local, 1, test.callbackPass(function() {
- clearNTimes(sync, 1, test.callbackFail(quotaError, test.succeed));
- }));
- }));
- }));
- },
]);
« no previous file with comments | « chrome/browser/extensions/api/storage/settings_apitest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698