Index: chrome/browser/extensions/api/omnibox/omnibox_unittest.cc |
diff --git a/chrome/browser/extensions/api/omnibox/omnibox_unittest.cc b/chrome/browser/extensions/api/omnibox/omnibox_unittest.cc |
index 20a05b56d701affad6f56312cf341286d3012532..75eeafc05a2323a55357117f1cf167d3e95a61f9 100644 |
--- a/chrome/browser/extensions/api/omnibox/omnibox_unittest.cc |
+++ b/chrome/browser/extensions/api/omnibox/omnibox_unittest.cc |
@@ -79,9 +79,8 @@ TEST(ExtensionOmniboxTest, DescriptionStylesSimple) { |
scoped_ptr<SendSuggestions::Params> params( |
SendSuggestions::Params::Create(*list)); |
EXPECT_TRUE(params); |
- EXPECT_TRUE(params->suggest_results[0].get()); |
lazyboy
2016/03/25 17:57:07
Here and in other places: should keep an equivalen
Devlin
2016/03/29 20:20:35
Done, though note that's not an equivalent check -
|
CompareClassification(styles_expected, StyleTypesToACMatchClassifications( |
- *params->suggest_results[0])); |
+ params->suggest_results[0])); |
// Same input, but swap the order. Ensure it still works. |
scoped_ptr<base::ListValue> swap_list = |
@@ -111,9 +110,9 @@ TEST(ExtensionOmniboxTest, DescriptionStylesSimple) { |
scoped_ptr<SendSuggestions::Params> swapped_params( |
SendSuggestions::Params::Create(*swap_list)); |
EXPECT_TRUE(swapped_params); |
- EXPECT_TRUE(swapped_params->suggest_results[0].get()); |
- CompareClassification(styles_expected, StyleTypesToACMatchClassifications( |
- *swapped_params->suggest_results[0])); |
+ CompareClassification( |
+ styles_expected, |
+ StyleTypesToACMatchClassifications(swapped_params->suggest_results[0])); |
} |
// 0123456789 |
@@ -174,9 +173,8 @@ TEST(ExtensionOmniboxTest, DescriptionStylesCombine) { |
scoped_ptr<SendSuggestions::Params> params( |
SendSuggestions::Params::Create(*list)); |
EXPECT_TRUE(params); |
- EXPECT_TRUE(params->suggest_results[0].get()); |
CompareClassification(styles_expected, StyleTypesToACMatchClassifications( |
- *params->suggest_results[0])); |
+ params->suggest_results[0])); |
// Try moving the "dim/match" style pair at offset 9. Output should be the |
// same. |
@@ -222,9 +220,8 @@ TEST(ExtensionOmniboxTest, DescriptionStylesCombine) { |
scoped_ptr<SendSuggestions::Params> moved_params( |
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])); |
+ moved_params->suggest_results[0])); |
} |
// 0123456789 |
@@ -281,9 +278,8 @@ TEST(ExtensionOmniboxTest, DescriptionStylesCombine2) { |
scoped_ptr<SendSuggestions::Params> params( |
SendSuggestions::Params::Create(*list)); |
EXPECT_TRUE(params); |
- EXPECT_TRUE(params->suggest_results[0].get()); |
CompareClassification(styles_expected, StyleTypesToACMatchClassifications( |
- *params->suggest_results[0])); |
+ params->suggest_results[0])); |
} |
// 0123456789 |