Struct traits used by GpuInfo

mojo/common/common_custom_types.typemap

Index: mojo/common/common_custom_types.typemap
diff --git a/mojo/common/common_custom_types.typemap b/mojo/common/common_custom_types.typemap
index e3197aa73fc22481284db15cd69c1cac3b6f33d0..2234d5e8e98ad53a5614572a65a40d82222831cd 100644
--- a/mojo/common/common_custom_types.typemap
+++ b/mojo/common/common_custom_types.typemap
@@ -7,8 +7,15 @@ public_headers = [
   "//base/files/file_path.h",
   "//base/values.h",
   "//base/time/time.h",
+  "//base/version.h",
+]
+traits_headers = [
+  "//ipc/ipc_message_utils.h",
+  "//mojo/common/common_custom_types_struct_traits.h",
+]
+sources = [

[Ken Rockot(use gerrit already), 2016/07/27 20:05:25]

Do not add sources to typemaps.

Please create a new source_set target like
//mojo/common:common_custom_struct_traits and add the cc and h files to its
sources. Then add that target to public_deps below.

[Alex Z., 2016/07/28 19:40:44]

I have found sources in other typemap files. For example,
https://cs.chromium.org/chromium/src/cc/ipc/quads.typemap?q=_struct_traits.cc....
Is there any particular reason why sources shouldn't be in typemaps?

[Ken Rockot(use gerrit already), 2016/07/28 19:46:36]

Hmm. I didn't think we supported this, but mojom.gni clearly does read sources
from the typemap. Nevermind :)

+  "//mojo/common/common_custom_types_struct_traits.cc",
 ]
-traits_headers = [ "//ipc/ipc_message_utils.h" ]
 public_deps = [
   "//ipc",
 ]
@@ -20,4 +27,5 @@ type_mappings = [
   "mojo.common.mojom.Time=base::Time[copyable_pass_by_value]",
   "mojo.common.mojom.TimeDelta=base::TimeDelta[copyable_pass_by_value]",
   "mojo.common.mojom.TimeTicks=base::TimeTicks[copyable_pass_by_value]",
+  "mojo.common.mojom.Version=base::Version",
 ]