Title: [210570] trunk
Revision
210570
Author
[email protected]
Date
2017-01-10 17:35:32 -0800 (Tue, 10 Jan 2017)

Log Message

Unreviewed, rolling out r210052.
https://bugs.webkit.org/show_bug.cgi?id=166915

"breaks web compatability" (Requested by keith_miller on
#webkit).

Reverted changeset:

"Add support for global"
https://bugs.webkit.org/show_bug.cgi?id=165171
http://trac.webkit.org/changeset/210052

Modified Paths

Removed Paths

Diff

Modified: trunk/JSTests/ChangeLog (210569 => 210570)


--- trunk/JSTests/ChangeLog	2017-01-11 01:28:00 UTC (rev 210569)
+++ trunk/JSTests/ChangeLog	2017-01-11 01:35:32 UTC (rev 210570)
@@ -1,3 +1,17 @@
+2017-01-10  Commit Queue  <[email protected]>
+
+        Unreviewed, rolling out r210052.
+        https://bugs.webkit.org/show_bug.cgi?id=166915
+
+        "breaks web compatability" (Requested by keith_miller on
+        #webkit).
+
+        Reverted changeset:
+
+        "Add support for global"
+        https://bugs.webkit.org/show_bug.cgi?id=165171
+        http://trac.webkit.org/changeset/210052
+
 2017-01-10  Mark Lam  <[email protected]>
 
         Property setters should not be called for bound arguments list entries.

Deleted: trunk/JSTests/stress/global.js (210569 => 210570)


--- trunk/JSTests/stress/global.js	2017-01-11 01:28:00 UTC (rev 210569)
+++ trunk/JSTests/stress/global.js	2017-01-11 01:35:32 UTC (rev 210570)
@@ -1,20 +0,0 @@
-// This file tests the global property on the global object.
-
-let g = new Function("return this")();
-
-if (g !== global)
-    throw new Error("global returned the wrong value");
-
-for (name in g) {
-    if (name === "global")
-        throw new Error("global should not be enumerable")
-}
-
-
-global = 5;
-if (global !== 5)
-    throw new Error("global should be assignable");
-
-delete global;
-if ("global" in g)
-    throw new Error("global should be configurable");

Modified: trunk/LayoutTests/ChangeLog (210569 => 210570)


--- trunk/LayoutTests/ChangeLog	2017-01-11 01:28:00 UTC (rev 210569)
+++ trunk/LayoutTests/ChangeLog	2017-01-11 01:35:32 UTC (rev 210570)
@@ -1,3 +1,17 @@
+2017-01-10  Commit Queue  <[email protected]>
+
+        Unreviewed, rolling out r210052.
+        https://bugs.webkit.org/show_bug.cgi?id=166915
+
+        "breaks web compatability" (Requested by keith_miller on
+        #webkit).
+
+        Reverted changeset:
+
+        "Add support for global"
+        https://bugs.webkit.org/show_bug.cgi?id=165171
+        http://trac.webkit.org/changeset/210052
+
 2017-01-10  Sam Weinig  <[email protected]>
 
         [WebIDL] Remove most of the custom bindings for the WebGL code

Modified: trunk/LayoutTests/js/dom/getOwnPropertyDescriptor-expected.txt (210569 => 210570)


--- trunk/LayoutTests/js/dom/getOwnPropertyDescriptor-expected.txt	2017-01-11 01:28:00 UTC (rev 210569)
+++ trunk/LayoutTests/js/dom/getOwnPropertyDescriptor-expected.txt	2017-01-11 01:35:32 UTC (rev 210570)
@@ -94,42 +94,42 @@
 PASS Object.getOwnPropertyDescriptor(Math.sin, 'name').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(Math.sin, 'name').enumerable is false
 PASS Object.getOwnPropertyDescriptor(Math.sin, 'name').configurable is true
-PASS Object.getOwnPropertyDescriptor(globalObject, 'globalObject').value is globalObject
-PASS Object.getOwnPropertyDescriptor(globalObject, 'globalObject').hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'globalObject').hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'globalObject').enumerable is true
-PASS Object.getOwnPropertyDescriptor(globalObject, 'globalObject').configurable is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'undefined').value is undefined
-PASS Object.getOwnPropertyDescriptor(globalObject, 'undefined').hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'undefined').hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'undefined').enumerable is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'undefined').configurable is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'NaN').value is NaN
-PASS Object.getOwnPropertyDescriptor(globalObject, 'NaN').hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'NaN').hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'NaN').enumerable is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'NaN').configurable is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'Infinity').value is Infinity
-PASS Object.getOwnPropertyDescriptor(globalObject, 'Infinity').hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'Infinity').hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'Infinity').enumerable is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'Infinity').configurable is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'window').get is globalWindowGetter
-PASS Object.getOwnPropertyDescriptor(globalObject, 'window').set is undefined
-PASS Object.getOwnPropertyDescriptor(globalObject, 'window').hasOwnProperty('value') is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'window').hasOwnProperty('writable') is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'window').enumerable is true
-PASS Object.getOwnPropertyDescriptor(globalObject, 'window').configurable is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'XMLHttpRequest').value is XMLHttpRequest
-PASS Object.getOwnPropertyDescriptor(globalObject, 'XMLHttpRequest').hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'XMLHttpRequest').hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'XMLHttpRequest').enumerable is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 'XMLHttpRequest').configurable is true
-PASS Object.getOwnPropertyDescriptor(globalObject, 0).value is globalObject[0]
-PASS Object.getOwnPropertyDescriptor(globalObject, 0).hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 0).hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 0).enumerable is false
-PASS Object.getOwnPropertyDescriptor(globalObject, 0).configurable is false
+PASS Object.getOwnPropertyDescriptor(global, 'global').value is global
+PASS Object.getOwnPropertyDescriptor(global, 'global').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'global').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'global').enumerable is true
+PASS Object.getOwnPropertyDescriptor(global, 'global').configurable is false
+PASS Object.getOwnPropertyDescriptor(global, 'undefined').value is undefined
+PASS Object.getOwnPropertyDescriptor(global, 'undefined').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'undefined').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'undefined').enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 'undefined').configurable is false
+PASS Object.getOwnPropertyDescriptor(global, 'NaN').value is NaN
+PASS Object.getOwnPropertyDescriptor(global, 'NaN').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'NaN').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'NaN').enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 'NaN').configurable is false
+PASS Object.getOwnPropertyDescriptor(global, 'Infinity').value is Infinity
+PASS Object.getOwnPropertyDescriptor(global, 'Infinity').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'Infinity').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'Infinity').enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 'Infinity').configurable is false
+PASS Object.getOwnPropertyDescriptor(global, 'window').get is globalWindowGetter
+PASS Object.getOwnPropertyDescriptor(global, 'window').set is undefined
+PASS Object.getOwnPropertyDescriptor(global, 'window').hasOwnProperty('value') is false
+PASS Object.getOwnPropertyDescriptor(global, 'window').hasOwnProperty('writable') is false
+PASS Object.getOwnPropertyDescriptor(global, 'window').enumerable is true
+PASS Object.getOwnPropertyDescriptor(global, 'window').configurable is false
+PASS Object.getOwnPropertyDescriptor(global, 'XMLHttpRequest').value is XMLHttpRequest
+PASS Object.getOwnPropertyDescriptor(global, 'XMLHttpRequest').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'XMLHttpRequest').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'XMLHttpRequest').enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 'XMLHttpRequest').configurable is true
+PASS Object.getOwnPropertyDescriptor(global, 0).value is global[0]
+PASS Object.getOwnPropertyDescriptor(global, 0).hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 0).hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 0).enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 0).configurable is false
 PASS Object.getOwnPropertyDescriptor(document.getElementsByTagName('div'), 0).value is document.getElementsByTagName('div')[0]
 PASS Object.getOwnPropertyDescriptor(document.getElementsByTagName('div'), 0).hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(document.getElementsByTagName('div'), 0).hasOwnProperty('set') is false

