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

Side by Side Diff: components/content_settings/core/common/content_settings_pattern_unittest.cc

Issue 1895993003: Add migration code to change existing domain scoped content settings to be origin scoped (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: enhance tests Created 4 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/content_settings/core/common/content_settings_pattern.h" 5 #include "components/content_settings/core/common/content_settings_pattern.h"
6 6
7 #include "testing/gtest/include/gtest/gtest.h" 7 #include "testing/gtest/include/gtest/gtest.h"
8 #include "url/gurl.h" 8 #include "url/gurl.h"
9 9
10 namespace { 10 namespace {
(...skipping 702 matching lines...) Expand 10 before | Expand all | Expand 10 after
713 // file:/// normalization. 713 // file:/// normalization.
714 EXPECT_STREQ("file:///tmp/test.html", 714 EXPECT_STREQ("file:///tmp/test.html",
715 Pattern("file:///tmp/bar/../test.html").ToString().c_str()); 715 Pattern("file:///tmp/bar/../test.html").ToString().c_str());
716 716
717 // Invalid patterns. 717 // Invalid patterns.
718 EXPECT_STREQ("", Pattern("*example.com").ToString().c_str()); 718 EXPECT_STREQ("", Pattern("*example.com").ToString().c_str());
719 EXPECT_STREQ("", Pattern("example.*").ToString().c_str()); 719 EXPECT_STREQ("", Pattern("example.*").ToString().c_str());
720 EXPECT_STREQ("", Pattern("*\xC4\x87ira.com").ToString().c_str()); 720 EXPECT_STREQ("", Pattern("*\xC4\x87ira.com").ToString().c_str());
721 EXPECT_STREQ("", Pattern("\xC4\x87ira.*").ToString().c_str()); 721 EXPECT_STREQ("", Pattern("\xC4\x87ira.*").ToString().c_str());
722 } 722 }
723
724 TEST(ContentSettingsPatternTest, MigrateFromDomainToOrigin) {
725 ContentSettingsPattern origin_pattern;
726 // Http scheme patterns.
727 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
728 ContentSettingsPattern::FromString("http://[*.]example.com"),
729 &origin_pattern));
730 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
731 ContentSettingsPattern::FromString("http://[*.]example.com:80"),
732 &origin_pattern));
733
734 // Https patterns with port wildcard.
735 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
736 ContentSettingsPattern::FromString("https://www.google.com"),
737 &origin_pattern));
738 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
739 ContentSettingsPattern::FromString("https://[*.]google.com"),
740 &origin_pattern));
741
742 // Patterns with no domain wildcard.
743 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
744 ContentSettingsPattern::FromString("*://www.google.com:8080"),
745 &origin_pattern));
746 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
747 ContentSettingsPattern::FromString("www.example.com:8080"),
748 &origin_pattern));
749 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
750 ContentSettingsPattern::FromString("www.google.com/*"), &origin_pattern));
751 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
752 ContentSettingsPattern::FromString("google"), &origin_pattern));
753 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
754 ContentSettingsPattern::FromString("https://www.google.com:443"),
755 &origin_pattern));
756
757 // Patterns with empty host.
758 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
759 ContentSettingsPattern::FromString("*"), &origin_pattern));
760 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
761 ContentSettingsPattern::FromString("[*.]"), &origin_pattern));
762 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
763 ContentSettingsPattern::FromString("http://*"), &origin_pattern));
764 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
765 ContentSettingsPattern::FromString("http://*:8080"), &origin_pattern));
766
767 // Other schemes and IP address patterns won't be migrated.
768 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
769 ContentSettingsPattern::FromString("192.168.0.1"), &origin_pattern));
770 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
771 ContentSettingsPattern::FromString("https://127.0.0.1"),
772 &origin_pattern));
773 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
774 ContentSettingsPattern::FromString("http://[::1]"), &origin_pattern));
775 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
776 ContentSettingsPattern::FromString("[::1]"), &origin_pattern));
777 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
778 ContentSettingsPattern::FromString("file:///foo/bar.html"),
779 &origin_pattern));
780 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
781 ContentSettingsPattern::FromString(
782 "filesystem:http://www.google.com/temporary/"),
783 &origin_pattern));
784 EXPECT_FALSE(ContentSettingsPattern::MigrateFromDomainToOrigin(
785 ContentSettingsPattern::FromString(
786 "chrome-extension://peoadpeiejnhkmpaakpnompolbglelel/"),
787 &origin_pattern));
788
789 // These are pattern styles which might be generated using FromURL().
790 EXPECT_TRUE(ContentSettingsPattern::MigrateFromDomainToOrigin(
791 ContentSettingsPattern::FromString("[*.]example.com"), &origin_pattern));
792 EXPECT_EQ("http://example.com:80", origin_pattern.ToString());
793
794 EXPECT_TRUE(ContentSettingsPattern::MigrateFromDomainToOrigin(
795 ContentSettingsPattern::FromString("[*.]google.com:80"),
796 &origin_pattern));
797 EXPECT_EQ("http://google.com:80", origin_pattern.ToString());
798
799 EXPECT_TRUE(ContentSettingsPattern::MigrateFromDomainToOrigin(
800 ContentSettingsPattern::FromString("[*.]example.com:123"),
801 &origin_pattern));
802 EXPECT_EQ("http://example.com:123", origin_pattern.ToString());
803
804 EXPECT_TRUE(ContentSettingsPattern::MigrateFromDomainToOrigin(
805 ContentSettingsPattern::FromString("https://[*.]google.com:443"),
806 &origin_pattern));
807 EXPECT_EQ("https://google.com:443", origin_pattern.ToString());
808 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698