Fixes a crash when entering a negative index with chrome.tabs.move ....

chrome/test/data/extensions/json_schema_test.js

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 function assert(truth) {
   if (!truth)
     throw new Error("Assertion failed.");
 }
 
 function assertValid(type, instance, schema, types) {
@@ skipping 401 matching lines @@
 
   assertNotValid("Number", Number.POSITIVE_INFINITY, schema,
                  [formatError("numberFiniteNotNan", ["Infinity"]),
                   formatError("numberMaxValue", [schema.maximum])
                  ]);
 
   assertNotValid("Number", Number.NEGATIVE_INFINITY, schema,
                  [formatError("numberFiniteNotNan", ["-Infinity"]),
                   formatError("numberMinValue", [schema.minimum])
                  ]);
+
+  //0 casts to bool differently, so it's important to test

[Aaron Boodman, 2010/11/16 20:21:28]

Space after "//". Add a period to the end of the comment. And again, add a
reference to the bug.

+  var zero_min_schema = {
+    'minimum': 0,
+    'maximum': 100
+  };
+  assertNotValid("Number", -1, zero_min_schema,
+                 [formatError("numberMinValue", [zero_min_schema.minimum])]);
+  assertValid("Number", 0, zero_min_schema);
+
+  var zero_max_schema = {
+    'minimum': -20,
+    'maximum': '0'
+  };
+  assertNotValid("Number", 1, zero_max_schema,
+                 [formatError("numberMaxValue", [zero_max_schema.maximum])]);
+  assertValid("Number", 0, zero_max_schema);
+
 }
 
 function testType() {
   // valid
   assertValid("Type", {}, {type:"object"});
   assertValid("Type", [], {type:"array"});
   assertValid("Type", function(){}, {type:"function"});
   assertValid("Type", "foobar", {type:"string"});
   assertValid("Type", "", {type:"string"});
   assertValid("Type", 88.8, {type:"number"});
@@ skipping 23 matching lines @@
                  [formatError("invalidType", ["integer", "number"])]);
   assertNotValid("Type", 1, {type: "boolean"},
                  [formatError("invalidType", ["boolean", "integer"])]);
   assertNotValid("Type", false, {type: "null"},
                  [formatError("invalidType", ["null", "boolean"])]);
   assertNotValid("Type", undefined, {type: "null"},
                  [formatError("invalidType", ["null", "undefined"])]);
   assertNotValid("Type", {}, {type: "function"},
                  [formatError("invalidType", ["function", "object"])]);
 }