| OLD | NEW |
| 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 "chrome/installer/util/google_update_settings.h" | 5 #include "chrome/installer/util/google_update_settings.h" |
| 6 | 6 |
| 7 #include <windows.h> | 7 #include <windows.h> |
| 8 #include <shlwapi.h> // For SHDeleteKey. | 8 #include <shlwapi.h> // For SHDeleteKey. |
| 9 #include <stddef.h> | 9 #include <stddef.h> |
| 10 | 10 |
| 11 #include <memory> | 11 #include <memory> |
| 12 | 12 |
| 13 #include "base/base_paths.h" | 13 #include "base/base_paths.h" |
| 14 #include "base/macros.h" | 14 #include "base/macros.h" |
| 15 #include "base/memory/ptr_util.h" | 15 #include "base/memory/ptr_util.h" |
| 16 #include "base/path_service.h" | 16 #include "base/path_service.h" |
| 17 #include "base/strings/utf_string_conversions.h" | 17 #include "base/strings/utf_string_conversions.h" |
| 18 #include "base/test/scoped_path_override.h" | 18 #include "base/test/scoped_path_override.h" |
| 19 #include "base/test/test_reg_util_win.h" | 19 #include "base/test/test_reg_util_win.h" |
| 20 #include "base/win/registry.h" | 20 #include "base/win/registry.h" |
| 21 #include "base/win/win_util.h" | 21 #include "base/win/win_util.h" |
| 22 #include "chrome/common/chrome_constants.h" | 22 #include "chrome/common/chrome_constants.h" |
| 23 #include "chrome/install_static/install_util.h" |
| 23 #include "chrome/install_static/test/scoped_install_details.h" | 24 #include "chrome/install_static/test/scoped_install_details.h" |
| 24 #include "chrome/installer/util/app_registration_data.h" | 25 #include "chrome/installer/util/app_registration_data.h" |
| 25 #include "chrome/installer/util/browser_distribution.h" | 26 #include "chrome/installer/util/browser_distribution.h" |
| 26 #include "chrome/installer/util/channel_info.h" | 27 #include "chrome/installer/util/channel_info.h" |
| 27 #include "chrome/installer/util/fake_installation_state.h" | 28 #include "chrome/installer/util/fake_installation_state.h" |
| 28 #include "chrome/installer/util/google_update_constants.h" | 29 #include "chrome/installer/util/google_update_constants.h" |
| 29 #include "chrome/installer/util/helper.h" | 30 #include "chrome/installer/util/helper.h" |
| 30 #include "chrome/installer/util/util_constants.h" | 31 #include "chrome/installer/util/util_constants.h" |
| 31 #include "chrome/installer/util/work_item_list.h" | 32 #include "chrome/installer/util/work_item_list.h" |
| 32 #include "testing/gtest/include/gtest/gtest.h" | 33 #include "testing/gtest/include/gtest/gtest.h" |
| (...skipping 666 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 699 KEY_SET_VALUE).WriteValue(app_policy_value.c_str(), | 700 KEY_SET_VALUE).WriteValue(app_policy_value.c_str(), |
| 700 static_cast<DWORD>(4))); | 701 static_cast<DWORD>(4))); |
| 701 is_overridden = true; | 702 is_overridden = true; |
| 702 EXPECT_EQ(GoogleUpdateSettings::UPDATES_DISABLED, | 703 EXPECT_EQ(GoogleUpdateSettings::UPDATES_DISABLED, |
| 703 GoogleUpdateSettings::GetAppUpdatePolicy(kTestProductGuid, | 704 GoogleUpdateSettings::GetAppUpdatePolicy(kTestProductGuid, |
| 704 &is_overridden)); | 705 &is_overridden)); |
| 705 EXPECT_FALSE(is_overridden); | 706 EXPECT_FALSE(is_overridden); |
| 706 } | 707 } |
| 707 | 708 |
| 708 TEST_F(GoogleUpdateSettingsTest, PerAppUpdatesDisabledByPolicy) { | 709 TEST_F(GoogleUpdateSettingsTest, PerAppUpdatesDisabledByPolicy) { |
| 709 BrowserDistribution* dist = BrowserDistribution::GetDistribution(); | 710 const wchar_t* app_guid = install_static::GetAppGuid(); |
| 710 EXPECT_TRUE( | 711 EXPECT_TRUE(SetUpdatePolicyForAppGuid( |
| 711 SetUpdatePolicyForAppGuid(dist->GetAppGuid(), | 712 app_guid, GoogleUpdateSettings::UPDATES_DISABLED)); |
| 712 GoogleUpdateSettings::UPDATES_DISABLED)); | |
| 713 bool is_overridden = false; | 713 bool is_overridden = false; |
| 714 GoogleUpdateSettings::UpdatePolicy update_policy = | 714 GoogleUpdateSettings::UpdatePolicy update_policy = |
| 715 GoogleUpdateSettings::GetAppUpdatePolicy(dist->GetAppGuid(), | 715 GoogleUpdateSettings::GetAppUpdatePolicy(app_guid, &is_overridden); |
| 716 &is_overridden); | |
| 717 EXPECT_TRUE(is_overridden); | 716 EXPECT_TRUE(is_overridden); |
| 718 EXPECT_EQ(GoogleUpdateSettings::UPDATES_DISABLED, update_policy); | 717 EXPECT_EQ(GoogleUpdateSettings::UPDATES_DISABLED, update_policy); |
| 719 EXPECT_FALSE(GoogleUpdateSettings::AreAutoupdatesEnabled()); | 718 EXPECT_FALSE(GoogleUpdateSettings::AreAutoupdatesEnabled()); |
| 720 | 719 |
| 721 EXPECT_TRUE(GoogleUpdateSettings::ReenableAutoupdates()); | 720 EXPECT_TRUE(GoogleUpdateSettings::ReenableAutoupdates()); |
| 722 update_policy = GoogleUpdateSettings::GetAppUpdatePolicy(dist->GetAppGuid(), | 721 update_policy = |
| 723 &is_overridden); | 722 GoogleUpdateSettings::GetAppUpdatePolicy(app_guid, &is_overridden); |
| 724 // Should still have a policy but now that policy should explicitly enable | 723 // Should still have a policy but now that policy should explicitly enable |
| 725 // updates. | 724 // updates. |
| 726 EXPECT_TRUE(is_overridden); | 725 EXPECT_TRUE(is_overridden); |
| 727 EXPECT_EQ(GoogleUpdateSettings::AUTOMATIC_UPDATES, update_policy); | 726 EXPECT_EQ(GoogleUpdateSettings::AUTOMATIC_UPDATES, update_policy); |
| 728 EXPECT_TRUE(GoogleUpdateSettings::AreAutoupdatesEnabled()); | 727 EXPECT_TRUE(GoogleUpdateSettings::AreAutoupdatesEnabled()); |
| 729 } | 728 } |
| 730 | 729 |
| 731 TEST_F(GoogleUpdateSettingsTest, PerAppUpdatesEnabledWithGlobalDisabled) { | 730 TEST_F(GoogleUpdateSettingsTest, PerAppUpdatesEnabledWithGlobalDisabled) { |
| 732 // Disable updates globally but enable them for Chrome (the app-specific | 731 // Disable updates globally but enable them for Chrome (the app-specific |
| 733 // setting should take precedence). | 732 // setting should take precedence). |
| 734 BrowserDistribution* dist = BrowserDistribution::GetDistribution(); | 733 const wchar_t* app_guid = install_static::GetAppGuid(); |
| 735 EXPECT_TRUE( | 734 EXPECT_TRUE(SetUpdatePolicyForAppGuid( |
| 736 SetUpdatePolicyForAppGuid(dist->GetAppGuid(), | 735 app_guid, GoogleUpdateSettings::AUTOMATIC_UPDATES)); |
| 737 GoogleUpdateSettings::AUTOMATIC_UPDATES)); | |
| 738 EXPECT_TRUE(SetGlobalUpdatePolicy(GoogleUpdateSettings::UPDATES_DISABLED)); | 736 EXPECT_TRUE(SetGlobalUpdatePolicy(GoogleUpdateSettings::UPDATES_DISABLED)); |
| 739 | 737 |
| 740 // Make sure we read this as still having updates enabled. | 738 // Make sure we read this as still having updates enabled. |
| 741 EXPECT_TRUE(GoogleUpdateSettings::AreAutoupdatesEnabled()); | 739 EXPECT_TRUE(GoogleUpdateSettings::AreAutoupdatesEnabled()); |
| 742 | 740 |
| 743 // Make sure that the reset action returns true and is a no-op. | 741 // Make sure that the reset action returns true and is a no-op. |
| 744 EXPECT_TRUE(GoogleUpdateSettings::ReenableAutoupdates()); | 742 EXPECT_TRUE(GoogleUpdateSettings::ReenableAutoupdates()); |
| 745 EXPECT_EQ(GoogleUpdateSettings::AUTOMATIC_UPDATES, | 743 EXPECT_EQ(GoogleUpdateSettings::AUTOMATIC_UPDATES, |
| 746 GetUpdatePolicyForAppGuid(dist->GetAppGuid())); | 744 GetUpdatePolicyForAppGuid(app_guid)); |
| 747 EXPECT_EQ(GoogleUpdateSettings::UPDATES_DISABLED, GetGlobalUpdatePolicy()); | 745 EXPECT_EQ(GoogleUpdateSettings::UPDATES_DISABLED, GetGlobalUpdatePolicy()); |
| 748 } | 746 } |
| 749 | 747 |
| 750 TEST_F(GoogleUpdateSettingsTest, GlobalUpdatesDisabledByPolicy) { | 748 TEST_F(GoogleUpdateSettingsTest, GlobalUpdatesDisabledByPolicy) { |
| 751 BrowserDistribution* dist = BrowserDistribution::GetDistribution(); | 749 const wchar_t* app_guid = install_static::GetAppGuid(); |
| 752 EXPECT_TRUE(SetGlobalUpdatePolicy(GoogleUpdateSettings::UPDATES_DISABLED)); | 750 EXPECT_TRUE(SetGlobalUpdatePolicy(GoogleUpdateSettings::UPDATES_DISABLED)); |
| 753 bool is_overridden = false; | 751 bool is_overridden = false; |
| 754 | 752 |
| 755 // The contract for GetAppUpdatePolicy states that |is_overridden| should be | 753 // The contract for GetAppUpdatePolicy states that |is_overridden| should be |
| 756 // set to false when updates are disabled on a non-app-specific basis. | 754 // set to false when updates are disabled on a non-app-specific basis. |
| 757 GoogleUpdateSettings::UpdatePolicy update_policy = | 755 GoogleUpdateSettings::UpdatePolicy update_policy = |
| 758 GoogleUpdateSettings::GetAppUpdatePolicy(dist->GetAppGuid(), | 756 GoogleUpdateSettings::GetAppUpdatePolicy(app_guid, &is_overridden); |
| 759 &is_overridden); | |
| 760 EXPECT_FALSE(is_overridden); | 757 EXPECT_FALSE(is_overridden); |
| 761 EXPECT_EQ(GoogleUpdateSettings::UPDATES_DISABLED, update_policy); | 758 EXPECT_EQ(GoogleUpdateSettings::UPDATES_DISABLED, update_policy); |
| 762 EXPECT_FALSE(GoogleUpdateSettings::AreAutoupdatesEnabled()); | 759 EXPECT_FALSE(GoogleUpdateSettings::AreAutoupdatesEnabled()); |
| 763 | 760 |
| 764 EXPECT_TRUE(GoogleUpdateSettings::ReenableAutoupdates()); | 761 EXPECT_TRUE(GoogleUpdateSettings::ReenableAutoupdates()); |
| 765 update_policy = GoogleUpdateSettings::GetAppUpdatePolicy(dist->GetAppGuid(), | 762 update_policy = |
| 766 &is_overridden); | 763 GoogleUpdateSettings::GetAppUpdatePolicy(app_guid, &is_overridden); |
| 767 // Policy should now be to enable updates, |is_overridden| should still be | 764 // Policy should now be to enable updates, |is_overridden| should still be |
| 768 // false. | 765 // false. |
| 769 EXPECT_FALSE(is_overridden); | 766 EXPECT_FALSE(is_overridden); |
| 770 EXPECT_EQ(GoogleUpdateSettings::AUTOMATIC_UPDATES, update_policy); | 767 EXPECT_EQ(GoogleUpdateSettings::AUTOMATIC_UPDATES, update_policy); |
| 771 EXPECT_TRUE(GoogleUpdateSettings::AreAutoupdatesEnabled()); | 768 EXPECT_TRUE(GoogleUpdateSettings::AreAutoupdatesEnabled()); |
| 772 } | 769 } |
| 773 | 770 |
| 774 TEST_F(GoogleUpdateSettingsTest, UpdatesDisabledByTimeout) { | 771 TEST_F(GoogleUpdateSettingsTest, UpdatesDisabledByTimeout) { |
| 775 // Disable updates altogether. | 772 // Disable updates altogether. |
| 776 EXPECT_TRUE(SetUpdateTimeoutOverride(0)); | 773 EXPECT_TRUE(SetUpdateTimeoutOverride(0)); |
| (...skipping 405 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 1182 StatsState::TRUE_SETTING), | 1179 StatsState::TRUE_SETTING), |
| 1183 StatsState(StatsState::kSystemLevel, | 1180 StatsState(StatsState::kSystemLevel, |
| 1184 StatsState::TRUE_SETTING, | 1181 StatsState::TRUE_SETTING, |
| 1185 StatsState::NO_SETTING), | 1182 StatsState::NO_SETTING), |
| 1186 StatsState(StatsState::kSystemLevel, | 1183 StatsState(StatsState::kSystemLevel, |
| 1187 StatsState::TRUE_SETTING, | 1184 StatsState::TRUE_SETTING, |
| 1188 StatsState::FALSE_SETTING), | 1185 StatsState::FALSE_SETTING), |
| 1189 StatsState(StatsState::kSystemLevel, | 1186 StatsState(StatsState::kSystemLevel, |
| 1190 StatsState::TRUE_SETTING, | 1187 StatsState::TRUE_SETTING, |
| 1191 StatsState::TRUE_SETTING))); | 1188 StatsState::TRUE_SETTING))); |
| OLD | NEW |