<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[200286] trunk/Source/WebCore</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/200286">200286</a></dd>
<dt>Author</dt> <dd>cdumez@apple.com</dd>
<dt>Date</dt> <dd>2016-04-29 21:48:34 -0700 (Fri, 29 Apr 2016)</dd>
</dl>
<h3>Log Message</h3>
<pre>Use LIKELY() / UNLIKELY() hints when suitable in the JavaScript bindings
https://bugs.webkit.org/show_bug.cgi?id=157210
Reviewed by Darin Adler.
Use LIKELY() / UNLIKELY() hints when suitable in the JavaScript bindings.
We already make use of them in the JS bindings but they are some cases
where we don't and they may be useful.
* bindings/scripts/CodeGeneratorJS.pm:
(GenerateOverloadedFunction):
(GenerateImplementation):
(GenerateParametersCheck):
(GenerateImplementationFunctionCall):
(GenerateOverloadedConstructorDefinition):
(GenerateConstructorDefinition):
* bindings/scripts/test/JS/JSTestActiveDOMObject.cpp:
(WebCore::jsTestActiveDOMObjectConstructor):
(WebCore::jsTestActiveDOMObjectPrototypeFunctionExcitingFunction):
* bindings/scripts/test/JS/JSTestClassWithJSBuiltinConstructor.cpp:
(WebCore::jsTestClassWithJSBuiltinConstructorConstructor):
* bindings/scripts/test/JS/JSTestCustomConstructorWithNoInterfaceObject.cpp:
(WebCore::jsTestCustomConstructorWithNoInterfaceObjectConstructor):
* bindings/scripts/test/JS/JSTestCustomNamedGetter.cpp:
(WebCore::jsTestCustomNamedGetterConstructor):
* bindings/scripts/test/JS/JSTestEventConstructor.cpp:
(WebCore::jsTestEventConstructorConstructor):
* bindings/scripts/test/JS/JSTestEventTarget.cpp:
(WebCore::JSTestEventTarget::getOwnPropertySlotByIndex):
(WebCore::jsTestEventTargetConstructor):
* bindings/scripts/test/JS/JSTestException.cpp:
(WebCore::jsTestExceptionConstructor):
* bindings/scripts/test/JS/JSTestGenerateIsReachable.cpp:
(WebCore::jsTestGenerateIsReachableConstructor):
* bindings/scripts/test/JS/JSTestGlobalObject.cpp:
(WebCore::jsTestGlobalObjectConstructor):
(WebCore::jsTestGlobalObjectInstanceFunctionEnabledAtRuntimeOperation):
* bindings/scripts/test/JS/JSTestInterface.cpp:
(WebCore::JSTestInterfaceConstructor::construct):
(WebCore::jsTestInterfaceConstructor):
(WebCore::jsTestInterfacePrototypeFunctionImplementsMethod2):
(WebCore::jsTestInterfacePrototypeFunctionSupplementalMethod2):
* bindings/scripts/test/JS/JSTestJSBuiltinConstructor.cpp:
(WebCore::jsTestJSBuiltinConstructorConstructor):
* bindings/scripts/test/JS/JSTestMediaQueryListListener.cpp:
(WebCore::jsTestMediaQueryListListenerConstructor):
(WebCore::jsTestMediaQueryListListenerPrototypeFunctionMethod):
* bindings/scripts/test/JS/JSTestNamedConstructor.cpp:
(WebCore::JSTestNamedConstructorNamedConstructor::construct):
(WebCore::jsTestNamedConstructorConstructor):
* bindings/scripts/test/JS/JSTestNode.cpp:
(WebCore::jsTestNodeConstructor):
* bindings/scripts/test/JS/JSTestNondeterministic.cpp:
(WebCore::jsTestNondeterministicConstructor):
* bindings/scripts/test/JS/JSTestObj.cpp:
(WebCore::JSTestObjConstructor::construct):
(WebCore::JSTestObj::getOwnPropertySlotByIndex):
(WebCore::jsTestObjTestSubObjEnabledBySettingConstructor):
(WebCore::jsTestObjConstructor):
(WebCore::jsTestObjPrototypeFunctionEnabledAtRuntimeOperation):
(WebCore::jsTestObjPrototypeFunctionVoidMethodWithArgs):
(WebCore::jsTestObjPrototypeFunctionByteMethodWithArgs):
(WebCore::jsTestObjPrototypeFunctionOctetMethodWithArgs):
(WebCore::jsTestObjPrototypeFunctionLongMethodWithArgs):
(WebCore::jsTestObjPrototypeFunctionObjMethodWithArgs):
(WebCore::jsTestObjPrototypeFunctionMethodThatRequiresAllArgsAndThrows):
(WebCore::jsTestObjPrototypeFunctionMethodWithCallbackArg):
(WebCore::jsTestObjPrototypeFunctionMethodWithNonCallbackArgAndCallbackArg):
(WebCore::jsTestObjPrototypeFunctionMethodWithCallbackFunctionArg):
(WebCore::jsTestObjPrototypeFunctionMethodWithNonCallbackArgAndCallbackFunctionArg):
(WebCore::jsTestObjConstructorFunctionStaticMethodWithCallbackArg):
(WebCore::jsTestObjPrototypeFunctionOverloadedMethod5):
(WebCore::jsTestObjPrototypeFunctionOverloadedMethod8):
(WebCore::jsTestObjPrototypeFunctionOverloadedMethod):
(WebCore::jsTestObjPrototypeFunctionOverloadedMethodWithOptionalParameter):
(WebCore::jsTestObjConstructorFunctionOverloadedMethod1):
(WebCore::jsTestObjPrototypeFunctionConvert1):
(WebCore::jsTestObjPrototypeFunctionStrictFunctionWithSequence):
(WebCore::jsTestObjPrototypeFunctionStrictFunctionWithArray):
(WebCore::jsTestObjPrototypeFunctionVariadicNodeMethod):
(WebCore::jsTestObjPrototypeFunctionTestPromiseFunctionWithFloatArgumentPromise):
(WebCore::jsTestObjPrototypeFunctionTestPromiseOverloadedFunction1Promise):
(WebCore::jsTestObjPrototypeFunctionTestPromiseOverloadedFunction2Promise):
(WebCore::jsTestObjPrototypeFunctionTestPromiseOverloadedFunction):
* bindings/scripts/test/JS/JSTestOverloadedConstructors.cpp:
(WebCore::constructJSTestOverloadedConstructors1):
(WebCore::constructJSTestOverloadedConstructors3):
(WebCore::JSTestOverloadedConstructorsConstructor::construct):
(WebCore::jsTestOverloadedConstructorsConstructor):
* bindings/scripts/test/JS/JSTestOverrideBuiltins.cpp:
(WebCore::jsTestOverrideBuiltinsConstructor):
* bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp:
(WebCore::jsTestSerializedScriptValueInterfaceConstructor):
* bindings/scripts/test/JS/JSTestTypedefs.cpp:
(WebCore::JSTestTypedefsConstructor::construct):
(WebCore::jsTestTypedefsConstructor):
* bindings/scripts/test/JS/JSattribute.cpp:
(WebCore::jsattributeConstructor):
* bindings/scripts/test/JS/JSreadonly.cpp:
(WebCore::jsreadonlyConstructor):</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptsCodeGeneratorJSpm">trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestActiveDOMObjectcpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestActiveDOMObject.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestClassWithJSBuiltinConstructorcpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestClassWithJSBuiltinConstructor.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestCustomConstructorWithNoInterfaceObjectcpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestCustomConstructorWithNoInterfaceObject.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestCustomNamedGettercpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestCustomNamedGetter.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestEventConstructorcpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestEventConstructor.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestEventTargetcpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestEventTarget.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestExceptioncpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestException.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestGenerateIsReachablecpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestGenerateIsReachable.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestGlobalObjectcpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestGlobalObject.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestInterfacecpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestInterface.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestJSBuiltinConstructorcpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestJSBuiltinConstructor.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestMediaQueryListListenercpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestMediaQueryListListener.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestNamedConstructorcpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestNamedConstructor.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestNodecpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestNode.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestNondeterministiccpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestNondeterministic.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestObjcpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestObj.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestOverloadedConstructorscpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestOverloadedConstructors.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestOverrideBuiltinscpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestOverrideBuiltins.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestSerializedScriptValueInterfacecpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSTestTypedefscpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSTestTypedefs.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSattributecpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSattribute.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsscriptstestJSJSreadonlycpp">trunk/Source/WebCore/bindings/scripts/test/JS/JSreadonly.cpp</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/ChangeLog 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -1,3 +1,106 @@
</span><ins>+2016-04-29 Chris Dumez <cdumez@apple.com>
+
+ Use LIKELY() / UNLIKELY() hints when suitable in the JavaScript bindings
+ https://bugs.webkit.org/show_bug.cgi?id=157210
+
+ Reviewed by Darin Adler.
+
+ Use LIKELY() / UNLIKELY() hints when suitable in the JavaScript bindings.
+ We already make use of them in the JS bindings but they are some cases
+ where we don't and they may be useful.
+
+ * bindings/scripts/CodeGeneratorJS.pm:
+ (GenerateOverloadedFunction):
+ (GenerateImplementation):
+ (GenerateParametersCheck):
+ (GenerateImplementationFunctionCall):
+ (GenerateOverloadedConstructorDefinition):
+ (GenerateConstructorDefinition):
+ * bindings/scripts/test/JS/JSTestActiveDOMObject.cpp:
+ (WebCore::jsTestActiveDOMObjectConstructor):
+ (WebCore::jsTestActiveDOMObjectPrototypeFunctionExcitingFunction):
+ * bindings/scripts/test/JS/JSTestClassWithJSBuiltinConstructor.cpp:
+ (WebCore::jsTestClassWithJSBuiltinConstructorConstructor):
+ * bindings/scripts/test/JS/JSTestCustomConstructorWithNoInterfaceObject.cpp:
+ (WebCore::jsTestCustomConstructorWithNoInterfaceObjectConstructor):
+ * bindings/scripts/test/JS/JSTestCustomNamedGetter.cpp:
+ (WebCore::jsTestCustomNamedGetterConstructor):
+ * bindings/scripts/test/JS/JSTestEventConstructor.cpp:
+ (WebCore::jsTestEventConstructorConstructor):
+ * bindings/scripts/test/JS/JSTestEventTarget.cpp:
+ (WebCore::JSTestEventTarget::getOwnPropertySlotByIndex):
+ (WebCore::jsTestEventTargetConstructor):
+ * bindings/scripts/test/JS/JSTestException.cpp:
+ (WebCore::jsTestExceptionConstructor):
+ * bindings/scripts/test/JS/JSTestGenerateIsReachable.cpp:
+ (WebCore::jsTestGenerateIsReachableConstructor):
+ * bindings/scripts/test/JS/JSTestGlobalObject.cpp:
+ (WebCore::jsTestGlobalObjectConstructor):
+ (WebCore::jsTestGlobalObjectInstanceFunctionEnabledAtRuntimeOperation):
+ * bindings/scripts/test/JS/JSTestInterface.cpp:
+ (WebCore::JSTestInterfaceConstructor::construct):
+ (WebCore::jsTestInterfaceConstructor):
+ (WebCore::jsTestInterfacePrototypeFunctionImplementsMethod2):
+ (WebCore::jsTestInterfacePrototypeFunctionSupplementalMethod2):
+ * bindings/scripts/test/JS/JSTestJSBuiltinConstructor.cpp:
+ (WebCore::jsTestJSBuiltinConstructorConstructor):
+ * bindings/scripts/test/JS/JSTestMediaQueryListListener.cpp:
+ (WebCore::jsTestMediaQueryListListenerConstructor):
+ (WebCore::jsTestMediaQueryListListenerPrototypeFunctionMethod):
+ * bindings/scripts/test/JS/JSTestNamedConstructor.cpp:
+ (WebCore::JSTestNamedConstructorNamedConstructor::construct):
+ (WebCore::jsTestNamedConstructorConstructor):
+ * bindings/scripts/test/JS/JSTestNode.cpp:
+ (WebCore::jsTestNodeConstructor):
+ * bindings/scripts/test/JS/JSTestNondeterministic.cpp:
+ (WebCore::jsTestNondeterministicConstructor):
+ * bindings/scripts/test/JS/JSTestObj.cpp:
+ (WebCore::JSTestObjConstructor::construct):
+ (WebCore::JSTestObj::getOwnPropertySlotByIndex):
+ (WebCore::jsTestObjTestSubObjEnabledBySettingConstructor):
+ (WebCore::jsTestObjConstructor):
+ (WebCore::jsTestObjPrototypeFunctionEnabledAtRuntimeOperation):
+ (WebCore::jsTestObjPrototypeFunctionVoidMethodWithArgs):
+ (WebCore::jsTestObjPrototypeFunctionByteMethodWithArgs):
+ (WebCore::jsTestObjPrototypeFunctionOctetMethodWithArgs):
+ (WebCore::jsTestObjPrototypeFunctionLongMethodWithArgs):
+ (WebCore::jsTestObjPrototypeFunctionObjMethodWithArgs):
+ (WebCore::jsTestObjPrototypeFunctionMethodThatRequiresAllArgsAndThrows):
+ (WebCore::jsTestObjPrototypeFunctionMethodWithCallbackArg):
+ (WebCore::jsTestObjPrototypeFunctionMethodWithNonCallbackArgAndCallbackArg):
+ (WebCore::jsTestObjPrototypeFunctionMethodWithCallbackFunctionArg):
+ (WebCore::jsTestObjPrototypeFunctionMethodWithNonCallbackArgAndCallbackFunctionArg):
+ (WebCore::jsTestObjConstructorFunctionStaticMethodWithCallbackArg):
+ (WebCore::jsTestObjPrototypeFunctionOverloadedMethod5):
+ (WebCore::jsTestObjPrototypeFunctionOverloadedMethod8):
+ (WebCore::jsTestObjPrototypeFunctionOverloadedMethod):
+ (WebCore::jsTestObjPrototypeFunctionOverloadedMethodWithOptionalParameter):
+ (WebCore::jsTestObjConstructorFunctionOverloadedMethod1):
+ (WebCore::jsTestObjPrototypeFunctionConvert1):
+ (WebCore::jsTestObjPrototypeFunctionStrictFunctionWithSequence):
+ (WebCore::jsTestObjPrototypeFunctionStrictFunctionWithArray):
+ (WebCore::jsTestObjPrototypeFunctionVariadicNodeMethod):
+ (WebCore::jsTestObjPrototypeFunctionTestPromiseFunctionWithFloatArgumentPromise):
+ (WebCore::jsTestObjPrototypeFunctionTestPromiseOverloadedFunction1Promise):
+ (WebCore::jsTestObjPrototypeFunctionTestPromiseOverloadedFunction2Promise):
+ (WebCore::jsTestObjPrototypeFunctionTestPromiseOverloadedFunction):
+ * bindings/scripts/test/JS/JSTestOverloadedConstructors.cpp:
+ (WebCore::constructJSTestOverloadedConstructors1):
+ (WebCore::constructJSTestOverloadedConstructors3):
+ (WebCore::JSTestOverloadedConstructorsConstructor::construct):
+ (WebCore::jsTestOverloadedConstructorsConstructor):
+ * bindings/scripts/test/JS/JSTestOverrideBuiltins.cpp:
+ (WebCore::jsTestOverrideBuiltinsConstructor):
+ * bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp:
+ (WebCore::jsTestSerializedScriptValueInterfaceConstructor):
+ * bindings/scripts/test/JS/JSTestTypedefs.cpp:
+ (WebCore::JSTestTypedefsConstructor::construct):
+ (WebCore::jsTestTypedefsConstructor):
+ * bindings/scripts/test/JS/JSattribute.cpp:
+ (WebCore::jsattributeConstructor):
+ * bindings/scripts/test/JS/JSreadonly.cpp:
+ (WebCore::jsreadonlyConstructor):
+
</ins><span class="cx"> 2016-04-29 Ryosuke Niwa <rniwa@webkit.org>
</span><span class="cx">
</span><span class="cx"> Rename getAssignedNodes to assignedNodes and support flattened option
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptsCodeGeneratorJSpm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -1612,7 +1612,7 @@
</span><span class="cx">
</span><span class="cx"> }
</span><span class="cx"> if ($leastNumMandatoryParams >= 1) {
</span><del>- push(@implContent, " if (argsCount < $leastNumMandatoryParams)\n");
</del><ins>+ push(@implContent, " if (UNLIKELY(argsCount < $leastNumMandatoryParams))\n");
</ins><span class="cx"> push(@implContent, " return throwVMError(state, createNotEnoughArgumentsError(state));\n");
</span><span class="cx"> }
</span><span class="cx"> push(@implContent, <<END);
</span><span class="lines">@@ -2290,9 +2290,9 @@
</span><span class="cx">
</span><span class="cx"> if ($indexedGetterFunction) {
</span><span class="cx"> if ($indexedGetterFunction->signature->type eq "DOMString") {
</span><del>- push(@implContent, " if (index <= MAX_ARRAY_INDEX) {\n");
</del><ins>+ push(@implContent, " if (LIKELY(index <= MAX_ARRAY_INDEX)) {\n");
</ins><span class="cx"> } else {
</span><del>- push(@implContent, " if (index < thisObject->wrapped().length()) {\n");
</del><ins>+ push(@implContent, " if (LIKELY(index < thisObject->wrapped().length())) {\n");
</ins><span class="cx"> }
</span><span class="cx"> # Assume that if there's a setter, the index will be writable
</span><span class="cx"> if ($interface->extendedAttributes->{"CustomIndexedSetter"}) {
</span><span class="lines">@@ -2402,7 +2402,7 @@
</span><span class="cx"> AddToImplIncludes("Frame.h");
</span><span class="cx"> AddToImplIncludes("Settings.h");
</span><span class="cx"> my $enable_function = ToMethodName($attribute->signature->extendedAttributes->{"EnabledBySetting"}) . "Enabled";
</span><del>- push(@implContent, " if (!castedThis->wrapped().frame())\n");
</del><ins>+ push(@implContent, " if (UNLIKELY(!castedThis->wrapped().frame()))\n");
</ins><span class="cx"> push(@implContent, " return JSValue::encode(jsUndefined());\n");
</span><span class="cx"> push(@implContent, " Settings& settings = castedThis->wrapped().frame()->settings();\n");
</span><span class="cx"> push(@implContent, " if (!settings.$enable_function())\n");
</span><span class="lines">@@ -2558,7 +2558,7 @@
</span><span class="cx"> push(@implContent, "EncodedJSValue ${constructorFunctionName}(ExecState* state, EncodedJSValue thisValue, PropertyName)\n");
</span><span class="cx"> push(@implContent, "{\n");
</span><span class="cx"> push(@implContent, " ${className}Prototype* domObject = jsDynamicCast<${className}Prototype*>(JSValue::decode(thisValue));\n");
</span><del>- push(@implContent, " if (!domObject)\n");
</del><ins>+ push(@implContent, " if (UNLIKELY(!domObject))\n");
</ins><span class="cx"> push(@implContent, " return throwVMTypeError(state);\n");
</span><span class="cx">
</span><span class="cx"> if (!$interface->extendedAttributes->{"NoInterfaceObject"}) {
</span><span class="lines">@@ -2619,7 +2619,7 @@
</span><span class="cx"> push(@implContent, " ASSERT_GC_OBJECT_INHERITS(thisObject, info());\n");
</span><span class="cx">
</span><span class="cx"> if ($interface->extendedAttributes->{"CustomIndexedSetter"}) {
</span><del>- push(@implContent, " if (index <= MAX_ARRAY_INDEX) {\n");
</del><ins>+ push(@implContent, " if (LIKELY(index <= MAX_ARRAY_INDEX)) {\n");
</ins><span class="cx"> push(@implContent, " thisObject->indexSetter(state, index, value);\n");
</span><span class="cx"> push(@implContent, " return true;\n");
</span><span class="cx"> push(@implContent, " }\n");
</span><span class="lines">@@ -2785,7 +2785,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> if ($attribute->signature->type eq "double" or $attribute->signature->type eq "float") {
</span><del>- push(@implContent, " if (!std::isfinite(nativeValue)) {\n");
</del><ins>+ push(@implContent, " if (UNLIKELY(!std::isfinite(nativeValue))) {\n");
</ins><span class="cx"> push(@implContent, " throwVMTypeError(state);\n");
</span><span class="cx"> push(@implContent, " return false;\n");
</span><span class="cx"> push(@implContent, " }\n");
</span><span class="lines">@@ -2810,10 +2810,10 @@
</span><span class="cx"> }
</span><span class="cx"> if ($svgPropertyType) {
</span><span class="cx"> if ($setterRaisesExceptionWithMessage) {
</span><del>- push(@implContent, " if (!ec.code)\n");
</del><ins>+ push(@implContent, " if (LIKELY(!ec.code))\n");
</ins><span class="cx"> push(@implContent, " impl.commitChange();\n");
</span><span class="cx"> } elsif ($setterRaisesException) {
</span><del>- push(@implContent, " if (!ec)\n");
</del><ins>+ push(@implContent, " if (LIKELY(!ec))\n");
</ins><span class="cx"> push(@implContent, " impl.commitChange();\n");
</span><span class="cx"> } else {
</span><span class="cx"> push(@implContent, " impl.commitChange();\n");
</span><span class="lines">@@ -3536,9 +3536,9 @@
</span><span class="cx"> push(@$outputArray, " }\n");
</span><span class="cx"> } else {
</span><span class="cx"> if ($codeGenerator->IsFunctionOnlyCallbackInterface($argType)) {
</span><del>- push(@$outputArray, " if (!state->argument($argsIndex).isFunction())\n");
</del><ins>+ push(@$outputArray, " if (UNLIKELY(!state->argument($argsIndex).isFunction()))\n");
</ins><span class="cx"> } else {
</span><del>- push(@$outputArray, " if (!state->argument($argsIndex).isObject())\n");
</del><ins>+ push(@$outputArray, " if (UNLIKELY(!state->argument($argsIndex).isObject()))\n");
</ins><span class="cx"> }
</span><span class="cx"> push(@$outputArray, " return throwArgumentMustBeFunctionError(*state, $argsIndex, \"$name\", \"$interfaceName\", $quotedFunctionName);\n");
</span><span class="cx"> if ($function->isStatic) {
</span><span class="lines">@@ -3624,7 +3624,7 @@
</span><span class="cx">
</span><span class="cx"> my $argValue = "state->argument($argsIndex)";
</span><span class="cx"> if ($codeGenerator->IsWrapperType($argType)) {
</span><del>- push(@$outputArray, " if (!${argValue}.isUndefinedOrNull() && !${argValue}.inherits(JS${argType}::info()))\n");
</del><ins>+ push(@$outputArray, " if (UNLIKELY(!${argValue}.isUndefinedOrNull() && !${argValue}.inherits(JS${argType}::info())))\n");
</ins><span class="cx"> push(@$outputArray, " return throwArgumentTypeError(*state, $argsIndex, \"$name\", \"$interfaceName\", $quotedFunctionName, \"$argType\");\n");
</span><span class="cx"> }
</span><span class="cx"> }
</span><span class="lines">@@ -3681,13 +3681,13 @@
</span><span class="cx">
</span><span class="cx"> my $isTearOff = $codeGenerator->IsSVGTypeNeedingTearOff($argType) && $interfaceName !~ /List$/;
</span><span class="cx"> if ($isTearOff or ShouldPassWrapperByReference($parameter, $interface)) {
</span><del>- push(@$outputArray, " if (!$name)\n");
</del><ins>+ push(@$outputArray, " if (UNLIKELY(!$name))\n");
</ins><span class="cx"> push(@$outputArray, " return throwVMTypeError(state);\n");
</span><span class="cx"> $value = $isTearOff ? "$name->propertyReference()" : "*$name";
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> if ($argType eq "double" or $argType eq "float") {
</span><del>- push(@$outputArray, " if (!std::isfinite($name))\n");
</del><ins>+ push(@$outputArray, " if (UNLIKELY(!std::isfinite($name)))\n");
</ins><span class="cx"> push(@$outputArray, " return throwVMTypeError(state);\n");
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -4020,10 +4020,10 @@
</span><span class="cx">
</span><span class="cx"> if ($svgPropertyType and !$function->isStatic) {
</span><span class="cx"> if ($raisesExceptionWithMessage) {
</span><del>- push(@implContent, $indent . "if (!ec.code)\n");
</del><ins>+ push(@implContent, $indent . "if (LIKELY(!ec.code))\n");
</ins><span class="cx"> push(@implContent, $indent . " impl.commitChange();\n");
</span><span class="cx"> } elsif ($raisesException) {
</span><del>- push(@implContent, $indent . "if (!ec)\n");
</del><ins>+ push(@implContent, $indent . "if (LIKELY(!ec))\n");
</ins><span class="cx"> push(@implContent, $indent . " impl.commitChange();\n");
</span><span class="cx"> } else {
</span><span class="cx"> push(@implContent, $indent . "impl.commitChange();\n");
</span><span class="lines">@@ -4908,7 +4908,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> if ($leastNumMandatoryParams >= 1) {
</span><del>- push(@$outputArray, " if (argsCount < $leastNumMandatoryParams)\n");
</del><ins>+ push(@$outputArray, " if (UNLIKELY(argsCount < $leastNumMandatoryParams))\n");
</ins><span class="cx"> push(@$outputArray, " return throwVMError(state, createNotEnoughArgumentsError(state));\n");
</span><span class="cx"> }
</span><span class="cx"> push(@$outputArray, <<END);
</span><span class="lines">@@ -5053,14 +5053,14 @@
</span><span class="cx"> if ($codeGenerator->ExtendedAttributeContains($interface->extendedAttributes->{"ConstructorCallWith"}, "ScriptExecutionContext")) {
</span><span class="cx"> push(@constructorArgList, "*context");
</span><span class="cx"> push(@$outputArray, " ScriptExecutionContext* context = castedThis->scriptExecutionContext();\n");
</span><del>- push(@$outputArray, " if (!context)\n");
</del><ins>+ push(@$outputArray, " if (UNLIKELY(!context))\n");
</ins><span class="cx"> push(@$outputArray, " return throwConstructorDocumentUnavailableError(*state, \"${interfaceName}\");\n");
</span><span class="cx"> }
</span><span class="cx"> if ($codeGenerator->ExtendedAttributeContains($interface->extendedAttributes->{"ConstructorCallWith"}, "Document")) {
</span><span class="cx"> $implIncludes{"Document.h"} = 1;
</span><span class="cx"> push(@constructorArgList, "document");
</span><span class="cx"> push(@$outputArray, " ScriptExecutionContext* context = castedThis->scriptExecutionContext();\n");
</span><del>- push(@$outputArray, " if (!context)\n");
</del><ins>+ push(@$outputArray, " if (UNLIKELY(!context))\n");
</ins><span class="cx"> push(@$outputArray, " return throwConstructorDocumentUnavailableError(*state, \"${interfaceName}\");\n");
</span><span class="cx"> push(@$outputArray, " ASSERT(context->isDocument());\n");
</span><span class="cx"> push(@$outputArray, " auto& document = downcast<Document>(*context);\n");
</span><span class="lines">@@ -5091,7 +5091,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> if ($interface->extendedAttributes->{"ConstructorRaisesException"}) {
</span><del>- push(@$outputArray, " if (ec) {\n");
</del><ins>+ push(@$outputArray, " if (UNLIKELY(ec)) {\n");
</ins><span class="cx"> push(@$outputArray, " setDOMException(state, ec);\n");
</span><span class="cx"> push(@$outputArray, " return JSValue::encode(JSValue());\n");
</span><span class="cx"> push(@$outputArray, " }\n");
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestActiveDOMObjectcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestActiveDOMObject.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestActiveDOMObject.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestActiveDOMObject.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -169,7 +169,7 @@
</span><span class="cx"> EncodedJSValue jsTestActiveDOMObjectConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestActiveDOMObjectPrototype* domObject = jsDynamicCast<JSTestActiveDOMObjectPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestActiveDOMObject::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span><span class="lines">@@ -206,7 +206,7 @@
</span><span class="cx"> Node* nextChild = JSNode::toWrapped(state->argument(0));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!nextChild)
</del><ins>+ if (UNLIKELY(!nextChild))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> impl.excitingFunction(*nextChild);
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestClassWithJSBuiltinConstructorcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestClassWithJSBuiltinConstructor.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestClassWithJSBuiltinConstructor.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestClassWithJSBuiltinConstructor.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -124,7 +124,7 @@
</span><span class="cx"> EncodedJSValue jsTestClassWithJSBuiltinConstructorConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestClassWithJSBuiltinConstructorPrototype* domObject = jsDynamicCast<JSTestClassWithJSBuiltinConstructorPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestClassWithJSBuiltinConstructor::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestCustomConstructorWithNoInterfaceObjectcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestCustomConstructorWithNoInterfaceObject.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestCustomConstructorWithNoInterfaceObject.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestCustomConstructorWithNoInterfaceObject.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -123,7 +123,7 @@
</span><span class="cx"> EncodedJSValue jsTestCustomConstructorWithNoInterfaceObjectConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestCustomConstructorWithNoInterfaceObjectPrototype* domObject = jsDynamicCast<JSTestCustomConstructorWithNoInterfaceObjectPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> JSValue constructor = JSTestCustomConstructorWithNoInterfaceObjectConstructor::create(state->vm(), JSTestCustomConstructorWithNoInterfaceObjectConstructor::createStructure(state->vm(), *domObject->globalObject(), domObject->globalObject()->objectPrototype()), *jsCast<JSDOMGlobalObject*>(domObject->globalObject()));
</span><span class="cx"> // Shadowing constructor property to ensure reusing the same constructor object
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestCustomNamedGettercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestCustomNamedGetter.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestCustomNamedGetter.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestCustomNamedGetter.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -161,7 +161,7 @@
</span><span class="cx"> EncodedJSValue jsTestCustomNamedGetterConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestCustomNamedGetterPrototype* domObject = jsDynamicCast<JSTestCustomNamedGetterPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestCustomNamedGetter::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestEventConstructorcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestEventConstructor.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestEventConstructor.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestEventConstructor.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -226,7 +226,7 @@
</span><span class="cx"> EncodedJSValue jsTestEventConstructorConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestEventConstructorPrototype* domObject = jsDynamicCast<JSTestEventConstructorPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestEventConstructor::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestEventTargetcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestEventTarget.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestEventTarget.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestEventTarget.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -149,7 +149,7 @@
</span><span class="cx"> {
</span><span class="cx"> auto* thisObject = jsCast<JSTestEventTarget*>(object);
</span><span class="cx"> ASSERT_GC_OBJECT_INHERITS(thisObject, info());
</span><del>- if (index < thisObject->wrapped().length()) {
</del><ins>+ if (LIKELY(index < thisObject->wrapped().length())) {
</ins><span class="cx"> unsigned attributes = DontDelete | ReadOnly;
</span><span class="cx"> slot.setValue(thisObject, attributes, toJS(state, thisObject->globalObject(), thisObject->wrapped().item(index)));
</span><span class="cx"> return true;
</span><span class="lines">@@ -160,7 +160,7 @@
</span><span class="cx"> EncodedJSValue jsTestEventTargetConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestEventTargetPrototype* domObject = jsDynamicCast<JSTestEventTargetPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestEventTarget::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestExceptioncpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestException.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestException.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestException.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -159,7 +159,7 @@
</span><span class="cx"> EncodedJSValue jsTestExceptionConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestExceptionPrototype* domObject = jsDynamicCast<JSTestExceptionPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestException::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestGenerateIsReachablecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestGenerateIsReachable.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestGenerateIsReachable.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestGenerateIsReachable.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -118,7 +118,7 @@
</span><span class="cx"> EncodedJSValue jsTestGenerateIsReachableConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestGenerateIsReachablePrototype* domObject = jsDynamicCast<JSTestGenerateIsReachablePrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestGenerateIsReachable::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestGlobalObjectcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestGlobalObject.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestGlobalObject.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestGlobalObject.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -188,7 +188,7 @@
</span><span class="cx"> EncodedJSValue jsTestGlobalObjectConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestGlobalObjectPrototype* domObject = jsDynamicCast<JSTestGlobalObjectPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestGlobalObject::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span><span class="lines">@@ -316,7 +316,7 @@
</span><span class="cx"> return jsTestGlobalObjectInstanceFunctionEnabledAtRuntimeOperation2(state);
</span><span class="cx"> #endif
</span><span class="cx">
</span><del>- if (argsCount < 1)
</del><ins>+ if (UNLIKELY(argsCount < 1))
</ins><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestInterfacecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestInterface.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestInterface.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestInterface.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -235,10 +235,10 @@
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><span class="cx"> ScriptExecutionContext* context = castedThis->scriptExecutionContext();
</span><del>- if (!context)
</del><ins>+ if (UNLIKELY(!context))
</ins><span class="cx"> return throwConstructorDocumentUnavailableError(*state, "TestInterface");
</span><span class="cx"> RefPtr<TestInterface> object = TestInterface::create(*context, str1, str2, ec);
</span><del>- if (ec) {
</del><ins>+ if (UNLIKELY(ec)) {
</ins><span class="cx"> setDOMException(state, ec);
</span><span class="cx"> return JSValue::encode(JSValue());
</span><span class="cx"> }
</span><span class="lines">@@ -589,7 +589,7 @@
</span><span class="cx"> EncodedJSValue jsTestInterfaceConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestInterfacePrototype* domObject = jsDynamicCast<JSTestInterfacePrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestInterface::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span><span class="lines">@@ -801,7 +801,7 @@
</span><span class="cx"> TestObj* objArg = JSTestObj::toWrapped(state->argument(1));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!objArg)
</del><ins>+ if (UNLIKELY(!objArg))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> JSValue result = toJS(state, castedThis->globalObject(), WTF::getPtr(impl.implementsMethod2(*context, strArg, *objArg, ec)));
</span><span class="cx">
</span><span class="lines">@@ -869,7 +869,7 @@
</span><span class="cx"> TestObj* objArg = JSTestObj::toWrapped(state->argument(1));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!objArg)
</del><ins>+ if (UNLIKELY(!objArg))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> JSValue result = toJS(state, castedThis->globalObject(), WTF::getPtr(WebCore::TestSupplemental::supplementalMethod2(impl, *context, strArg, *objArg, ec)));
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestJSBuiltinConstructorcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestJSBuiltinConstructor.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestJSBuiltinConstructor.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestJSBuiltinConstructor.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -161,7 +161,7 @@
</span><span class="cx"> EncodedJSValue jsTestJSBuiltinConstructorConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestJSBuiltinConstructorPrototype* domObject = jsDynamicCast<JSTestJSBuiltinConstructorPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestJSBuiltinConstructor::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestMediaQueryListListenercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestMediaQueryListListener.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestMediaQueryListListener.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestMediaQueryListListener.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -126,7 +126,7 @@
</span><span class="cx"> EncodedJSValue jsTestMediaQueryListListenerConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestMediaQueryListListenerPrototype* domObject = jsDynamicCast<JSTestMediaQueryListListenerPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestMediaQueryListListener::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span><span class="lines">@@ -158,7 +158,7 @@
</span><span class="cx"> auto& impl = castedThis->wrapped();
</span><span class="cx"> if (UNLIKELY(state->argumentCount() < 1))
</span><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><del>- if (!state->argument(0).isFunction())
</del><ins>+ if (UNLIKELY(!state->argument(0).isFunction()))
</ins><span class="cx"> return throwArgumentMustBeFunctionError(*state, 0, "listener", "TestMediaQueryListListener", "method");
</span><span class="cx"> RefPtr<MediaQueryListListener> listener = JSMediaQueryListListener::create(asObject(state->uncheckedArgument(0)), castedThis->globalObject());
</span><span class="cx"> impl.method(WTFMove(listener));
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestNamedConstructorcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestNamedConstructor.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestNamedConstructor.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestNamedConstructor.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -96,7 +96,7 @@
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><span class="cx"> RefPtr<TestNamedConstructor> object = TestNamedConstructor::createForJSConstructor(*castedThis->document(), str1, str2, str3, ec);
</span><del>- if (ec) {
</del><ins>+ if (UNLIKELY(ec)) {
</ins><span class="cx"> setDOMException(state, ec);
</span><span class="cx"> return JSValue::encode(JSValue());
</span><span class="cx"> }
</span><span class="lines">@@ -159,7 +159,7 @@
</span><span class="cx"> EncodedJSValue jsTestNamedConstructorConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestNamedConstructorPrototype* domObject = jsDynamicCast<JSTestNamedConstructorPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestNamedConstructor::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestNodecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestNode.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestNode.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestNode.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -139,7 +139,7 @@
</span><span class="cx"> EncodedJSValue jsTestNodeConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestNodePrototype* domObject = jsDynamicCast<JSTestNodePrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestNode::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestNondeterministiccpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestNondeterministic.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestNondeterministic.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestNondeterministic.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -326,7 +326,7 @@
</span><span class="cx"> EncodedJSValue jsTestNondeterministicConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestNondeterministicPrototype* domObject = jsDynamicCast<JSTestNondeterministicPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestNondeterministic::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestObjcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestObj.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestObj.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestObj.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -527,10 +527,10 @@
</span><span class="cx"> auto* castedThis = jsCast<JSTestObjConstructor*>(state->callee());
</span><span class="cx"> if (UNLIKELY(state->argumentCount() < 2))
</span><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><del>- if (!state->argument(0).isObject())
</del><ins>+ if (UNLIKELY(!state->argument(0).isObject()))
</ins><span class="cx"> return throwArgumentMustBeFunctionError(*state, 0, "testCallback", "TestObj", nullptr);
</span><span class="cx"> RefPtr<TestCallback> testCallback = JSTestCallback::create(asObject(state->uncheckedArgument(0)), castedThis->globalObject());
</span><del>- if (!state->argument(1).isFunction())
</del><ins>+ if (UNLIKELY(!state->argument(1).isFunction()))
</ins><span class="cx"> return throwArgumentMustBeFunctionError(*state, 1, "testCallbackFunction", "TestObj", nullptr);
</span><span class="cx"> RefPtr<TestCallbackFunction> testCallbackFunction = JSTestCallbackFunction::create(asObject(state->uncheckedArgument(1)), castedThis->globalObject());
</span><span class="cx"> RefPtr<TestObj> object = TestObj::create(*testCallback, *testCallbackFunction);
</span><span class="lines">@@ -878,7 +878,7 @@
</span><span class="cx"> {
</span><span class="cx"> auto* thisObject = jsCast<JSTestObj*>(object);
</span><span class="cx"> ASSERT_GC_OBJECT_INHERITS(thisObject, info());
</span><del>- if (index <= MAX_ARRAY_INDEX) {
</del><ins>+ if (LIKELY(index <= MAX_ARRAY_INDEX)) {
</ins><span class="cx"> unsigned attributes = DontDelete | ReadOnly;
</span><span class="cx"> slot.setValue(thisObject, attributes, jsStringOrUndefined(state, thisObject->wrapped().item(index)));
</span><span class="cx"> return true;
</span><span class="lines">@@ -971,7 +971,7 @@
</span><span class="cx"> if (UNLIKELY(!castedThis)) {
</span><span class="cx"> return throwGetterTypeError(*state, "TestObj", "TestSubObjEnabledBySetting");
</span><span class="cx"> }
</span><del>- if (!castedThis->wrapped().frame())
</del><ins>+ if (UNLIKELY(!castedThis->wrapped().frame()))
</ins><span class="cx"> return JSValue::encode(jsUndefined());
</span><span class="cx"> Settings& settings = castedThis->wrapped().frame()->settings();
</span><span class="cx"> if (!settings.testSettingEnabled())
</span><span class="lines">@@ -2185,7 +2185,7 @@
</span><span class="cx"> EncodedJSValue jsTestObjConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestObjPrototype* domObject = jsDynamicCast<JSTestObjPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestObj::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span><span class="lines">@@ -3417,7 +3417,7 @@
</span><span class="cx"> return jsTestObjPrototypeFunctionEnabledAtRuntimeOperation2(state);
</span><span class="cx"> #endif
</span><span class="cx">
</span><del>- if (argsCount < 1)
</del><ins>+ if (UNLIKELY(argsCount < 1))
</ins><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> }
</span><span class="lines">@@ -3453,7 +3453,7 @@
</span><span class="cx"> TestObj* objArg = JSTestObj::toWrapped(state->argument(2));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!objArg)
</del><ins>+ if (UNLIKELY(!objArg))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> impl.voidMethodWithArgs(longArg, strArg, *objArg);
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><span class="lines">@@ -3490,7 +3490,7 @@
</span><span class="cx"> TestObj* objArg = JSTestObj::toWrapped(state->argument(2));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!objArg)
</del><ins>+ if (UNLIKELY(!objArg))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> JSValue result = jsNumber(impl.byteMethodWithArgs(byteArg, strArg, *objArg));
</span><span class="cx"> return JSValue::encode(result);
</span><span class="lines">@@ -3527,7 +3527,7 @@
</span><span class="cx"> TestObj* objArg = JSTestObj::toWrapped(state->argument(2));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!objArg)
</del><ins>+ if (UNLIKELY(!objArg))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> JSValue result = jsNumber(impl.octetMethodWithArgs(octetArg, strArg, *objArg));
</span><span class="cx"> return JSValue::encode(result);
</span><span class="lines">@@ -3564,7 +3564,7 @@
</span><span class="cx"> TestObj* objArg = JSTestObj::toWrapped(state->argument(2));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!objArg)
</del><ins>+ if (UNLIKELY(!objArg))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> JSValue result = jsNumber(impl.longMethodWithArgs(longArg, strArg, *objArg));
</span><span class="cx"> return JSValue::encode(result);
</span><span class="lines">@@ -3601,7 +3601,7 @@
</span><span class="cx"> TestObj* objArg = JSTestObj::toWrapped(state->argument(2));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!objArg)
</del><ins>+ if (UNLIKELY(!objArg))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> JSValue result = toJS(state, castedThis->globalObject(), WTF::getPtr(impl.objMethodWithArgs(longArg, strArg, *objArg)));
</span><span class="cx"> return JSValue::encode(result);
</span><span class="lines">@@ -3765,7 +3765,7 @@
</span><span class="cx"> TestObj* objArg = JSTestObj::toWrapped(state->argument(1));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!objArg)
</del><ins>+ if (UNLIKELY(!objArg))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> JSValue result = toJS(state, castedThis->globalObject(), WTF::getPtr(impl.methodThatRequiresAllArgsAndThrows(strArg, *objArg, ec)));
</span><span class="cx">
</span><span class="lines">@@ -4487,7 +4487,7 @@
</span><span class="cx"> auto& impl = castedThis->wrapped();
</span><span class="cx"> if (UNLIKELY(state->argumentCount() < 1))
</span><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><del>- if (!state->argument(0).isObject())
</del><ins>+ if (UNLIKELY(!state->argument(0).isObject()))
</ins><span class="cx"> return throwArgumentMustBeFunctionError(*state, 0, "callback", "TestObj", "methodWithCallbackArg");
</span><span class="cx"> RefPtr<TestCallback> callback = JSTestCallback::create(asObject(state->uncheckedArgument(0)), castedThis->globalObject());
</span><span class="cx"> impl.methodWithCallbackArg(WTFMove(callback));
</span><span class="lines">@@ -4507,7 +4507,7 @@
</span><span class="cx"> int nonCallback = toInt32(state, state->argument(0), NormalConversion);
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!state->argument(1).isObject())
</del><ins>+ if (UNLIKELY(!state->argument(1).isObject()))
</ins><span class="cx"> return throwArgumentMustBeFunctionError(*state, 1, "callback", "TestObj", "methodWithNonCallbackArgAndCallbackArg");
</span><span class="cx"> RefPtr<TestCallback> callback = JSTestCallback::create(asObject(state->uncheckedArgument(1)), castedThis->globalObject());
</span><span class="cx"> impl.methodWithNonCallbackArgAndCallbackArg(nonCallback, WTFMove(callback));
</span><span class="lines">@@ -4542,7 +4542,7 @@
</span><span class="cx"> auto& impl = castedThis->wrapped();
</span><span class="cx"> if (UNLIKELY(state->argumentCount() < 1))
</span><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><del>- if (!state->argument(0).isFunction())
</del><ins>+ if (UNLIKELY(!state->argument(0).isFunction()))
</ins><span class="cx"> return throwArgumentMustBeFunctionError(*state, 0, "callback", "TestObj", "methodWithCallbackFunctionArg");
</span><span class="cx"> RefPtr<TestCallbackFunction> callback = JSTestCallbackFunction::create(asObject(state->uncheckedArgument(0)), castedThis->globalObject());
</span><span class="cx"> impl.methodWithCallbackFunctionArg(WTFMove(callback));
</span><span class="lines">@@ -4562,7 +4562,7 @@
</span><span class="cx"> int nonCallback = toInt32(state, state->argument(0), NormalConversion);
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!state->argument(1).isFunction())
</del><ins>+ if (UNLIKELY(!state->argument(1).isFunction()))
</ins><span class="cx"> return throwArgumentMustBeFunctionError(*state, 1, "callback", "TestObj", "methodWithNonCallbackArgAndCallbackFunctionArg");
</span><span class="cx"> RefPtr<TestCallbackFunction> callback = JSTestCallbackFunction::create(asObject(state->uncheckedArgument(1)), castedThis->globalObject());
</span><span class="cx"> impl.methodWithNonCallbackArgAndCallbackFunctionArg(nonCallback, WTFMove(callback));
</span><span class="lines">@@ -4603,7 +4603,7 @@
</span><span class="cx"> {
</span><span class="cx"> if (UNLIKELY(state->argumentCount() < 1))
</span><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><del>- if (!state->argument(0).isObject())
</del><ins>+ if (UNLIKELY(!state->argument(0).isObject()))
</ins><span class="cx"> return throwArgumentMustBeFunctionError(*state, 0, "callback", "TestObj", "staticMethodWithCallbackArg");
</span><span class="cx"> RefPtr<TestCallback> callback = createFunctionOnlyCallback<JSTestCallback>(state, jsCast<JSDOMGlobalObject*>(state->lexicalGlobalObject()), state->uncheckedArgument(0));
</span><span class="cx"> TestObj::staticMethodWithCallbackArg(WTFMove(callback));
</span><span class="lines">@@ -4739,7 +4739,7 @@
</span><span class="cx"> auto& impl = castedThis->wrapped();
</span><span class="cx"> if (UNLIKELY(state->argumentCount() < 1))
</span><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><del>- if (!state->argument(0).isObject())
</del><ins>+ if (UNLIKELY(!state->argument(0).isObject()))
</ins><span class="cx"> return throwArgumentMustBeFunctionError(*state, 0, "callback", "TestObj", "overloadedMethod");
</span><span class="cx"> RefPtr<TestCallback> callback = JSTestCallback::create(asObject(state->uncheckedArgument(0)), castedThis->globalObject());
</span><span class="cx"> impl.overloadedMethod(WTFMove(callback));
</span><span class="lines">@@ -4793,7 +4793,7 @@
</span><span class="cx"> TestObj* objArg = JSTestObj::toWrapped(state->argument(0));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!objArg)
</del><ins>+ if (UNLIKELY(!objArg))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> impl.overloadedMethod(*objArg);
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><span class="lines">@@ -4897,7 +4897,7 @@
</span><span class="cx"> return jsTestObjPrototypeFunctionOverloadedMethod11(state);
</span><span class="cx"> if ()
</span><span class="cx"> return jsTestObjPrototypeFunctionOverloadedMethod12(state);
</span><del>- if (argsCount < 1)
</del><ins>+ if (UNLIKELY(argsCount < 1))
</ins><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> }
</span><span class="lines">@@ -4951,7 +4951,7 @@
</span><span class="cx"> return jsTestObjPrototypeFunctionOverloadedMethodWithOptionalParameter1(state);
</span><span class="cx"> if ((argsCount == 1 && (arg0.isNull() || (arg0.isObject() && asObject(arg0)->inherits(JSTestObj::info())))) || (argsCount == 2 && (arg0.isNull() || (arg0.isObject() && asObject(arg0)->inherits(JSTestObj::info())))))
</span><span class="cx"> return jsTestObjPrototypeFunctionOverloadedMethodWithOptionalParameter2(state);
</span><del>- if (argsCount < 1)
</del><ins>+ if (UNLIKELY(argsCount < 1))
</ins><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> }
</span><span class="lines">@@ -5020,7 +5020,7 @@
</span><span class="cx"> return jsTestObjConstructorFunctionOverloadedMethod12(state);
</span><span class="cx"> #endif
</span><span class="cx">
</span><del>- if (argsCount < 1)
</del><ins>+ if (UNLIKELY(argsCount < 1))
</ins><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> }
</span><span class="lines">@@ -5192,7 +5192,7 @@
</span><span class="cx"> TestNode* value = JSTestNode::toWrapped(state->argument(0));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!value)
</del><ins>+ if (UNLIKELY(!value))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> impl.convert1(*value);
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><span class="lines">@@ -5322,12 +5322,12 @@
</span><span class="cx"> if (UNLIKELY(state->argumentCount() < 2))
</span><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><span class="cx"> ExceptionCode ec = 0;
</span><del>- if (!state->argument(0).isUndefinedOrNull() && !state->argument(0).inherits(JSTestObj::info()))
</del><ins>+ if (UNLIKELY(!state->argument(0).isUndefinedOrNull() && !state->argument(0).inherits(JSTestObj::info())))
</ins><span class="cx"> return throwArgumentTypeError(*state, 0, "objArg", "TestObj", "strictFunctionWithSequence", "TestObj");
</span><span class="cx"> TestObj* objArg = JSTestObj::toWrapped(state->argument(0));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!objArg)
</del><ins>+ if (UNLIKELY(!objArg))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> Vector<unsigned> a = toNativeArray<unsigned>(state, state->argument(1));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="lines">@@ -5349,12 +5349,12 @@
</span><span class="cx"> if (UNLIKELY(state->argumentCount() < 2))
</span><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><span class="cx"> ExceptionCode ec = 0;
</span><del>- if (!state->argument(0).isUndefinedOrNull() && !state->argument(0).inherits(JSTestObj::info()))
</del><ins>+ if (UNLIKELY(!state->argument(0).isUndefinedOrNull() && !state->argument(0).inherits(JSTestObj::info())))
</ins><span class="cx"> return throwArgumentTypeError(*state, 0, "objArg", "TestObj", "strictFunctionWithArray", "TestObj");
</span><span class="cx"> TestObj* objArg = JSTestObj::toWrapped(state->argument(0));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!objArg)
</del><ins>+ if (UNLIKELY(!objArg))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> Vector<int> array = toNativeArray<int>(state, state->argument(1));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="lines">@@ -5418,7 +5418,7 @@
</span><span class="cx"> Node* head = JSNode::toWrapped(state->argument(0));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!head)
</del><ins>+ if (UNLIKELY(!head))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> Vector<Node*> tail;
</span><span class="cx"> for (unsigned i = 1, count = state->argumentCount(); i < count; ++i) {
</span><span class="lines">@@ -5487,7 +5487,7 @@
</span><span class="cx"> float a = state->argument(0).toFloat(state);
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!std::isfinite(a))
</del><ins>+ if (UNLIKELY(!std::isfinite(a)))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> impl.testPromiseFunctionWithFloatArgument(a, DeferredWrapper(state, castedThis->globalObject(), promiseDeferred));
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><span class="lines">@@ -5553,7 +5553,7 @@
</span><span class="cx"> float a = state->argument(0).toFloat(state);
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!std::isfinite(a))
</del><ins>+ if (UNLIKELY(!std::isfinite(a)))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> impl.testPromiseOverloadedFunction(a, DeferredWrapper(state, castedThis->globalObject(), promiseDeferred));
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><span class="lines">@@ -5578,7 +5578,7 @@
</span><span class="cx"> FetchRequest* request = JSFetchRequest::toWrapped(state->argument(0));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!request)
</del><ins>+ if (UNLIKELY(!request))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> impl.testPromiseOverloadedFunction(*request, DeferredWrapper(state, castedThis->globalObject(), promiseDeferred));
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><span class="lines">@@ -5592,7 +5592,7 @@
</span><span class="cx"> JSValue arg0(state->argument(0));
</span><span class="cx"> if ((argsCount == 1 && ((arg0.isObject() && asObject(arg0)->inherits(JSFetchRequest::info())))))
</span><span class="cx"> return jsTestObjPrototypeFunctionTestPromiseOverloadedFunction2(state);
</span><del>- if (argsCount < 1)
</del><ins>+ if (UNLIKELY(argsCount < 1))
</ins><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestOverloadedConstructorscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestOverloadedConstructors.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestOverloadedConstructors.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestOverloadedConstructors.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -73,7 +73,7 @@
</span><span class="cx"> ArrayBuffer* arrayBuffer = toArrayBuffer(state->argument(0));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!arrayBuffer)
</del><ins>+ if (UNLIKELY(!arrayBuffer))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> RefPtr<TestOverloadedConstructors> object = TestOverloadedConstructors::create(*arrayBuffer);
</span><span class="cx"> return JSValue::encode(asObject(toJS(state, castedThis->globalObject(), object.get())));
</span><span class="lines">@@ -99,7 +99,7 @@
</span><span class="cx"> Blob* blob = JSBlob::toWrapped(state->argument(0));
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!blob)
</del><ins>+ if (UNLIKELY(!blob))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> RefPtr<TestOverloadedConstructors> object = TestOverloadedConstructors::create(*blob);
</span><span class="cx"> return JSValue::encode(asObject(toJS(state, castedThis->globalObject(), object.get())));
</span><span class="lines">@@ -141,7 +141,7 @@
</span><span class="cx"> return constructJSTestOverloadedConstructors4(state);
</span><span class="cx"> if ()
</span><span class="cx"> return constructJSTestOverloadedConstructors5(state);
</span><del>- if (argsCount < 1)
</del><ins>+ if (UNLIKELY(argsCount < 1))
</ins><span class="cx"> return throwVMError(state, createNotEnoughArgumentsError(state));
</span><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> }
</span><span class="lines">@@ -202,7 +202,7 @@
</span><span class="cx"> EncodedJSValue jsTestOverloadedConstructorsConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestOverloadedConstructorsPrototype* domObject = jsDynamicCast<JSTestOverloadedConstructorsPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestOverloadedConstructors::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestOverrideBuiltinscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestOverrideBuiltins.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestOverrideBuiltins.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestOverrideBuiltins.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -159,7 +159,7 @@
</span><span class="cx"> EncodedJSValue jsTestOverrideBuiltinsConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestOverrideBuiltinsPrototype* domObject = jsDynamicCast<JSTestOverrideBuiltinsPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestOverrideBuiltins::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestSerializedScriptValueInterfacecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -218,7 +218,7 @@
</span><span class="cx"> EncodedJSValue jsTestSerializedScriptValueInterfaceConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestSerializedScriptValueInterfacePrototype* domObject = jsDynamicCast<JSTestSerializedScriptValueInterfacePrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestSerializedScriptValueInterface::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSTestTypedefscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSTestTypedefs.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSTestTypedefs.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSTestTypedefs.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -131,7 +131,7 @@
</span><span class="cx"> String hello = state->argument(0).toString(state)->value(state);
</span><span class="cx"> if (UNLIKELY(state->hadException()))
</span><span class="cx"> return JSValue::encode(jsUndefined());
</span><del>- if (!state->argument(1).isObject())
</del><ins>+ if (UNLIKELY(!state->argument(1).isObject()))
</ins><span class="cx"> return throwArgumentMustBeFunctionError(*state, 1, "testCallback", "TestTypedefs", nullptr);
</span><span class="cx"> RefPtr<TestCallback> testCallback = JSTestCallback::create(asObject(state->uncheckedArgument(1)), castedThis->globalObject());
</span><span class="cx"> RefPtr<TestTypedefs> object = TestTypedefs::create(hello, *testCallback);
</span><span class="lines">@@ -327,7 +327,7 @@
</span><span class="cx"> EncodedJSValue jsTestTypedefsConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSTestTypedefsPrototype* domObject = jsDynamicCast<JSTestTypedefsPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSTestTypedefs::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSattributecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSattribute.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSattribute.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSattribute.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -159,7 +159,7 @@
</span><span class="cx"> EncodedJSValue jsattributeConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSattributePrototype* domObject = jsDynamicCast<JSattributePrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSattribute::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsscriptstestJSJSreadonlycpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/scripts/test/JS/JSreadonly.cpp (200285 => 200286)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/scripts/test/JS/JSreadonly.cpp 2016-04-30 04:19:00 UTC (rev 200285)
+++ trunk/Source/WebCore/bindings/scripts/test/JS/JSreadonly.cpp 2016-04-30 04:48:34 UTC (rev 200286)
</span><span class="lines">@@ -118,7 +118,7 @@
</span><span class="cx"> EncodedJSValue jsreadonlyConstructor(ExecState* state, EncodedJSValue thisValue, PropertyName)
</span><span class="cx"> {
</span><span class="cx"> JSreadonlyPrototype* domObject = jsDynamicCast<JSreadonlyPrototype*>(JSValue::decode(thisValue));
</span><del>- if (!domObject)
</del><ins>+ if (UNLIKELY(!domObject))
</ins><span class="cx"> return throwVMTypeError(state);
</span><span class="cx"> return JSValue::encode(JSreadonly::getConstructor(state->vm(), domObject->globalObject()));
</span><span class="cx"> }
</span></span></pre>
</div>
</div>
</body>
</html>