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

Unified Diff: chrome/test/data/extensions/api_test/window_open/argument_overflow/test.html

Issue 7042021: Check for integer overflow when validating API function arguments. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Update copyright headers Created 9 years, 7 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/window_open/argument_overflow/test.html
diff --git a/chrome/test/data/extensions/api_test/window_open/argument_overflow/test.html b/chrome/test/data/extensions/api_test/window_open/argument_overflow/test.html
new file mode 100644
index 0000000000000000000000000000000000000000..468ea4f2fb938719df3db5ef018e6693265f65ce
--- /dev/null
+++ b/chrome/test/data/extensions/api_test/window_open/argument_overflow/test.html
@@ -0,0 +1,17 @@
+<script>
+function check_overflow_check(value) {
+ try {
+ chrome.windows.create({ "left": value }, function() { });
+ } catch (e) {
+ chrome.test.assertTrue(e.message.indexOf(
+ "Value must fit in a 32-bit signed integer.") != -1);
+ chrome.test.succeed();
+ return;
+ }
+}
+chrome.test.runTests([
+ function overflow2To31() { check_overflow_check(0x80000000); },
+ function overflowMinus2To31Minus1() { check_overflow_check(-0x80000001); },
+ function overflow2To32() { check_overflow_check(0x100000000); },
+]);
+</script>

Powered by Google App Engine
This is Rietveld 408576698