Mojo C++ bindings: make String16 and gfx::Size available in Blink

mojo/common/common_custom_types.mojom

Index: mojo/common/common_custom_types.mojom
diff --git a/mojo/common/common_custom_types.mojom b/mojo/common/common_custom_types.mojom
index 6c0f4eba773e7103215a1394ea31fc71fde0635b..6dcdf76fec5f044a91ee0361cc9e278462eb4943 100644
--- a/mojo/common/common_custom_types.mojom
+++ b/mojo/common/common_custom_types.mojom
@@ -23,8 +23,13 @@ struct TimeDelta {
 [Native]
 struct TimeTicks;
 
-[Native]
-struct String16;
+// Corresponds to |base::string16| in base/strings/string16.h
+// Corresponds to |blink::WebString| in

[Tom Sepez, 2016/09/29 20:13:08]

Wouldn't this violate DEPS, in some sense; I didn't think /mojo was allowed to
know about blink (base OK since mojom built on base).

[Zhiqiang Zhang (Slow), 2016/09/29 20:39:31]

There some other types which has been mapped to blink structures, such as:
String->WTF::String
Array->WTF::Vector

In general, my purpose is to send structures from WebKit/Source to chromium
directly, so we could avoid having a layer in content/renderer to do the type
conversion.

There's another example, which maps mojo Url to blink::KURL. See "KURL.typemap"

I think the *.mojom-blink.* bindings are only used in Blink, which serializes
Blink structures for mojo communication, so it's probably OK?

Maybe rockot@ has better answer.

[Ken Rockot(use gerrit already), 2016/09/29 22:00:48]

This is fine. The typemap which actually imposes DEPS is defined in
third_party/WebKit and only used by third_party/WebKit code.

It's a little weird to reference blink types even in comments, but I think it's
OK in this case, for clarity.

+// third_party/WebKit/public/platform/WebString.h, and only accept non-null
+// |blink::WebString|.
+struct String16 {
+  array<uint16> data;
+};
 
 // Corresponds to |base::UnguessableToken| in base/unguessable_token.h
 struct UnguessableToken {