Add PPB_VarArray_Dev support - part 1.

ppapi/shared_impl/var_value_conversions_unittest.cc

 // Copyright (c) 2013 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.
 
 #include "ppapi/shared_impl/var_value_conversions.h"
 
 #include <cmath>
 #include <cstring>
 
 #include "base/logging.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/values.h"
 #include "ppapi/c/pp_bool.h"
 #include "ppapi/c/pp_var.h"
+#include "ppapi/shared_impl/array_var.h"
 #include "ppapi/shared_impl/dictionary_var.h"
 #include "ppapi/shared_impl/proxy_lock.h"
 #include "ppapi/shared_impl/scoped_pp_var.h"
 #include "ppapi/shared_impl/test_globals.h"
 #include "ppapi/shared_impl/var.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace ppapi {
 namespace {
 
@@ skipping 52 matching lines @@
            !value_iter.IsAtEnd(); value_iter.Advance()) {
         DictionaryVar::KeyValueMap::const_iterator var_iter =
             dict_var->key_value_map().find(value_iter.key());
         if (var_iter == dict_var->key_value_map().end() ||
             !Equals(value_iter.value(), var_iter->second.get()))
           return false;
       }
       return true;
     }
     case base::Value::TYPE_LIST: {
-      // TODO(yzshen): add support once array var is supported.
-      return false;
+      const base::ListValue& list_value =
+          static_cast<const base::ListValue&>(value);
+      ArrayVar* array_var = ArrayVar::FromPPVar(var);
+      if (!array_var || list_value.GetSize() != array_var->elements().size())
+        return false;
+
+      base::ListValue::const_iterator value_iter = list_value.begin();
+      ArrayVar::ElementVector::const_iterator var_iter =
+          array_var->elements().begin();
+      for (; value_iter != list_value.end() &&
+                 var_iter != array_var->elements().end();

[raymes, 2013/03/05 18:19:16]

indentation here looks wrong

[yzshen1, 2013/03/14 22:36:48]

It is 4 spaces further than "value_iter != list_value.end()".
Every way I tried seemed weird. :) (4 more spaces than "for" / "(" /
"value_iter").
Which one do you prefer?

On 2013/03/05 18:19:16, raymes wrote:

+           ++value_iter, ++var_iter) {
+        if (!Equals(**value_iter, var_iter->get()))
+          return false;
+      }
+      return true;
     }
   }
   NOTREACHED();
   return false;
 }
 
 class VarValueConversionsTest : public testing::Test {
  public:
   VarValueConversionsTest() {
   }
@@ skipping 21 matching lines @@
   {
     // Undefined var is not allowed even if it is stored in a dictionary var.
     scoped_refptr<DictionaryVar> dict_var(new DictionaryVar());
     ASSERT_TRUE(dict_var->SetWithStringKey("key_1", PP_MakeUndefined()));
     ScopedPPVar var(ScopedPPVar::PassRef(), dict_var->GetPPVar());
     ASSERT_EQ(NULL, CreateValueFromVar(var.get()));
   }
 
   {
     // Var holding a ref to itself is not a valid input.
-    scoped_refptr<DictionaryVar> dict_var_1(new DictionaryVar());
-    ScopedPPVar var_1(ScopedPPVar::PassRef(), dict_var_1->GetPPVar());
-    scoped_refptr<DictionaryVar> dict_var_2(new DictionaryVar());
-    ScopedPPVar var_2(ScopedPPVar::PassRef(), dict_var_2->GetPPVar());
+    scoped_refptr<DictionaryVar> dict_var(new DictionaryVar());
+    ScopedPPVar var_1(ScopedPPVar::PassRef(), dict_var->GetPPVar());
+    scoped_refptr<ArrayVar> array_var(new ArrayVar());
+    ScopedPPVar var_2(ScopedPPVar::PassRef(), array_var->GetPPVar());
 
-    ASSERT_TRUE(dict_var_1->SetWithStringKey("key_1", var_2.get()));
+    ASSERT_TRUE(dict_var->SetWithStringKey("key_1", var_2.get()));
     scoped_ptr<base::Value> value(CreateValueFromVar(var_1.get()));
-    ASSERT_TRUE(value.get() != NULL);
+    ASSERT_TRUE(value.get());
 
-    ASSERT_TRUE(dict_var_2->SetWithStringKey("key_2", var_1.get()));
+    ASSERT_TRUE(array_var->Set(0, var_1.get()));
     value.reset(CreateValueFromVar(var_1.get()));
     ASSERT_EQ(NULL, value.get());
 
     // Make sure |var_1| doesn't indirectly hold a ref to itself, otherwise it
     // is leaked.
-    dict_var_1->DeleteWithStringKey("key_1");
+    dict_var->DeleteWithStringKey("key_1");
   }
 
   {
     // Test valid inputs.
     scoped_refptr<DictionaryVar> dict_var_1(new DictionaryVar());
     ScopedPPVar dict_pp_var_1(ScopedPPVar::PassRef(), dict_var_1->GetPPVar());
     scoped_refptr<DictionaryVar> dict_var_2(new DictionaryVar());
     ScopedPPVar dict_pp_var_2(ScopedPPVar::PassRef(), dict_var_2->GetPPVar());
+    scoped_refptr<ArrayVar> array_var(new ArrayVar());
+    ScopedPPVar array_pp_var(ScopedPPVar::PassRef(), array_var->GetPPVar());
     scoped_refptr<StringVar> string_var(new StringVar("string_value"));
     ScopedPPVar string_pp_var(ScopedPPVar::PassRef(), string_var->GetPPVar());
 
     ASSERT_TRUE(dict_var_1->SetWithStringKey("null_key", PP_MakeNull()));
     ASSERT_TRUE(dict_var_1->SetWithStringKey("string_key",
                                              string_pp_var.get()));
     ASSERT_TRUE(dict_var_1->SetWithStringKey("dict_key", dict_pp_var_2.get()));
 
     ASSERT_TRUE(dict_var_2->SetWithStringKey("double_key", PP_MakeDouble(1)));
-    ASSERT_TRUE(dict_var_2->SetWithStringKey("int_key", PP_MakeInt32(2)));
-    ASSERT_TRUE(dict_var_2->SetWithStringKey("bool_key", PP_MakeBool(PP_TRUE)));
+    ASSERT_TRUE(dict_var_2->SetWithStringKey("array_key", array_pp_var.get()));
+
+    ASSERT_TRUE(array_var->Set(0, PP_MakeInt32(2)));
+    ASSERT_TRUE(array_var->Set(1, PP_MakeBool(PP_TRUE)));

[raymes, 2013/03/05 18:19:16]

Seems like we could have some more simple conversion tests for array (e.g. 0
element array, 1 element array) both to Var and from Var.

[yzshen1, 2013/03/14 22:36:48]

Done.

 
     scoped_ptr<base::Value> value(CreateValueFromVar(dict_pp_var_1.get()));
-    ASSERT_TRUE(value.get() != NULL);
+    ASSERT_TRUE(value.get());
     ASSERT_TRUE(Equals(*value, dict_pp_var_1.get()));
   }
 
   {
     // Test that dictionary keys containing '.' are handled correctly.
     scoped_refptr<DictionaryVar> dict_var(new DictionaryVar());
     ScopedPPVar dict_pp_var(ScopedPPVar::PassRef(), dict_var->GetPPVar());
 
     ASSERT_TRUE(dict_var->SetWithStringKey("double.key", PP_MakeDouble(1)));
     ASSERT_TRUE(dict_var->SetWithStringKey("int.key..name", PP_MakeInt32(2)));
 
     scoped_ptr<base::Value> value(CreateValueFromVar(dict_pp_var.get()));
-    ASSERT_TRUE(value.get() != NULL);
+    ASSERT_TRUE(value.get());
     ASSERT_TRUE(Equals(*value, dict_pp_var.get()));
   }
 }
 
 TEST_F(VarValueConversionsTest, CreateVarFromValue) {
   base::DictionaryValue dict_value;
   dict_value.Set("null_key", base::Value::CreateNullValue());
   dict_value.SetString("string_key", "string_value");
   dict_value.SetDouble("dict_key.double_key", 1);
-  dict_value.SetInteger("dict_key.int_key", 2);
-  dict_value.SetBoolean("dict_key.bool_key", true);
+
+  scoped_ptr<base::ListValue> list_value(new base::ListValue());
+  list_value->AppendInteger(2);
+  list_value->AppendBoolean(true);
+
+  dict_value.Set("dict_key.array_key", list_value.release());
 
   ScopedPPVar var(ScopedPPVar::PassRef(), CreateVarFromValue(dict_value));
   ASSERT_TRUE(Equals(dict_value, var.get()));
 }
 
 }  // namespace ppapi