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

Unified Diff: chrome/browser/extensions/extension_omnibox_unittest.cc

Issue 10265022: Moving extensions inside chrome/browser/extensions/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 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/extension_omnibox_unittest.cc
diff --git a/chrome/browser/extensions/extension_omnibox_unittest.cc b/chrome/browser/extensions/extension_omnibox_unittest.cc
deleted file mode 100644
index a358b18c4dfdd88e0b944fa69dda10c7dd2aa546..0000000000000000000000000000000000000000
--- a/chrome/browser/extensions/extension_omnibox_unittest.cc
+++ /dev/null
@@ -1,133 +0,0 @@
-// Copyright (c) 2010 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/extension_omnibox_api.h"
-#include "testing/gtest/include/gtest/gtest.h"
-#include "testing/platform_test.h"
-
-namespace {
-
-const int kNone = ACMatchClassification::NONE;
-const int kUrl = ACMatchClassification::URL;
-const int kMatch = ACMatchClassification::MATCH;
-const int kDim = ACMatchClassification::DIM;
-
-void AppendStyle(const std::string& type,
- int offset, int length,
- ListValue* styles) {
- DictionaryValue* style = new DictionaryValue;
- style->SetString("type", type);
- style->SetInteger("offset", offset);
- style->SetInteger("length", length);
- styles->Append(style);
-}
-
-void CompareClassification(const ACMatchClassifications& expected,
- const ACMatchClassifications& actual) {
- EXPECT_EQ(expected.size(), actual.size());
- for (size_t i = 0; i < expected.size() && i < actual.size(); ++i) {
- EXPECT_EQ(expected[i].offset, actual[i].offset) << "Index:" << i;
- EXPECT_EQ(expected[i].style, actual[i].style) << "Index:" << i;
- }
-}
-
-} // 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);
-
- 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);
-
- // 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);
-}
-
-// 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);
-
- 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);
-
- // 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);
-}
-
-// 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);
-
- 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);
-}

Powered by Google App Engine
This is Rietveld 408576698