Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(192)

Unified Diff: mojo/public/cpp/bindings/tests/wtf_types_unittest.cc

Issue 2034273002: Mojo C++ bindings: introduce mojo::WTFMap for blink bindings. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: removed the code of adding enum key support to WTF::HashMap; which turned out to be non-trivial; wi… Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « mojo/public/cpp/bindings/tests/wtf_map_unittest.cc ('k') | mojo/public/cpp/bindings/wtf_array.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/public/cpp/bindings/tests/wtf_types_unittest.cc
diff --git a/mojo/public/cpp/bindings/tests/wtf_types_unittest.cc b/mojo/public/cpp/bindings/tests/wtf_types_unittest.cc
index 636366df7981fdaf05e3b099c885666c319d5847..f691085eff0382858715f2a586a73b4f4d2c992e 100644
--- a/mojo/public/cpp/bindings/tests/wtf_types_unittest.cc
+++ b/mojo/public/cpp/bindings/tests/wtf_types_unittest.cc
@@ -39,6 +39,11 @@ class TestWTFImpl : public TestWTF {
callback.Run(std::move(arr));
}
+ void EchoStringMap(Map<String, String> str_map,
+ const EchoStringMapCallback& callback) override {
+ callback.Run(std::move(str_map));
+ }
+
private:
Binding<TestWTF> binding_;
};
@@ -62,6 +67,16 @@ WTFArray<WTF::String> ConstructStringArray() {
return strs;
}
+WTFMap<WTF::String, WTF::String> ConstructStringMap() {
+ WTFMap<WTF::String, WTF::String> str_map;
+ // A null string as value.
+ str_map.insert("0", WTF::String());
+ str_map.insert("1", kHelloWorld);
+ str_map.insert("2", WTF::String::fromUTF8(kUTF8HelloWorld));
+
+ return str_map;
+}
+
} // namespace
TEST_F(WTFTypesTest, Serialization_WTFArrayToWTFArray) {
@@ -80,7 +95,7 @@ TEST_F(WTFTypesTest, Serialization_WTFArrayToWTFArray) {
&validate_params, &context);
WTFArray<WTF::String> strs2;
- mojo::internal::Deserialize<Array<mojo::String>>(data, &strs2, nullptr);
+ mojo::internal::Deserialize<Array<mojo::String>>(data, &strs2, &context);
EXPECT_TRUE(strs.Equals(strs2));
}
@@ -101,7 +116,7 @@ TEST_F(WTFTypesTest, Serialization_WTFVectorToWTFVector) {
&validate_params, &context);
WTF::Vector<WTF::String> strs2;
- mojo::internal::Deserialize<Array<mojo::String>>(data, &strs2, nullptr);
+ mojo::internal::Deserialize<Array<mojo::String>>(data, &strs2, &context);
EXPECT_EQ(strs, strs2);
}
@@ -121,7 +136,7 @@ TEST_F(WTFTypesTest, Serialization_WTFArrayToMojoArray) {
&validate_params, &context);
Array<mojo::String> strs2;
- mojo::internal::Deserialize<Array<mojo::String>>(data, &strs2, nullptr);
+ mojo::internal::Deserialize<Array<mojo::String>>(data, &strs2, &context);
ASSERT_EQ(4u, strs2.size());
EXPECT_TRUE(strs2[0].is_null());
@@ -130,6 +145,66 @@ TEST_F(WTFTypesTest, Serialization_WTFArrayToMojoArray) {
EXPECT_TRUE(kUTF8HelloWorld == strs2[3]);
}
+TEST_F(WTFTypesTest, Serialization_WTFMapToWTFMap) {
+ using WTFType = WTFMap<WTF::String, WTF::String>;
+ using MojomType = Map<mojo::String, mojo::String>;
+
+ WTFType str_map = ConstructStringMap();
+ WTFType cloned_str_map = str_map.Clone();
+
+ mojo::internal::SerializationContext context;
+ size_t size =
+ mojo::internal::PrepareToSerialize<MojomType>(cloned_str_map, &context);
+
+ mojo::internal::FixedBufferForTesting buf(size);
+ typename MojomType::Data_* data;
+ mojo::internal::ContainerValidateParams validate_params(
+ new mojo::internal::ContainerValidateParams(
+ 0, false,
+ new mojo::internal::ContainerValidateParams(0, false, nullptr)),
+ new mojo::internal::ContainerValidateParams(
+ 0, true,
+ new mojo::internal::ContainerValidateParams(0, false, nullptr)));
+ mojo::internal::Serialize<MojomType>(cloned_str_map, &buf, &data,
+ &validate_params, &context);
+
+ WTFType str_map2;
+ mojo::internal::Deserialize<MojomType>(data, &str_map2, &context);
+
+ EXPECT_TRUE(str_map.Equals(str_map2));
+}
+
+TEST_F(WTFTypesTest, Serialization_WTFMapToMojoMap) {
+ using WTFType = WTFMap<WTF::String, WTF::String>;
+ using MojomType = Map<mojo::String, mojo::String>;
+
+ WTFType str_map = ConstructStringMap();
+
+ mojo::internal::SerializationContext context;
+ size_t size =
+ mojo::internal::PrepareToSerialize<MojomType>(str_map, &context);
+
+ mojo::internal::FixedBufferForTesting buf(size);
+ typename MojomType::Data_* data;
+ mojo::internal::ContainerValidateParams validate_params(
+ new mojo::internal::ContainerValidateParams(
+ 0, false,
+ new mojo::internal::ContainerValidateParams(0, false, nullptr)),
+ new mojo::internal::ContainerValidateParams(
+ 0, true,
+ new mojo::internal::ContainerValidateParams(0, false, nullptr)));
+ mojo::internal::Serialize<MojomType>(str_map, &buf, &data, &validate_params,
+ &context);
+
+ MojomType str_map2;
+ mojo::internal::Deserialize<MojomType>(data, &str_map2, &context);
+
+ ASSERT_EQ(3u, str_map2.size());
+ EXPECT_TRUE(str_map2["0"].is_null());
+ EXPECT_TRUE(kHelloWorld == str_map2["1"]);
+ EXPECT_TRUE(kUTF8HelloWorld == str_map2["2"]);
+}
+
TEST_F(WTFTypesTest, SendString) {
blink::TestWTFPtr ptr;
TestWTFImpl impl(ConvertInterfaceRequest<TestWTF>(GetProxy(&ptr)));
@@ -171,7 +246,7 @@ TEST_F(WTFTypesTest, SendStringArray) {
// - serialized;
// - deserialized as mojo::WTFArray<WTF::String>.
ptr->EchoStringArray(std::move(arrs[i]),
- [&loop, &expected_arr, &i](WTFArray<WTF::String> arr) {
+ [&loop, &expected_arr](WTFArray<WTF::String> arr) {
EXPECT_TRUE(expected_arr.Equals(arr));
loop.Quit();
});
@@ -179,5 +254,34 @@ TEST_F(WTFTypesTest, SendStringArray) {
}
}
+TEST_F(WTFTypesTest, SendStringMap) {
+ blink::TestWTFPtr ptr;
+ TestWTFImpl impl(ConvertInterfaceRequest<TestWTF>(GetProxy(&ptr)));
+
+ WTFMap<WTF::String, WTF::String> maps[3];
+ // maps[0] is empty.
+ // maps[1] is null.
+ maps[1] = nullptr;
+ maps[2] = ConstructStringMap();
+
+ for (size_t i = 0; i < arraysize(maps); ++i) {
+ WTFMap<WTF::String, WTF::String> expected_map = maps[i].Clone();
+ base::RunLoop loop;
+ // Test that a mojo::WTFMap<WTF::String, WTF::String> is unchanged after the
+ // following conversion:
+ // - serialized;
+ // - deserialized as mojo::Map<mojo::String, mojo::String>;
+ // - serialized;
+ // - deserialized as mojo::WTFMap<WTF::String, WTF::String>.
+ ptr->EchoStringMap(
+ std::move(maps[i]),
+ [&loop, &expected_map](WTFMap<WTF::String, WTF::String> map) {
+ EXPECT_TRUE(expected_map.Equals(map));
+ loop.Quit();
+ });
+ loop.Run();
+ }
+}
+
} // namespace test
} // namespace mojo
« no previous file with comments | « mojo/public/cpp/bindings/tests/wtf_map_unittest.cc ('k') | mojo/public/cpp/bindings/wtf_array.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698