Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (188541 => 188542)
--- trunk/Source/_javascript_Core/ChangeLog 2015-08-17 20:39:04 UTC (rev 188541)
+++ trunk/Source/_javascript_Core/ChangeLog 2015-08-17 21:40:30 UTC (rev 188542)
@@ -1,3 +1,20 @@
+2015-07-28 Sam Weinig <s...@webkit.org>
+
+ Cleanup the builtin _javascript_ files
+ https://bugs.webkit.org/show_bug.cgi?id=147382
+
+ Reviewed by Geoffrey Garen.
+
+ * builtins/Array.prototype.js:
+ * builtins/ArrayConstructor.js:
+ * builtins/ArrayIterator.prototype.js:
+ * builtins/Function.prototype.js:
+ * builtins/Iterator.prototype.js:
+ * builtins/ObjectConstructor.js:
+ * builtins/StringConstructor.js:
+ * builtins/StringIterator.prototype.js:
+ Unify the style of the built _javascript_ files.
+
2015-08-17 Alex Christensen <achristen...@webkit.org>
Move some commands from ./CMakeLists.txt to Source/cmake
Modified: trunk/Source/_javascript_Core/builtins/Array.prototype.js (188541 => 188542)
--- trunk/Source/_javascript_Core/builtins/Array.prototype.js 2015-08-17 20:39:04 UTC (rev 188541)
+++ trunk/Source/_javascript_Core/builtins/Array.prototype.js 2015-08-17 21:40:30 UTC (rev 188542)
@@ -27,6 +27,7 @@
function reduce(callback /*, initialValue */)
{
"use strict";
+
if (this === null)
throw new @TypeError("Array.prototype.reduce requires that |this| not be null");
@@ -64,6 +65,7 @@
function reduceRight(callback /*, initialValue */)
{
"use strict";
+
if (this === null)
throw new @TypeError("Array.prototype.reduceRight requires that |this| not be null");
@@ -98,8 +100,10 @@
return accumulator;
}
-function every(callback /*, thisArg */) {
+function every(callback /*, thisArg */)
+{
"use strict";
+
if (this === null)
throw new @TypeError("Array.prototype.every requires that |this| not be null");
@@ -124,8 +128,10 @@
return true;
}
-function forEach(callback /*, thisArg */) {
+function forEach(callback /*, thisArg */)
+{
"use strict";
+
if (this === null)
throw new @TypeError("Array.prototype.forEach requires that |this| not be null");
@@ -146,8 +152,10 @@
}
}
-function filter(callback /*, thisArg */) {
+function filter(callback /*, thisArg */)
+{
"use strict";
+
if (this === null)
throw new @TypeError("Array.prototype.filter requires that |this| not be null");
@@ -175,8 +183,10 @@
return result;
}
-function map(callback /*, thisArg */) {
+function map(callback /*, thisArg */)
+{
"use strict";
+
if (this === null)
throw new @TypeError("Array.prototype.map requires that |this| not be null");
@@ -202,8 +212,10 @@
return result;
}
-function some(callback /*, thisArg */) {
+function some(callback /*, thisArg */)
+{
"use strict";
+
if (this === null)
throw new @TypeError("Array.prototype.some requires that |this| not be null");
@@ -229,6 +241,7 @@
function fill(value /* [, start [, end]] */)
{
"use strict";
+
if (this === null)
throw new @TypeError("Array.prototype.fill requires that |this| not be null");
@@ -267,8 +280,10 @@
return O;
}
-function find(callback /*, thisArg */) {
+function find(callback /*, thisArg */)
+{
"use strict";
+
if (this === null)
throw new @TypeError("Array.prototype.find requires that |this| not be null");
@@ -290,8 +305,10 @@
return undefined;
}
-function findIndex(callback /*, thisArg */) {
+function findIndex(callback /*, thisArg */)
+{
"use strict";
+
if (this === null)
throw new @TypeError("Array.prototype.findIndex requires that |this| not be null");
@@ -312,8 +329,10 @@
return -1;
}
-function includes(searchElement /*, fromIndex*/) {
+function includes(searchElement /*, fromIndex*/)
+{
"use strict";
+
if (this === null)
throw new @TypeError("Array.prototype.includes requires that |this| not be null");
Modified: trunk/Source/_javascript_Core/builtins/ArrayConstructor.js (188541 => 188542)
--- trunk/Source/_javascript_Core/builtins/ArrayConstructor.js 2015-08-17 20:39:04 UTC (rev 188541)
+++ trunk/Source/_javascript_Core/builtins/ArrayConstructor.js 2015-08-17 21:40:30 UTC (rev 188542)
@@ -36,7 +36,8 @@
return array;
}
-function from(items /*, mapFn, thisArg */) {
+function from(items /*, mapFn, thisArg */)
+{
"use strict";
var thisObj = this;
Modified: trunk/Source/_javascript_Core/builtins/ArrayIterator.prototype.js (188541 => 188542)
--- trunk/Source/_javascript_Core/builtins/ArrayIterator.prototype.js 2015-08-17 20:39:04 UTC (rev 188541)
+++ trunk/Source/_javascript_Core/builtins/ArrayIterator.prototype.js 2015-08-17 21:40:30 UTC (rev 188542)
@@ -23,7 +23,8 @@
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
-function next() {
+function next()
+{
"use strict";
if (this == null)
Modified: trunk/Source/_javascript_Core/builtins/Function.prototype.js (188541 => 188542)
--- trunk/Source/_javascript_Core/builtins/Function.prototype.js 2015-08-17 20:39:04 UTC (rev 188541)
+++ trunk/Source/_javascript_Core/builtins/Function.prototype.js 2015-08-17 21:40:30 UTC (rev 188542)
@@ -23,12 +23,16 @@
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
-function call(thisArgument) {
+function call(thisArgument)
+{
"use strict";
+
return this.@call(...arguments);
}
-function apply(thisValue, argumentValues) {
+function apply(thisValue, argumentValues)
+{
"use strict";
+
return this.@apply(thisValue, argumentValues);
}
Modified: trunk/Source/_javascript_Core/builtins/Iterator.prototype.js (188541 => 188542)
--- trunk/Source/_javascript_Core/builtins/Iterator.prototype.js 2015-08-17 20:39:04 UTC (rev 188541)
+++ trunk/Source/_javascript_Core/builtins/Iterator.prototype.js 2015-08-17 21:40:30 UTC (rev 188542)
@@ -25,6 +25,7 @@
function symbolIterator()
{
- 'use strict';
+ "use strict";
+
return this;
}
Modified: trunk/Source/_javascript_Core/builtins/ObjectConstructor.js (188541 => 188542)
--- trunk/Source/_javascript_Core/builtins/ObjectConstructor.js 2015-08-17 20:39:04 UTC (rev 188541)
+++ trunk/Source/_javascript_Core/builtins/ObjectConstructor.js 2015-08-17 21:40:30 UTC (rev 188542)
@@ -23,7 +23,8 @@
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
-function assign(target/*[*/, /*...*/sources/*] */) {
+function assign(target/*[*/, /*...*/sources/*] */)
+{
"use strict";
if (target == null)
Modified: trunk/Source/_javascript_Core/builtins/StringConstructor.js (188541 => 188542)
--- trunk/Source/_javascript_Core/builtins/StringConstructor.js 2015-08-17 20:39:04 UTC (rev 188541)
+++ trunk/Source/_javascript_Core/builtins/StringConstructor.js 2015-08-17 21:40:30 UTC (rev 188542)
@@ -23,7 +23,8 @@
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
-function raw(template) {
+function raw(template)
+{
"use strict";
if (template === null || template === undefined)
Modified: trunk/Source/_javascript_Core/builtins/StringIterator.prototype.js (188541 => 188542)
--- trunk/Source/_javascript_Core/builtins/StringIterator.prototype.js 2015-08-17 20:39:04 UTC (rev 188541)
+++ trunk/Source/_javascript_Core/builtins/StringIterator.prototype.js 2015-08-17 21:40:30 UTC (rev 188542)
@@ -23,7 +23,8 @@
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
-function next() {
+function next()
+{
"use strict";
if (this == null)