Web Inspector: Don't shadow global Object constructor in WebInspector.Object class...
authorutatane.tea@gmail.com <utatane.tea@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 13 Oct 2015 19:03:46 +0000 (19:03 +0000)
committerutatane.tea@gmail.com <utatane.tea@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 13 Oct 2015 19:03:46 +0000 (19:03 +0000)
https://bugs.webkit.org/show_bug.cgi?id=150093

Reviewed by Joseph Pecoraro.

Currently, class expression does not define any variables. So even in `class Object { }`,
we were able to touch the global Object constructor (like `Object.keys`).
However, that is a bug. After https://bugs.webkit.org/show_bug.cgi?id=150089 fixes that bug,
the global `Object` is shadowed by the user-defined `class Object` name.

To solve this issue, we changed the `class Object` to `class WebInspectorObject`.
And we expose this class as a `WebInspector.Object`.

* UserInterface/Base/Object.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@191001 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebInspectorUI/ChangeLog
Source/WebInspectorUI/UserInterface/Base/Object.js

index 0a5fc4a..18990d2 100644 (file)
@@ -1,3 +1,20 @@
+2015-10-13  Yusuke Suzuki  <utatane.tea@gmail.com>
+
+        Web Inspector: Don't shadow global Object constructor in WebInspector.Object class definition
+        https://bugs.webkit.org/show_bug.cgi?id=150093
+
+        Reviewed by Joseph Pecoraro.
+
+        Currently, class expression does not define any variables. So even in `class Object { }`,
+        we were able to touch the global Object constructor (like `Object.keys`).
+        However, that is a bug. After https://bugs.webkit.org/show_bug.cgi?id=150089 fixes that bug,
+        the global `Object` is shadowed by the user-defined `class Object` name.
+
+        To solve this issue, we changed the `class Object` to `class WebInspectorObject`.
+        And we expose this class as a `WebInspector.Object`.
+
+        * UserInterface/Base/Object.js:
+
 2015-10-10  Nikita Vasilyev  <nvasilyev@apple.com>
 
         Web Inspector: Improve error messages for internal Inspector code
index 4965fa0..22a3983 100644 (file)
@@ -23,7 +23,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-WebInspector.Object = class Object
+WebInspector.Object = class WebInspectorObject
 {
     // Static