Fix and test use of property descriptor objects.

test/mjsunit/harmony/proxies.js

 // Copyright 2008 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 1050 matching lines @@
   get: function(pr, pk) { throw "myexn" }
 }))
 
 TestInThrow(Proxy.create({
   get: function(pr, pk) {
     return function(k) { throw "myexn" }
   }
 }))
 
 
-/* TODO(rossberg): does not work yet, JSProxy::GetPropertyAttributeWithHandler
- * is not fully implemented.*/
 function TestInForDerived(handler) {
   TestWithProxies(TestInForDerived2, handler)
 }
 
 function TestInForDerived2(handler, create) {
   var p = create(handler)
   var o = Object.create(p)
   assertTrue("a" in o)
   assertEquals("a", key)
-// TODO(rossberg): integer indexes not correctly imlemeted yet
+// TODO(rossberg): integer indexes not correctly implemeted yet
 //  assertTrue(99 in o)
 //  assertEquals("99", key)
   assertFalse("z" in o)
   assertEquals("z", key)
 
   assertEquals(2, ("a" in o) ? 2 : 0)
   assertEquals(0, !("a" in o) ? 2 : 0)
   assertEquals(0, ("zzz" in o) ? 2 : 0)
   assertEquals(2, !("zzz" in o) ? 2 : 0)
 
@@ skipping 64 matching lines @@
 */
 
 TestInForDerived(Proxy.create({
   get: function(pr, pk) {
     return function(k) { key = k; return k < "z" ? {value: 42} : void 0 }
   }
 }))
 
 
 
+// Property descriptor conversion.
+
+var descget
+
+function TestDescriptorGetOrder(handler) {
+  var p = Proxy.create(handler)
+  var o = Object.create(p, {b: {value: 0}})
+  TestDescriptorGetOrder2(function(n) { p[n] }, "vV")
+  TestDescriptorGetOrder2(function(n) { n in p }, "")
+  TestDescriptorGetOrder2(function(n) { o[n] }, "vV")
+  TestDescriptorGetOrder2(function(n) { n in o }, "eEcCvVwWgs")
+}
+
+function TestDescriptorGetOrder2(f, access) {
+  descget = ""
+  f("a")
+  assertEquals(access, descget)
+// TODO(rossberg): integer indexes not correctly implemented yet.
+//  descget = ""
+//  f(99)
+//  assertEquals(access, descget)
+  descget = ""
+  f("z")
+  assertEquals("", descget)
+}
+
+TestDescriptorGetOrder({
+  getPropertyDescriptor: function(k) {
+    if (k >= "z") return void 0
+    // Return a proxy as property descriptor, so that we can log accesses.
+    return Proxy.create({
+      get: function(r, attr) {
+        descget += attr[0].toUpperCase()
+        return true
+      },
+      has: function(attr) {
+        descget += attr[0]
+        switch (attr) {
+          case "writable":
+          case "enumerable":
+          case "configurable":
+          case "value":
+            return true
+          case "get":
+          case "set":
+            return false
+          default:
+            assertUnreachable()
+        }
+      }
+    })
+  }
+})
+
+
+
 // Own Properties (Object.prototype.hasOwnProperty).
 
 var key
 
 function TestHasOwn(handler) {
   TestWithProxies(TestHasOwn2, handler)
 }
 
 function TestHasOwn2(handler, create) {
   var p = create(handler)
@@ skipping 950 matching lines @@
 
 function TestConstructThrow2(f) {
   assertThrows(function(){ new f(11) }, "myexn")
   Object.freeze(f)
   assertThrows(function(){ new f(11) }, "myexn")
 }
 
 TestConstructThrow(function() { throw "myexn" })
 TestConstructThrow(Proxy.createFunction({}, function() { throw "myexn" }))
 TestConstructThrow(CreateFrozen({}, function() { throw "myexn" }))