Handle the case where the v8 scriptable object has a property defined with value undefined.

ppapi/tests/blink_deprecated_test_plugin.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // A simple C++ Pepper plugin for exercising deprecated PPAPI interfaces in
 // Blink layout tests.
 //
 // Most layout tests should prefer to use the normal Blink test plugin, with the
 // MIME type application/x-blink-test-plugin. For layout tests that absolutely
 // need to test deprecated synchronous scripting interfaces, this plugin can be
@@ skipping 119 matching lines @@
         base::Bind(&InstanceSO::TestExecuteScript, base::Unretained(this))));
     methods_.insert(std::make_pair(
         "testGetProperty",
         base::Bind(&InstanceSO::TestGetProperty, base::Unretained(this))));
     methods_.insert(std::make_pair(
         "testPassTestObject",
         base::Bind(&InstanceSO::TestPassTestObject, base::Unretained(this))));
     properties_.insert(std::make_pair(
         "testObject", base::Bind(&InstanceSO::TestObjectAccessor,
                                  base::Unretained(this))));
+    properties_.insert(std::make_pair(
+        "testGetUndefined", base::Bind(&InstanceSO::TestGetUndefinedAccessor,
+                                       base::Unretained(this))));
   }
   ~InstanceSO() override {}
 
  private:
   // Requires one argument. The argument is passed through as-is to
   // pp::InstancePrivate::ExecuteScript().
   pp::Var TestExecuteScript(const std::vector<pp::Var>& args,
                             pp::Var* exception) {
     if (args.size() != 1) {
       *exception = pp::Var("testExecuteScript requires one argument");
@@ skipping 33 matching lines @@
   }
 
   void TestObjectAccessor(bool set, pp::Var* var) {
     if (set)
       return;
     if (test_object_.is_undefined())
       test_object_ = pp::VarPrivate(instance_, new TestObjectSO(instance_));
     *var = test_object_;
   }
 
+  void TestGetUndefinedAccessor(bool set, pp::Var* var) {
+    if (set)
+      return;
+    *var = pp::Var();
+  }
+
   pp::VarPrivate test_object_;
 };
 
 class BlinkDeprecatedTestInstance : public pp::InstancePrivate {
  public:
   explicit BlinkDeprecatedTestInstance(PP_Instance instance)
       : pp::InstancePrivate(instance) {}
   ~BlinkDeprecatedTestInstance() override {
     LogMessage("%s", "Destroying");
   }
@@ skipping 38 matching lines @@
 
 }  // namespace
 
 namespace pp {
 
 Module* CreateModule() {
   return new BlinkDeprecatedTestModule();
 }
 
 }  // namespace pp