Write mojom and EnumTraits for display::HDCPState.

ui/display/mojo/display_constants_struct_traits.cc

 // Copyright 2017 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 "ui/display/mojo/display_constants_struct_traits.h"
 
 namespace mojo {
 
 display::mojom::DisplayConnectionType EnumTraits<
     display::mojom::DisplayConnectionType,
@@ skipping 78 matching lines @@
       *out = display::DisplayConnectionType::DISPLAY_CONNECTION_TYPE_NETWORK;
       return true;
 
     case display::mojom::DisplayConnectionType::DISPLAY_CONNECTION_TYPE_VIRTUAL:
       *out = display::DisplayConnectionType::DISPLAY_CONNECTION_TYPE_VIRTUAL;
       return true;
   }
   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