commit 2a17a53c02d6504e5d3a98f8de03e992b5f60408
author Alex Vakulenko <avakulenko@chromium.org> Tue Feb 24 14:51:13 2015 -0800
committer ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> Thu Feb 26 20:30:54 2015 +0000
tree a1855948b125b183f7131bfb62bfb1a262d4ce73
parent 431f43049cc9189f9dbcf88ab5f20f16224af908

buffet: Add support for 'default' properties in CDD

The internals of supporting optional command parameters/state
properties was built into buffet earlier, but this was not exposed
in JSON reading/writing routines, so it was impossible to use this
feature.

Added JSON serialization/deserialization code and unit tests
to verify the operation.

BUG=brillo:357
TEST=`FEATURES=test emerge-link buffet`

Change-Id: I29c8d3d7c0894a9c837e73d0fdb16bafdfadfeca
Reviewed-on: https://chromium-review.googlesource.com/253070
Trybot-Ready: Alex Vakulenko <avakulenko@chromium.org>
Tested-by: Alex Vakulenko <avakulenko@chromium.org>
Reviewed-by: Christopher Wiley <wiley@chromium.org>
Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>

buffet/commands/object_schema.cc

diff --git a/buffet/commands/object_schema.cc b/buffet/commands/object_schema.cc
index 41e7639..6437262 100644
--- a/buffet/commands/object_schema.cc
+++ b/buffet/commands/object_schema.cc
@@ -236,6 +236,18 @@
       commands::attributes::kOneOf_Enum, &list))
     return DetectArrayType(list, base_schema);
 
+  // If we have "default", try to use it for type detection.
+  if (dict->Get(commands::attributes::kDefault, &value)) {
+    if (value->IsType(base::Value::TYPE_DOUBLE))
+      return PropType::GetTypeStringFromType(ValueType::Double);
+    if (value->IsType(base::Value::TYPE_INTEGER))
+      return PropType::GetTypeStringFromType(ValueType::Int);
+    if (value->IsType(base::Value::TYPE_BOOLEAN))
+      return PropType::GetTypeStringFromType(ValueType::Boolean);
+    if (value->IsType(base::Value::TYPE_STRING))
+      return PropType::GetTypeStringFromType(ValueType::String);
+  }
+
   return std::string();
 }