Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1481)

Unified Diff: chrome/browser/extensions/api/omnibox/omnibox_unittest.cc

Issue 1833773002: [Extensions] Convert APIs to use movable types [8] (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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

Powered by Google App Engine
This is Rietveld 408576698