Content settings pattern trim trailing dots from hostnames before matching a URL.

chrome/browser/content_settings/content_settings_pattern_unittest.cc

 // Copyright (c) 2011 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 "chrome/browser/content_settings/content_settings_pattern.h"
 
 #include "googleurl/src/gurl.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
@@ skipping 119 matching lines @@
   EXPECT_TRUE(ContentSettingsPattern::Wildcard().Matches(
       GURL("file:///foo/bar.txt")));
 
   EXPECT_STREQ("*", ContentSettingsPattern::Wildcard().ToString().c_str());
 
   EXPECT_EQ(ContentSettingsPattern::IDENTITY,
             ContentSettingsPattern::Wildcard().Compare(
                 ContentSettingsPattern::Wildcard()));
 }
 
+TEST(ContentSettingsPatternTest, TrimEndingDotFromHost) {
+  EXPECT_TRUE(Pattern("www.example.com").IsValid());
+  EXPECT_TRUE(Pattern("www.example.com").Matches(
+      GURL("http://www.example.com")));
+  EXPECT_TRUE(Pattern("www.example.com").Matches(
+      GURL("http://www.example.com.")));
+
+  EXPECT_TRUE(Pattern("www.example.com.").IsValid());
+  EXPECT_STREQ("www.example.com",
+               Pattern("www.example.com.").ToString().c_str());
+
+  EXPECT_TRUE(Pattern("www.example.com.") == Pattern("www.example.com"));
+}
+
 TEST(ContentSettingsPatternTest, FromString_WithNoWildcards) {
   // HTTP patterns with default port.
   EXPECT_TRUE(Pattern("http://www.example.com:80").IsValid());
   EXPECT_STREQ("http://www.example.com:80",
                Pattern("http://www.example.com:80").ToString().c_str());
   // HTTP patterns with none default port.
   EXPECT_TRUE(Pattern("http://www.example.com:81").IsValid());
   EXPECT_STREQ("http://www.example.com:81",
                Pattern("http://www.example.com:81").ToString().c_str());
 
@@ skipping 352 matching lines @@
   // file:/// normalization.
   EXPECT_STREQ("file:///tmp/test.html",
                Pattern("file:///tmp/bar/../test.html").ToString().c_str());
 
   // Invalid patterns.
   EXPECT_STREQ("", Pattern("*example.com").ToString().c_str());
   EXPECT_STREQ("", Pattern("example.*").ToString().c_str());
   EXPECT_STREQ("", Pattern("*\xC4\x87ira.com").ToString().c_str());
   EXPECT_STREQ("", Pattern("\xC4\x87ira.*").ToString().c_str());
 }