Only Register() platform specific content settings types on different platforms

components/content_settings/core/browser/website_settings_registry_unittest.cc

 // Copyright 2015 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 "components/content_settings/core/browser/website_settings_registry.h"
 
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "base/values.h"
 #include "components/content_settings/core/browser/website_settings_info.h"
 #include "components/content_settings/core/common/content_settings.h"
 #include "components/content_settings/core/common/content_settings_types.h"
 #include "components/pref_registry/pref_registry_syncable.h"
 #include "components/prefs/pref_registry.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace content_settings {
 
 class WebsiteSettingsRegistryTest : public testing::Test {
  protected:
   WebsiteSettingsRegistry* registry() { return &registry_; }
 
  private:
   WebsiteSettingsRegistry registry_;
 };
 
+#if !defined(OS_IOS)
 TEST_F(WebsiteSettingsRegistryTest, Get) {

[msramek, 2016/05/24 14:29:24]

I think the spirit of this test is not to test specifically whether APP_BANNER
is registered, but whether Get() works properly. Why not change APP_BANNER to
COOKIES to run the test everywhere?

Then we could add one more test to test specifically the platform-dependent
behavior, e.g. GetPlatformDependent that would test something that is not on
iOS, and something that is not on mobile.

[raymes, 2016/05/25 03:36:07]

+1

[lshang, 2016/05/25 10:38:16]

Done.

Cookies is ContentSettingsInfo, it don't get registered in
WebsiteSettingsRegistry. But yeah I agree with your idea and thanks for the
reminding! I added a GetPlatformDependent test in ContentSettingsRegistryTest to
test platform-dependent behavior.

For the 5 types in WebsiteSettingsInfo, auto-select-certificate is unsure
whether or not it is used on iOS, so I make it apply to all platforms until we
get some evidence. The other 4 types(ssl-cert-decision, app-banner,
site-engagement and usb-chooser-data) are confirmed to be not used. I checked in
.gypi file, they are under non-ios sources. So I changed app-banner to
auto-select-certificate in these tests.

   // CONTENT_SETTINGS_TYPE_APP_BANNER should be registered.
   const WebsiteSettingsInfo* info =
       registry()->Get(CONTENT_SETTINGS_TYPE_APP_BANNER);
   ASSERT_TRUE(info);
   EXPECT_EQ(CONTENT_SETTINGS_TYPE_APP_BANNER, info->type());
   EXPECT_EQ("app-banner", info->name());
 }
+#endif
 
 TEST_F(WebsiteSettingsRegistryTest, GetByName) {
   // Random string shouldn't be registered.
   EXPECT_FALSE(registry()->GetByName("abc"));
+  const WebsiteSettingsInfo* info;
 
+#if !defined(OS_IOS)

[msramek, 2016/05/24 14:29:24]

Ditto here.

[lshang, 2016/05/25 10:38:17]

Done.

   // "app-banner" should be registered.
-  const WebsiteSettingsInfo* info = registry()->GetByName("app-banner");
+  info = registry()->GetByName("app-banner");
   ASSERT_TRUE(info);
   EXPECT_EQ(CONTENT_SETTINGS_TYPE_APP_BANNER, info->type());
   EXPECT_EQ("app-banner", info->name());
   EXPECT_EQ(registry()->Get(CONTENT_SETTINGS_TYPE_APP_BANNER), info);
+#endif
 
   // Register a new setting.
   registry()->Register(static_cast<ContentSettingsType>(10), "test", nullptr,
                        WebsiteSettingsInfo::UNSYNCABLE,
                        WebsiteSettingsInfo::LOSSY,
                        WebsiteSettingsInfo::TOP_LEVEL_DOMAIN_ONLY_SCOPE,
+                       WebsiteSettingsInfo::ALL_PLATFORMS,
                        WebsiteSettingsInfo::INHERIT_IN_INCOGNITO);
   info = registry()->GetByName("test");
   ASSERT_TRUE(info);
   EXPECT_EQ(10, info->type());
   EXPECT_EQ("test", info->name());
   EXPECT_EQ(registry()->Get(static_cast<ContentSettingsType>(10)), info);
 }
 
 TEST_F(WebsiteSettingsRegistryTest, Properties) {
+  const WebsiteSettingsInfo* info;
+#if !defined(OS_IOS)

[msramek, 2016/05/24 14:29:24]

Ditto.

[lshang, 2016/05/25 10:38:16]

Done.

   // "app-banner" should be registered.
-  const WebsiteSettingsInfo* info =
-      registry()->Get(CONTENT_SETTINGS_TYPE_APP_BANNER);
+  info = registry()->Get(CONTENT_SETTINGS_TYPE_APP_BANNER);
   ASSERT_TRUE(info);
   EXPECT_EQ("profile.content_settings.exceptions.app_banner",
             info->pref_name());
   EXPECT_EQ("profile.default_content_setting_values.app_banner",
             info->default_value_pref_name());
   ASSERT_FALSE(info->initial_default_value());
   EXPECT_EQ(PrefRegistry::LOSSY_PREF, info->GetPrefRegistrationFlags());
-
+#endif
   // Register a new setting.
   registry()->Register(static_cast<ContentSettingsType>(10), "test",
                        base::WrapUnique(new base::FundamentalValue(999)),
                        WebsiteSettingsInfo::SYNCABLE,
                        WebsiteSettingsInfo::LOSSY,
                        WebsiteSettingsInfo::TOP_LEVEL_DOMAIN_ONLY_SCOPE,
+                       WebsiteSettingsInfo::ALL_PLATFORMS,
                        WebsiteSettingsInfo::INHERIT_IN_INCOGNITO);
   info = registry()->Get(static_cast<ContentSettingsType>(10));
   ASSERT_TRUE(info);
   EXPECT_EQ("profile.content_settings.exceptions.test", info->pref_name());
   EXPECT_EQ("profile.default_content_setting_values.test",
             info->default_value_pref_name());
   int setting;
   ASSERT_TRUE(info->initial_default_value()->GetAsInteger(&setting));
   EXPECT_EQ(999, setting);
+#if defined(OS_IOS)
+  EXPECT_EQ(PrefRegistry::LOSSY_PREF, info->GetPrefRegistrationFlags());
+#else
   EXPECT_EQ(PrefRegistry::LOSSY_PREF |
                 user_prefs::PrefRegistrySyncable::SYNCABLE_PREF,
             info->GetPrefRegistrationFlags());
+#endif
   EXPECT_EQ(WebsiteSettingsInfo::TOP_LEVEL_DOMAIN_ONLY_SCOPE,
             info->scoping_type());
   EXPECT_EQ(WebsiteSettingsInfo::INHERIT_IN_INCOGNITO,
             info->incognito_behavior());
 }
 
 TEST_F(WebsiteSettingsRegistryTest, Iteration) {
   registry()->Register(static_cast<ContentSettingsType>(10), "test",
                        base::WrapUnique(new base::FundamentalValue(999)),
                        WebsiteSettingsInfo::SYNCABLE,
                        WebsiteSettingsInfo::LOSSY,
                        WebsiteSettingsInfo::TOP_LEVEL_DOMAIN_ONLY_SCOPE,
+                       WebsiteSettingsInfo::ALL_PLATFORMS,
                        WebsiteSettingsInfo::INHERIT_IN_INCOGNITO);
 
   bool found = false;
   for (const WebsiteSettingsInfo* info : *registry()) {
     EXPECT_EQ(registry()->Get(info->type()), info);
     if (info->type() == 10) {
       EXPECT_FALSE(found);
       found = true;
     }
   }
 
   EXPECT_TRUE(found);
 }
 
 }  // namespace content_settings