Add a test that JSGlobalProxy properly causes Object.observe to trigger access check callbacks

test/cctest/test-object-observe.cc

 // Copyright 2012 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 694 matching lines @@
     const RecordExpectation expected_records2[] = {
       { instance, "add", "5", Handle<Value>() },
       { instance, "add", "foo", Handle<Value>() }
     };
     EXPECT_RECORDS(CompileRun("records2"), expected_records2);
   }
   CHECK(CompileRun("records")->IsNull());
 }
 
 
+static bool NamedAccessAlwaysDisallowed(Local<Object>, Local<Value>, AccessType,
+                                        Local<Value>) {
+  return false;
+}
+
+
+static bool IndexedAccessAlwaysDisallowed(Local<Object>, uint32_t, AccessType,
+                                          Local<Value>) {
+  return false;
+}
+
+
+TEST(AccessCheckedGlobalProxy) {
+  HandleScope scope(CcTest::isolate());
+  Handle<ObjectTemplate> global_template =
+      ObjectTemplate::New(CcTest::isolate());
+  global_template->SetAccessCheckCallbacks(NamedAccessAlwaysDisallowed,
+                                           IndexedAccessAlwaysDisallowed);
+  Handle<Context> context =
+      Context::New(CcTest::isolate(), NULL, global_template);
+  Handle<Object> global_proxy = context->Global();
+  {
+    LocalContext context2(CcTest::isolate());
+    context2->Global()->Set(String::NewFromUtf8(CcTest::isolate(), "global"),
+                            global_proxy);
+    CompileRun("var records = null;"
+               "var observer = function(r) { records = r };"
+               "Object.observe(global, observer);");
+
+    {
+      Context::Scope context_scope(context);
+      CompileRun("foo = 42");
+      CHECK_EQ(42, CompileRun("foo")->Int32Value());
+    }
+    CHECK(CompileRun("records")->IsNull());
+    CHECK(CompileRun("global.foo")->IsUndefined());
+  }
+}
+
+
 TEST(HiddenPropertiesLeakage) {
   HandleScope scope(CcTest::isolate());
   LocalContext context(CcTest::isolate());
   CompileRun("var obj = {};"
              "var records = null;"
              "var observer = function(r) { records = r };"
              "Object.observe(obj, observer);");
   Handle<Value> obj =
       context->Global()->Get(String::NewFromUtf8(CcTest::isolate(), "obj"));
   Handle<Object>::Cast(obj)
       ->SetHiddenValue(String::NewFromUtf8(CcTest::isolate(), "foo"),
                        Null(CcTest::isolate()));
   CompileRun("");  // trigger delivery
   CHECK(CompileRun("records")->IsNull());
 }