Modified: trunk/LayoutTests/js/resources/getOwnPropertyDescriptor.js (210569 => 210570)


--- trunk/LayoutTests/js/resources/getOwnPropertyDescriptor.js	2017-01-11 01:28:00 UTC (rev 210569)
+++ trunk/LayoutTests/js/resources/getOwnPropertyDescriptor.js	2017-01-11 01:35:32 UTC (rev 210570)
@@ -36,15 +36,15 @@
 descriptorShouldBe("function(){}", "'length'", {writable: false, enumerable: false, configurable: true, value:0});
 descriptorShouldBe("Math.sin", "'length'", {writable: false, enumerable: false, configurable: true, value:1});
 descriptorShouldBe("Math.sin", "'name'", {writable: false, enumerable: false, configurable: true, value:"'sin'"});
-var globalObject = this;
-descriptorShouldBe("globalObject", "'globalObject'", {writable: true, enumerable: true, configurable: false, value:"globalObject"});
-descriptorShouldBe("globalObject", "'undefined'", {writable: false, enumerable: false, configurable: false, value:"undefined"});
-descriptorShouldBe("globalObject", "'NaN'", {writable: false, enumerable: false, configurable: false, value:"NaN"});
-descriptorShouldBe("globalObject", "'Infinity'", {writable: false, enumerable: false, configurable: false, value:"Infinity"});
-var globalWindowGetter = Object.getOwnPropertyDescriptor(globalObject, 'window').get;
-descriptorShouldBe("globalObject", "'window'", {get: 'globalWindowGetter', set: undefined, enumerable: true, configurable: false});
-descriptorShouldBe("globalObject", "'XMLHttpRequest'", {writable: true, enumerable: false, configurable: true, value:"XMLHttpRequest"});
-descriptorShouldBe("globalObject", "0", {writable: true, enumerable: false, configurable: false, value:"globalObject[0]"});
+var global = this;
+descriptorShouldBe("global", "'global'", {writable: true, enumerable: true, configurable: false, value:"global"});
+descriptorShouldBe("global", "'undefined'", {writable: false, enumerable: false, configurable: false, value:"undefined"});
+descriptorShouldBe("global", "'NaN'", {writable: false, enumerable: false, configurable: false, value:"NaN"});
+descriptorShouldBe("global", "'Infinity'", {writable: false, enumerable: false, configurable: false, value:"Infinity"});
+var globalWindowGetter = Object.getOwnPropertyDescriptor(global, 'window').get;
+descriptorShouldBe("global", "'window'", {get: 'globalWindowGetter', set: undefined, enumerable: true, configurable: false});
+descriptorShouldBe("global", "'XMLHttpRequest'", {writable: true, enumerable: false, configurable: true, value:"XMLHttpRequest"});
+descriptorShouldBe("global", "0", {writable: true, enumerable: false, configurable: false, value:"global[0]"});
 descriptorShouldBe("document.getElementsByTagName('div')", "0", {writable: false, enumerable: true, configurable: true, value:"document.getElementsByTagName('div')[0]"});
 descriptorShouldBe("document.getElementsByClassName('pass')", "0", {writable: false, enumerable: true, configurable: true, value:"document.getElementsByClassName('pass')[0]"});
 var canvas = document.createElement("canvas");

