Title: [208643] trunk/Source/_javascript_Core
- Revision
- 208643
- Author
- [email protected]
- Date
- 2016-11-12 00:16:44 -0800 (Sat, 12 Nov 2016)
Log Message
Follow-up fix to r208639.
Unreviewed fix. This is a straightfoward change where I forgot to
switch from uncheckedArgument() to argument() in once case after
dropping an argumentCount check. All other cases do this properly.
This addresses an ASSERT seen on the bots running tests.
* runtime/JSDataViewPrototype.cpp:
(JSC::setData):
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (208642 => 208643)
--- trunk/Source/_javascript_Core/ChangeLog 2016-11-12 06:31:21 UTC (rev 208642)
+++ trunk/Source/_javascript_Core/ChangeLog 2016-11-12 08:16:44 UTC (rev 208643)
@@ -1,3 +1,15 @@
+2016-11-12 Joseph Pecoraro <[email protected]>
+
+ Follow-up fix to r208639.
+
+ Unreviewed fix. This is a straightfoward change where I forgot to
+ switch from uncheckedArgument() to argument() in once case after
+ dropping an argumentCount check. All other cases do this properly.
+ This addresses an ASSERT seen on the bots running tests.
+
+ * runtime/JSDataViewPrototype.cpp:
+ (JSC::setData):
+
2016-11-11 Joseph Pecoraro <[email protected]>
test262: DataView with explicit undefined byteLength should be the same as it not being present
Modified: trunk/Source/_javascript_Core/runtime/JSDataViewPrototype.cpp (208642 => 208643)
--- trunk/Source/_javascript_Core/runtime/JSDataViewPrototype.cpp 2016-11-12 06:31:21 UTC (rev 208642)
+++ trunk/Source/_javascript_Core/runtime/JSDataViewPrototype.cpp 2016-11-12 08:16:44 UTC (rev 208643)
@@ -172,7 +172,7 @@
if (!dataView)
return throwVMTypeError(exec, scope, ASCIILiteral("Receiver of DataView method must be a DataView"));
- unsigned byteOffset = exec->uncheckedArgument(0).toIndex(exec, "byteOffset");
+ unsigned byteOffset = exec->argument(0).toIndex(exec, "byteOffset");
RETURN_IF_EXCEPTION(scope, encodedJSValue());
const unsigned dataSize = sizeof(typename Adaptor::Type);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
