| Index: chrome/common/extensions/extension_unittest.cc
|
| diff --git a/chrome/common/extensions/extension_unittest.cc b/chrome/common/extensions/extension_unittest.cc
|
| index 07bd6c6fc3aa707735d52f1f1d675793a068b319..ecc1d4ae4050a3aac689cc5459936938add7025b 100644
|
| --- a/chrome/common/extensions/extension_unittest.cc
|
| +++ b/chrome/common/extensions/extension_unittest.cc
|
| @@ -1105,9 +1105,13 @@ TEST(ExtensionTest, ApiPermissions) {
|
|
|
| TEST(ExtensionTest, GetDistinctHosts) {
|
| std::vector<std::string> expected;
|
| + std::vector<std::string> expected_no_rcd;
|
| expected.push_back("www.foo.com");
|
| expected.push_back("www.bar.com");
|
| expected.push_back("www.baz.com");
|
| + expected_no_rcd.push_back("www.foo.");
|
| + expected_no_rcd.push_back("www.bar.");
|
| + expected_no_rcd.push_back("www.baz.");
|
| URLPatternList actual;
|
|
|
| {
|
| @@ -1121,7 +1125,9 @@ TEST(ExtensionTest, GetDistinctHosts) {
|
| actual.push_back(
|
| URLPattern(URLPattern::SCHEME_HTTP, "http://www.baz.com/path"));
|
| CompareLists(expected,
|
| - Extension::GetDistinctHosts(actual));
|
| + Extension::GetDistinctHosts(actual, true));
|
| + CompareLists(expected_no_rcd,
|
| + Extension::GetDistinctHosts(actual, false));
|
| }
|
|
|
| {
|
| @@ -1133,7 +1139,9 @@ TEST(ExtensionTest, GetDistinctHosts) {
|
| actual.push_back(
|
| URLPattern(URLPattern::SCHEME_HTTP, "http://www.baz.com/path"));
|
| CompareLists(expected,
|
| - Extension::GetDistinctHosts(actual));
|
| + Extension::GetDistinctHosts(actual, true));
|
| + CompareLists(expected_no_rcd,
|
| + Extension::GetDistinctHosts(actual, false));
|
| }
|
|
|
| {
|
| @@ -1143,7 +1151,9 @@ TEST(ExtensionTest, GetDistinctHosts) {
|
| actual.push_back(
|
| URLPattern(URLPattern::SCHEME_HTTPS, "https://www.bar.com/path"));
|
| CompareLists(expected,
|
| - Extension::GetDistinctHosts(actual));
|
| + Extension::GetDistinctHosts(actual, true));
|
| + CompareLists(expected_no_rcd,
|
| + Extension::GetDistinctHosts(actual, false));
|
| }
|
|
|
| {
|
| @@ -1153,7 +1163,9 @@ TEST(ExtensionTest, GetDistinctHosts) {
|
| actual.push_back(
|
| URLPattern(URLPattern::SCHEME_HTTP, "http://www.bar.com/pathypath"));
|
| CompareLists(expected,
|
| - Extension::GetDistinctHosts(actual));
|
| + Extension::GetDistinctHosts(actual, true));
|
| + CompareLists(expected_no_rcd,
|
| + Extension::GetDistinctHosts(actual, false));
|
| }
|
|
|
| {
|
| @@ -1167,9 +1179,14 @@ TEST(ExtensionTest, GetDistinctHosts) {
|
|
|
| expected.push_back("monkey.www.bar.com");
|
| expected.push_back("bar.com");
|
| + expected_no_rcd.push_back("monkey.www.bar.");
|
| + expected_no_rcd.push_back("bar.");
|
| +
|
|
|
| CompareLists(expected,
|
| - Extension::GetDistinctHosts(actual));
|
| + Extension::GetDistinctHosts(actual, true));
|
| + CompareLists(expected_no_rcd,
|
| + Extension::GetDistinctHosts(actual, false));
|
| }
|
|
|
| {
|
| @@ -1194,9 +1211,12 @@ TEST(ExtensionTest, GetDistinctHosts) {
|
| URLPattern(URLPattern::SCHEME_HTTP, "http://www.foo.xyzzy/path"));
|
|
|
| expected.push_back("www.foo.xyzzy");
|
| + expected_no_rcd.push_back("www.foo.xyzzy");
|
|
|
| CompareLists(expected,
|
| - Extension::GetDistinctHosts(actual));
|
| + Extension::GetDistinctHosts(actual, true));
|
| + CompareLists(expected_no_rcd,
|
| + Extension::GetDistinctHosts(actual, false));
|
| }
|
| }
|
|
|
|
|