Modified Omnibox extension api to use JSON Schema Compiler

chrome/browser/extensions/api/omnibox/omnibox_unittest.cc

 // Copyright (c) 2012 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 "base/values.h"
 #include "chrome/browser/extensions/api/omnibox/omnibox_api.h"
+#include "chrome/common/extensions/api/omnibox.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "testing/platform_test.h"
 
 namespace extensions {
 
 namespace {
 
 const int kNone = ACMatchClassification::NONE;
 const int kUrl = ACMatchClassification::URL;
 const int kMatch = ACMatchClassification::MATCH;
@@ skipping 21 matching lines @@
 }  // namespace
 
 // Test output key: n = character with no styling, d = dim, m = match, u = url
 // u = 1, m = 2, d = 4. u+d = 5, etc.
 
 //   0123456789
 //    mmmm
 // +       ddd
 // = nmmmmndddn
 TEST(ExtensionOmniboxTest, DescriptionStylesSimple) {
-  ListValue styles_value;
-  AppendStyle("match", 1, 4, &styles_value);
-  AppendStyle("dim", 6, 3, &styles_value);
+  scoped_ptr<ListValue> styles_value(new ListValue);
+  AppendStyle("match", 1, 4, styles_value.get());
+  AppendStyle("dim", 6, 3, styles_value.get());
+
+  scoped_ptr<DictionaryValue> dict(new DictionaryValue);
+  dict->SetStringWithoutPathExpansion("content", "content");
+  dict->SetStringWithoutPathExpansion("description", "dscription");
+  dict->SetWithoutPathExpansion("descriptionStyles", styles_value.release());

[not at google - send to devlin, 2013/03/16 00:04:25]

you'll probably find chrome/common/extensions/value_builder.h useful. It will
clean up all of these tests a bit.

[Aaron Jacobs, 2013/03/21 21:59:55]

Done.

+
+  scoped_ptr<ListValue> dict_list(new ListValue);
+  dict_list->Set(0, dict.release());
+
+  ListValue list;
+  // Set the request_id
+  list.Set(0, new base::FundamentalValue(42));
+  list.Set(1, dict_list.release());
 
   ACMatchClassifications styles_expected;
   styles_expected.push_back(ACMatchClassification(0, kNone));
   styles_expected.push_back(ACMatchClassification(1, kMatch));
   styles_expected.push_back(ACMatchClassification(5, kNone));
   styles_expected.push_back(ACMatchClassification(6, kDim));
   styles_expected.push_back(ACMatchClassification(9, kNone));
 
-  ExtensionOmniboxSuggestion suggestions;
-  suggestions.description.resize(10);
-  EXPECT_TRUE(suggestions.ReadStylesFromValue(styles_value));
-  CompareClassification(styles_expected, suggestions.description_styles);
+  scoped_ptr<api::omnibox::SendSuggestions::Params> params(
+      api::omnibox::SendSuggestions::Params::Create(list));

[not at google - send to devlin, 2013/03/16 00:04:25]

for consistency, use same typedefs in the api file?

[Aaron Jacobs, 2013/03/21 21:59:55]

Done.

+  EXPECT_TRUE(params);
+  EXPECT_TRUE(params->suggest_results[0].get());

[not at google - send to devlin, 2013/03/16 00:04:25]

.get() unnecessary

[Aaron Jacobs, 2013/03/21 21:59:55]

I can't remove it (it's a linked_ptr, not a scoped_ptr).

+  CompareClassification(styles_expected, StyleTypesToACMatchClassifications(
+      *params->suggest_results[0]));
 
   // Same input, but swap the order. Ensure it still works.
-  styles_value.Clear();
-  AppendStyle("dim", 6, 3, &styles_value);
-  AppendStyle("match", 1, 4, &styles_value);
-  EXPECT_TRUE(suggestions.ReadStylesFromValue(styles_value));
-  CompareClassification(styles_expected, suggestions.description_styles);
+  scoped_ptr<ListValue> swapped_styles_value(new ListValue);
+  AppendStyle("dim", 6, 3, swapped_styles_value.get());
+  AppendStyle("match", 1, 4, swapped_styles_value.get());
+
+  scoped_ptr<DictionaryValue> swapped_dict(new DictionaryValue);
+  swapped_dict->SetStringWithoutPathExpansion("content", "content");
+  swapped_dict->SetStringWithoutPathExpansion("description", "dscription");
+  swapped_dict->SetWithoutPathExpansion("descriptionStyles",
+      swapped_styles_value.release());
+
+  scoped_ptr<ListValue> swapped_dict_list(new ListValue);
+  swapped_dict_list->Set(0, swapped_dict.release());
+
+  ListValue swapped_list;
+  // Set the request_id
+  swapped_list.Set(0, new base::FundamentalValue(42));
+  swapped_list.Set(1, swapped_dict_list.release());
+
+  scoped_ptr<api::omnibox::SendSuggestions::Params> swapped_params(
+      api::omnibox::SendSuggestions::Params::Create(swapped_list));
+  EXPECT_TRUE(swapped_params);
+  EXPECT_TRUE(swapped_params->suggest_results[0].get());
+  CompareClassification(styles_expected, StyleTypesToACMatchClassifications(
+      *swapped_params->suggest_results[0]));
 }
 
 //   0123456789
 //   uuuuu
 // +          dd
 // +          mm
 // + mmmm
 // +  dd
 // = 3773unnnn66
 TEST(ExtensionOmniboxTest, DescriptionStylesCombine) {
-  ListValue styles_value;
-  AppendStyle("url", 0, 5, &styles_value);
-  AppendStyle("dim", 9, 2, &styles_value);
-  AppendStyle("match", 9, 2, &styles_value);
-  AppendStyle("match", 0, 4, &styles_value);
-  AppendStyle("dim", 1, 2, &styles_value);
+  scoped_ptr<ListValue> styles_value(new ListValue);
+  AppendStyle("url", 0, 5, styles_value.get());
+  AppendStyle("dim", 9, 2, styles_value.get());
+  AppendStyle("match", 9, 2, styles_value.get());
+  AppendStyle("match", 0, 4, styles_value.get());
+  AppendStyle("dim", 1, 2, styles_value.get());
+
+  scoped_ptr<DictionaryValue> dict(new DictionaryValue);
+  dict->SetStringWithoutPathExpansion("content", "content");
+  dict->SetStringWithoutPathExpansion("description", "dscription");
+  dict->SetWithoutPathExpansion("descriptionStyles", styles_value.release());
+
+  scoped_ptr<ListValue> dict_list(new ListValue);
+  dict_list->Set(0, dict.release());
+
+  ListValue list;
+  // Set the request_id
+  list.Set(0, new base::FundamentalValue(42));
+  list.Set(1, dict_list.release());
 
   ACMatchClassifications styles_expected;
   styles_expected.push_back(ACMatchClassification(0, kUrl | kMatch));
   styles_expected.push_back(ACMatchClassification(1, kUrl | kMatch | kDim));
   styles_expected.push_back(ACMatchClassification(3, kUrl | kMatch));
   styles_expected.push_back(ACMatchClassification(4, kUrl));
   styles_expected.push_back(ACMatchClassification(5, kNone));
   styles_expected.push_back(ACMatchClassification(9, kMatch | kDim));
 
-  ExtensionOmniboxSuggestion suggestions;
-  suggestions.description.resize(10);
-  EXPECT_TRUE(suggestions.ReadStylesFromValue(styles_value));
-  CompareClassification(styles_expected, suggestions.description_styles);
+  scoped_ptr<api::omnibox::SendSuggestions::Params> params(
+      api::omnibox::SendSuggestions::Params::Create(list));
+  EXPECT_TRUE(params);
+  EXPECT_TRUE(params->suggest_results[0].get());
+  CompareClassification(styles_expected, StyleTypesToACMatchClassifications(
+      *params->suggest_results[0]));
 
   // Try moving the "dim/match" style pair at offset 9. Output should be the
   // same.
-  styles_value.Clear();
-  AppendStyle("url", 0, 5, &styles_value);
-  AppendStyle("match", 0, 4, &styles_value);
-  AppendStyle("dim", 9, 2, &styles_value);
-  AppendStyle("match", 9, 2, &styles_value);
-  AppendStyle("dim", 1, 2, &styles_value);
-  EXPECT_TRUE(suggestions.ReadStylesFromValue(styles_value));
-  CompareClassification(styles_expected, suggestions.description_styles);
+  scoped_ptr<ListValue> moved_styles_value(new ListValue);
+  AppendStyle("url", 0, 5, moved_styles_value.get());
+  AppendStyle("match", 0, 4, moved_styles_value.get());
+  AppendStyle("dim", 9, 2, moved_styles_value.get());
+  AppendStyle("match", 9, 2, moved_styles_value.get());
+  AppendStyle("dim", 1, 2, moved_styles_value.get());
+
+  scoped_ptr<DictionaryValue> moved_dict(new DictionaryValue);
+  moved_dict->SetStringWithoutPathExpansion("content", "content");
+  moved_dict->SetStringWithoutPathExpansion("description", "dscription");
+  moved_dict->SetWithoutPathExpansion("descriptionStyles",
+      moved_styles_value.release());
+
+  scoped_ptr<ListValue> moved_dict_list(new ListValue);
+  moved_dict_list->Set(0, moved_dict.release());
+
+  ListValue moved_list;
+  // Set the request_id
+  moved_list.Set(0, new base::FundamentalValue(42));
+  moved_list.Set(1, moved_dict_list.release());
+
+  scoped_ptr<api::omnibox::SendSuggestions::Params> moved_params(
+      api::omnibox::SendSuggestions::Params::Create(moved_list));
+  EXPECT_TRUE(moved_params);
+  EXPECT_TRUE(moved_params->suggest_results[0].get());
+  CompareClassification(styles_expected, StyleTypesToACMatchClassifications(
+      *moved_params->suggest_results[0]));
 }
 
 //   0123456789
 //   uuuuu
 // + mmmmm
 // + mmm
 // +   ddd
 // + ddd
 // = 77777nnnnn
 TEST(ExtensionOmniboxTest, DescriptionStylesCombine2) {
-  ListValue styles_value;
-  AppendStyle("url", 0, 5, &styles_value);
-  AppendStyle("match", 0, 5, &styles_value);
-  AppendStyle("match", 0, 3, &styles_value);
-  AppendStyle("dim", 2, 3, &styles_value);
-  AppendStyle("dim", 0, 3, &styles_value);
+  scoped_ptr<ListValue> styles_value(new ListValue);
+  AppendStyle("url", 0, 5, styles_value.get());
+  AppendStyle("match", 0, 5, styles_value.get());
+  AppendStyle("match", 0, 3, styles_value.get());
+  AppendStyle("dim", 2, 3, styles_value.get());
+  AppendStyle("dim", 0, 3, styles_value.get());
+
+  scoped_ptr<DictionaryValue> dict(new DictionaryValue);
+  dict->SetStringWithoutPathExpansion("content", "content");
+  dict->SetStringWithoutPathExpansion("description", "dscription");
+  dict->SetWithoutPathExpansion("descriptionStyles", styles_value.release());
+
+  scoped_ptr<ListValue> dict_list(new ListValue);
+  dict_list->Set(0, dict.release());
+
+  ListValue list;
+  // Set the request_id
+  list.Set(0, new base::FundamentalValue(42));
+  list.Set(1, dict_list.release());
 
   ACMatchClassifications styles_expected;
   styles_expected.push_back(ACMatchClassification(0, kUrl | kMatch | kDim));
   styles_expected.push_back(ACMatchClassification(5, kNone));
 
-  ExtensionOmniboxSuggestion suggestions;
-  suggestions.description.resize(10);
-  EXPECT_TRUE(suggestions.ReadStylesFromValue(styles_value));
-  CompareClassification(styles_expected, suggestions.description_styles);
+  scoped_ptr<api::omnibox::SendSuggestions::Params> params(
+      api::omnibox::SendSuggestions::Params::Create(list));
+  EXPECT_TRUE(params);
+  EXPECT_TRUE(params->suggest_results[0].get());
+  CompareClassification(styles_expected, StyleTypesToACMatchClassifications(
+      *params->suggest_results[0]));
 }
 
 }  // namespace extensions