Generate a bit less code to handle failed arity checks.

Source/bindings/v8/custom/V8XMLHttpRequestCustom.cpp

 /*
  * Copyright (C) 2008, 2009, 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 146 matching lines @@
     // Four cases:
     // open(method, url)
     // open(method, url, async)
     // open(method, url, async, user)
     // open(method, url, async, user, passwd)
 
     ExceptionState exceptionState(ExceptionState::ExecutionContext, "open", "XMLHttpRequest", info.Holder(), info.GetIsolate());
 
     if (info.Length() < 2) {
         exceptionState.throwTypeError(ExceptionMessages::notEnoughArguments(2, info.Length()));
-    } else {
-        XMLHttpRequest* xmlHttpRequest = V8XMLHttpRequest::toNative(info.Holder());
+        exceptionState.throwIfNeeded();
+        return;
+    }
 
-        V8TRYCATCH_FOR_V8STRINGRESOURCE_VOID(V8StringResource<>, method, info[0]);
-        V8TRYCATCH_FOR_V8STRINGRESOURCE_VOID(V8StringResource<>, urlstring, info[1]);
+    XMLHttpRequest* xmlHttpRequest = V8XMLHttpRequest::toNative(info.Holder());
 
-        ExecutionContext* context = getExecutionContext();
-        KURL url = context->completeURL(urlstring);
+    V8TRYCATCH_FOR_V8STRINGRESOURCE_VOID(V8StringResource<>, method, info[0]);
+    V8TRYCATCH_FOR_V8STRINGRESOURCE_VOID(V8StringResource<>, urlstring, info[1]);
 
-        if (info.Length() >= 3) {
-            bool async = info[2]->BooleanValue();
+    ExecutionContext* context = getExecutionContext();
+    KURL url = context->completeURL(urlstring);
 
-            if (info.Length() >= 4 && !info[3]->IsUndefined()) {
-                V8TRYCATCH_FOR_V8STRINGRESOURCE_VOID(V8StringResource<WithNullCheck>, user, info[3]);
+    if (info.Length() >= 3) {
+        bool async = info[2]->BooleanValue();
 
-                if (info.Length() >= 5 && !info[4]->IsUndefined()) {
-                    V8TRYCATCH_FOR_V8STRINGRESOURCE_VOID(V8StringResource<WithNullCheck>, password, info[4]);
-                    xmlHttpRequest->open(method, url, async, user, password, exceptionState);
-                } else {
-                    xmlHttpRequest->open(method, url, async, user, exceptionState);
-                }
+        if (info.Length() >= 4 && !info[3]->IsUndefined()) {
+            V8TRYCATCH_FOR_V8STRINGRESOURCE_VOID(V8StringResource<WithNullCheck>, user, info[3]);
+
+            if (info.Length() >= 5 && !info[4]->IsUndefined()) {
+                V8TRYCATCH_FOR_V8STRINGRESOURCE_VOID(V8StringResource<WithNullCheck>, password, info[4]);
+                xmlHttpRequest->open(method, url, async, user, password, exceptionState);
             } else {
-                xmlHttpRequest->open(method, url, async, exceptionState);
+                xmlHttpRequest->open(method, url, async, user, exceptionState);
             }
         } else {
-            xmlHttpRequest->open(method, url, exceptionState);
+            xmlHttpRequest->open(method, url, async, exceptionState);
         }
+    } else {
+        xmlHttpRequest->open(method, url, exceptionState);
     }
 
     exceptionState.throwIfNeeded();
 }
 
 static bool isDocumentType(v8::Handle<v8::Value> value, v8::Isolate* isolate, WrapperWorldType currentWorldType)
 {
     // FIXME: add other document types.
     return V8Document::hasInstance(value, isolate, currentWorldType) || V8HTMLDocument::hasInstance(value, isolate, currentWorldType);
 }
@@ skipping 40 matching lines @@
         } else {
             V8TRYCATCH_FOR_V8STRINGRESOURCE_VOID(V8StringResource<WithNullCheck>, argString, arg);
             xmlHttpRequest->send(argString, exceptionState);
         }
     }
 
     exceptionState.throwIfNeeded();
 }
 
 } // namespace WebCore