Modified: trunk/Source/_javascript_Core/ChangeLog (210569 => 210570)


--- trunk/Source/_javascript_Core/ChangeLog	2017-01-11 01:28:00 UTC (rev 210569)
+++ trunk/Source/_javascript_Core/ChangeLog	2017-01-11 01:35:32 UTC (rev 210570)
@@ -1,3 +1,17 @@
+2017-01-10  Commit Queue  <[email protected]>
+
+        Unreviewed, rolling out r210052.
+        https://bugs.webkit.org/show_bug.cgi?id=166915
+
+        "breaks web compatability" (Requested by keith_miller on
+        #webkit).
+
+        Reverted changeset:
+
+        "Add support for global"
+        https://bugs.webkit.org/show_bug.cgi?id=165171
+        http://trac.webkit.org/changeset/210052
+
 2017-01-10  Sam Weinig  <[email protected]>
 
         [WebIDL] Remove most of the custom bindings for the WebGL code

Modified: trunk/Source/_javascript_Core/runtime/JSGlobalObject.cpp (210569 => 210570)


--- trunk/Source/_javascript_Core/runtime/JSGlobalObject.cpp	2017-01-11 01:28:00 UTC (rev 210569)
+++ trunk/Source/_javascript_Core/runtime/JSGlobalObject.cpp	2017-01-11 01:35:32 UTC (rev 210570)
@@ -272,7 +272,6 @@
   encodeURI             globalFuncEncodeURI                          DontEnum|Function 1
   encodeURIComponent    globalFuncEncodeURIComponent                 DontEnum|Function 1
   EvalError             JSGlobalObject::m_evalErrorConstructor       DontEnum|CellProperty
-  global                JSGlobalObject::m_globalThis                 DontEnum|CellProperty
   ReferenceError        JSGlobalObject::m_referenceErrorConstructor  DontEnum|CellProperty
   SyntaxError           JSGlobalObject::m_syntaxErrorConstructor     DontEnum|CellProperty
   URIError              JSGlobalObject::m_URIErrorConstructor        DontEnum|CellProperty
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to