Write mojom and EnumTraits for display::HDCPState.

ui/display/mojo/display_constants_struct_traits.cc

Index: ui/display/mojo/display_constants_struct_traits.cc
diff --git a/ui/display/mojo/display_constants_struct_traits.cc b/ui/display/mojo/display_constants_struct_traits.cc
index fd710869c244dcafc36936dc2809db25cc1f469e..f94c41cea39f3a614ec303555b23205e4b4c9a11 100644
--- a/ui/display/mojo/display_constants_struct_traits.cc
+++ b/ui/display/mojo/display_constants_struct_traits.cc
@@ -96,4 +96,42 @@ bool EnumTraits<display::mojom::DisplayConnectionType,
   return false;
 }
 
+// static
+display::mojom::HDCPState
+EnumTraits<display::mojom::HDCPState, display::HDCPState>::ToMojom(
+    display::HDCPState type) {
+  switch (type) {
+    case display::HDCPState::HDCP_STATE_UNDESIRED:
+      return display::mojom::HDCPState::HDCP_STATE_UNDESIRED;
+

[Daniel Erat, 2017/03/09 14:46:32]

i know you're just being consistent with the existing style here, but i'm
curious about why this file has blank lines between cases (which i don't think
i've seen elsewhere in switch statements).

[thanhph, 2017/03/09 15:33:11]

Done, I remove new lines between 2 switch cases in this file.

+    case display::HDCPState::HDCP_STATE_DESIRED:
+      return display::mojom::HDCPState::HDCP_STATE_DESIRED;
+
+    case display::HDCPState::HDCP_STATE_ENABLED:
+      return display::mojom::HDCPState::HDCP_STATE_ENABLED;
+  }
+  NOTREACHED();
+  return display::mojom::HDCPState::HDCP_STATE_UNDESIRED;
+}
+
+// static
+bool EnumTraits<display::mojom::HDCPState, display::HDCPState>::FromMojom(
+    display::mojom::HDCPState type,
+    display::HDCPState* out) {
+  switch (type) {
+    case display::mojom::HDCPState::HDCP_STATE_UNDESIRED:
+      *out = display::HDCPState::HDCP_STATE_UNDESIRED;
+      return true;
+
+    case display::mojom::HDCPState::HDCP_STATE_DESIRED:
+      *out = display::HDCPState::HDCP_STATE_DESIRED;
+      return true;
+
+    case display::mojom::HDCPState::HDCP_STATE_ENABLED:
+      *out = display::HDCPState::HDCP_STATE_ENABLED;
+      return true;
+  }

[Daniel Erat, 2017/03/09 14:46:32]

nit: should this have a NOTREACHED() too?

[thanhph, 2017/03/09 15:33:12]

The deserialization should fail if the code reaches here. Returning false here
seems to be sufficient and let the caller address the failure.

+  return false;
+}
+
 }  // namespace mojo