Bug 394769. Remove the additional lateness argument for timeouts and intervals. r=jst

dom/base/nsGlobalWindow.cpp

@@ -9360,12 +9360,6 @@ nsGlobalWindow::RunTimeout(nsTimeout *aTimeout)
                           handler->GetScriptVersion(), nsnull,
                           &is_undefined);
     } else {
-      // Let the script handler know about the "secret" final argument that
-      // indicates timeout lateness in milliseconds
-      TimeDuration lateness = now - timeout->mWhen;
-
-      handler->SetLateness(lateness.ToMilliseconds());
-
       nsCOMPtr<nsIVariant> dummy;
       nsCOMPtr<nsISupports> me(static_cast<nsIDOMWindow *>(this));
       scx->CallEventHandler(me, FastGetGlobalJSObject(),

dom/base/nsIScriptTimeoutHandler.h

@@ -42,9 +42,9 @@
 class nsIArray;
 
 #define NS_ISCRIPTTIMEOUTHANDLER_IID \
-{ /* {21ba4f96-30b8-4215-a75d-d438eb16a50c} */ \
+{ /* {17a9ce1a-d73b-45d1-8145-a0ae57bcc76e} */ \
-  0x21ba4f96, 0x30b8, 0x4215, \
+  0x17a9ce1a, 0xd73b, 0x45d1, \
-  { 0xa7, 0x5d, 0xd4, 0x38, 0xeb, 0x16, 0xa5, 0x0c } }
+ { 0x81, 0x45, 0xa0, 0xae, 0x57, 0xbc, 0xc7, 0x6e } }
 
 /**
  * Abstraction of the script objects etc required to do timeouts in a
@@ -78,10 +78,6 @@ public:
 
   // Get the language version for this timeout.
   virtual PRUint32 GetScriptVersion() = 0;
-
-  // Set the "secret" final lateness arg.  This will be called before
-  // GetArgv(), which should reflect this lateness value.
-  virtual void SetLateness(PRIntervalTime aHowLate) = 0;
 };
 
 NS_DEFINE_STATIC_IID_ACCESSOR(nsIScriptTimeoutHandler,

dom/base/nsJSEnvironment.cpp

@@ -4117,10 +4117,6 @@ NS_IMPL_CYCLE_COLLECTING_RELEASE(nsJSArgArray)
 nsresult
 nsJSArgArray::GetArgs(PRUint32 *argc, void **argv)
 {
-  if (!mArgv) {
-    NS_WARNING("nsJSArgArray has no argv!");
-    return NS_ERROR_UNEXPECTED;
-  }
   *argv = (void *)mArgv;
   *argc = mArgc;
   return NS_OK;

dom/base/nsJSTimeoutHandler.cpp

@@ -51,6 +51,7 @@
 #include "nsDOMError.h"
 #include "nsGlobalWindow.h"
 #include "nsIContentSecurityPolicy.h"
+#include "nsAlgorithm.h"
 
 static const char kSetIntervalStr[] = "setInterval";
 static const char kSetTimeoutStr[] = "setTimeout";
@@ -85,9 +86,6 @@ public:
   virtual nsIArray *GetArgv() {
     return mArgv;
   }
-  // Called by the timeout mechanism so the secret 'lateness' arg can be
-  // added.
-  virtual void SetLateness(PRIntervalTime aHowLate);
 
   nsresult Init(nsGlobalWindow *aWindow, bool *aIsInterval,
                 PRInt32 *aInterval);
@@ -324,11 +322,13 @@ nsJSScriptTimeoutHandler::Init(nsGlobalWindow *aWindow, bool *aIsInterval,
 
     mFunObj = funobj;
 
-    // Create our arg array - leave an extra slot for a secret final argument
+    // Create our arg array.  argc is the number of arguments passed
-    // that indicates to the called function how "late" the timeout is.  We
+    // to setTimeout or setInterval; the first two are our callback
-    // will fill that in when SetLateness is called.
+    // and the delay, so only arguments after that need to go in our
+    // array.
     nsCOMPtr<nsIArray> array;
-    rv = NS_CreateJSArgv(cx, (argc > 1) ? argc - 1 : argc, nsnull,
+    // NS_MAX(argc - 2, 0) wouldn't work right because argc is unsigned.
+    rv = NS_CreateJSArgv(cx, NS_MAX(argc, 2u) - 2, nsnull,
                          getter_AddRefs(array));
     if (NS_FAILED(rv)) {
       return NS_ERROR_OUT_OF_MEMORY;
@@ -339,12 +339,14 @@ nsJSScriptTimeoutHandler::Init(nsGlobalWindow *aWindow, bool *aIsInterval,
     nsCOMPtr<nsIJSArgArray> jsarray(do_QueryInterface(array));
     jsarray->GetArgs(&dummy, reinterpret_cast<void **>(&jsargv));
 
-    // must have worked - we own the impl! :)
+    // jsargv might be null if we have argc <= 2
-    NS_ASSERTION(jsargv, "No argv!");
+    if (jsargv) {
       for (PRInt32 i = 2; (PRUint32)i < argc; ++i) {
         jsargv[i - 2] = argv[i];
       }
-    // final arg slot remains null, array has rooted vals.
+    } else {
+      NS_ASSERTION(argc <= 2, "Why do we have no jsargv when we have arguments?");
+    }
     mArgv = array;
   } else {
     NS_WARNING("No func and no expr - why are we here?");
@@ -353,20 +355,6 @@ nsJSScriptTimeoutHandler::Init(nsGlobalWindow *aWindow, bool *aIsInterval,
   return NS_OK;
 }
 
-void nsJSScriptTimeoutHandler::SetLateness(PRIntervalTime aHowLate)
-{
-  nsCOMPtr<nsIJSArgArray> jsarray(do_QueryInterface(mArgv));
-  if (jsarray) {
-    PRUint32 argc;
-    jsval *jsargv;
-    nsresult rv = jsarray->GetArgs(&argc, reinterpret_cast<void **>(&jsargv));
-    if (NS_SUCCEEDED(rv) && jsargv && argc)
-      jsargv[argc-1] = INT_TO_JSVAL((jsint) aHowLate);
-  } else {
-    NS_ERROR("How can our argv not handle this?");
-  }
-}
-
 const PRUnichar *
 nsJSScriptTimeoutHandler::GetHandlerText()
 {

dom/tests/mochitest/bugs/Makefile.in

@@ -73,6 +73,7 @@ _TEST_FILES	= \
 		test_bug384122.html \
 		test_bug389366.html \
 		test_bug393974.html \
+		test_bug394769.html \
 		test_bug396843.html \
 		test_bug397571.html \
 		test_bug400204.html \

dom/tests/mochitest/bugs/test_bug394769.html

@@ -0,0 +1,42 @@
+<!DOCTYPE HTML>
+<html>
+<!--
+https://bugzilla.mozilla.org/show_bug.cgi?id=394769
+-->
+<head>
+  <meta charset="utf-8">
+  <title>Test for Bug 394769</title>
+  <script type="application/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
+  <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css"/>
+</head>
+<body>
+<a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=394769">Mozilla Bug 394769</a>
+<p id="display"></p>
+<div id="content" style="display: none">
+  
+</div>
+<pre id="test">
+<script type="application/javascript">
+
+/** Test for Bug 394769 **/
+SimpleTest.waitForExplicitFinish();
+function h() {
+  is(arguments.length, 1, "Should only have one argument");
+  SimpleTest.finish();
+}
+
+function g() {
+  is(arguments.length, 0, "Should not have lateness argument for function with delay");
+  setTimeout(h, 0, "arg");
+}
+
+function f() {
+  is(arguments.length, 0, "Should not have lateness argument for function with no delay");
+  setTimeout(g, 0);
+}
+
+setTimeout(f);
+</script>
+</pre>
+</body>
+</html>