| Index: chrome/browser/extensions/extension_service_unittest.cc
|
| diff --git a/chrome/browser/extensions/extension_service_unittest.cc b/chrome/browser/extensions/extension_service_unittest.cc
|
| index bbbff27cd49548bf7c9fbfc000b7342a6c604ee7..e138a2debd744bfb35ed94da73e717c0266af9db 100644
|
| --- a/chrome/browser/extensions/extension_service_unittest.cc
|
| +++ b/chrome/browser/extensions/extension_service_unittest.cc
|
| @@ -3021,10 +3021,14 @@ TEST_F(ExtensionServiceTest, AddPendingExtensionFromSync) {
|
| const std::string kFakeId(all_zero);
|
| const GURL kFakeUpdateURL("http:://fake.update/url");
|
| const bool kFakeInstallSilently(true);
|
| + const bool kFakeRemoteInstall(false);
|
|
|
| - EXPECT_TRUE(service_->pending_extension_manager()->AddFromSync(
|
| - kFakeId, kFakeUpdateURL, &IsExtension,
|
| - kFakeInstallSilently));
|
| + EXPECT_TRUE(
|
| + service_->pending_extension_manager()->AddFromSync(kFakeId,
|
| + kFakeUpdateURL,
|
| + &IsExtension,
|
| + kFakeInstallSilently,
|
| + kFakeRemoteInstall));
|
|
|
| const extensions::PendingExtensionInfo* pending_extension_info;
|
| ASSERT_TRUE((pending_extension_info = service_->pending_extension_manager()->
|
| @@ -3032,6 +3036,7 @@ TEST_F(ExtensionServiceTest, AddPendingExtensionFromSync) {
|
| EXPECT_EQ(kFakeUpdateURL, pending_extension_info->update_url());
|
| EXPECT_EQ(&IsExtension, pending_extension_info->should_allow_install_);
|
| EXPECT_EQ(kFakeInstallSilently, pending_extension_info->install_silently());
|
| + EXPECT_EQ(kFakeRemoteInstall, pending_extension_info->remote_install());
|
| }
|
|
|
| namespace {
|
| @@ -3039,14 +3044,18 @@ const char kGoodId[] = "ldnnhddmnhbkjipkidpdiheffobcpfmf";
|
| const char kGoodUpdateURL[] = "http://good.update/url";
|
| const bool kGoodIsFromSync = true;
|
| const bool kGoodInstallSilently = true;
|
| +const bool kGoodRemoteInstall = false;
|
| } // namespace
|
|
|
| // Test updating a pending extension.
|
| TEST_F(ExtensionServiceTest, UpdatePendingExtension) {
|
| InitializeEmptyExtensionService();
|
| - EXPECT_TRUE(service_->pending_extension_manager()->AddFromSync(
|
| - kGoodId, GURL(kGoodUpdateURL), &IsExtension,
|
| - kGoodInstallSilently));
|
| + EXPECT_TRUE(
|
| + service_->pending_extension_manager()->AddFromSync(kGoodId,
|
| + GURL(kGoodUpdateURL),
|
| + &IsExtension,
|
| + kGoodInstallSilently,
|
| + kGoodRemoteInstall));
|
| EXPECT_TRUE(service_->pending_extension_manager()->IsIdPending(kGoodId));
|
|
|
| base::FilePath path = data_dir_.AppendASCII("good.crx");
|
| @@ -3071,7 +3080,7 @@ bool IsTheme(const Extension* extension) {
|
| TEST_F(ExtensionServiceTest, DISABLED_UpdatePendingTheme) {
|
| InitializeEmptyExtensionService();
|
| EXPECT_TRUE(service_->pending_extension_manager()->AddFromSync(
|
| - theme_crx, GURL(), &IsTheme, false));
|
| + theme_crx, GURL(), &IsTheme, false, false));
|
| EXPECT_TRUE(service_->pending_extension_manager()->IsIdPending(theme_crx));
|
|
|
| base::FilePath path = data_dir_.AppendASCII("theme.crx");
|
| @@ -3130,9 +3139,12 @@ TEST_F(ExtensionServiceTest, UpdatePendingExternalCrxWinsOverSync) {
|
| InitializeEmptyExtensionService();
|
|
|
| // Add a crx to be installed from the update mechanism.
|
| - EXPECT_TRUE(service_->pending_extension_manager()->AddFromSync(
|
| - kGoodId, GURL(kGoodUpdateURL), &IsExtension,
|
| - kGoodInstallSilently));
|
| + EXPECT_TRUE(
|
| + service_->pending_extension_manager()->AddFromSync(kGoodId,
|
| + GURL(kGoodUpdateURL),
|
| + &IsExtension,
|
| + kGoodInstallSilently,
|
| + kGoodRemoteInstall));
|
|
|
| // Check that there is a pending crx, with is_from_sync set to true.
|
| const extensions::PendingExtensionInfo* pending_extension_info;
|
| @@ -3157,9 +3169,12 @@ TEST_F(ExtensionServiceTest, UpdatePendingExternalCrxWinsOverSync) {
|
| pending_extension_info->install_source());
|
|
|
| // Add a crx to be installed from the update mechanism.
|
| - EXPECT_FALSE(service_->pending_extension_manager()->AddFromSync(
|
| - kGoodId, GURL(kGoodUpdateURL), &IsExtension,
|
| - kGoodInstallSilently));
|
| + EXPECT_FALSE(
|
| + service_->pending_extension_manager()->AddFromSync(kGoodId,
|
| + GURL(kGoodUpdateURL),
|
| + &IsExtension,
|
| + kGoodInstallSilently,
|
| + kGoodRemoteInstall));
|
|
|
| // Check that the external, non-sync update was not overridden.
|
| ASSERT_TRUE((pending_extension_info = service_->pending_extension_manager()->
|
| @@ -3174,7 +3189,7 @@ TEST_F(ExtensionServiceTest, UpdatePendingExternalCrxWinsOverSync) {
|
| TEST_F(ExtensionServiceTest, UpdatePendingCrxThemeMismatch) {
|
| InitializeEmptyExtensionService();
|
| EXPECT_TRUE(service_->pending_extension_manager()->AddFromSync(
|
| - theme_crx, GURL(), &IsExtension, true));
|
| + theme_crx, GURL(), &IsExtension, true, false));
|
|
|
| EXPECT_TRUE(service_->pending_extension_manager()->IsIdPending(theme_crx));
|
|
|
| @@ -3195,8 +3210,12 @@ TEST_F(ExtensionServiceTest, UpdatePendingCrxThemeMismatch) {
|
| TEST_F(ExtensionServiceTest, UpdatePendingExtensionFailedShouldInstallTest) {
|
| InitializeEmptyExtensionService();
|
| // Add pending extension with a flipped is_theme.
|
| - EXPECT_TRUE(service_->pending_extension_manager()->AddFromSync(
|
| - kGoodId, GURL(kGoodUpdateURL), &IsTheme, kGoodInstallSilently));
|
| + EXPECT_TRUE(
|
| + service_->pending_extension_manager()->AddFromSync(kGoodId,
|
| + GURL(kGoodUpdateURL),
|
| + &IsTheme,
|
| + kGoodInstallSilently,
|
| + kGoodRemoteInstall));
|
| EXPECT_TRUE(service_->pending_extension_manager()->IsIdPending(kGoodId));
|
|
|
| base::FilePath path = data_dir_.AppendASCII("good.crx");
|
| @@ -3243,7 +3262,8 @@ TEST_F(ExtensionServiceTest, UpdatePendingExtensionAlreadyInstalled) {
|
| kGoodInstallSilently,
|
| Manifest::INTERNAL,
|
| Extension::NO_FLAGS,
|
| - false);
|
| + false,
|
| + kGoodRemoteInstall);
|
| UpdateExtension(good->id(), path, ENABLED);
|
|
|
| EXPECT_FALSE(service_->pending_extension_manager()->IsIdPending(kGoodId));
|
| @@ -5451,7 +5471,8 @@ TEST_F(ExtensionServiceTest, DisableExtensionFromSync) {
|
| const Extension* extension = service_->GetExtensionById(good0, true);
|
| ASSERT_TRUE(extension);
|
| ASSERT_TRUE(service_->IsExtensionEnabled(good0));
|
| - extensions::ExtensionSyncData disable_good_crx(*extension, false, false);
|
| + extensions::ExtensionSyncData disable_good_crx(
|
| + *extension, false, false, false);
|
|
|
| // Then sync data arrives telling us to disable |good0|.
|
| syncer::SyncDataList sync_data;
|
| @@ -5498,7 +5519,8 @@ TEST_F(ExtensionServiceTest, DontDisableExtensionWithPendingEnableFromSync) {
|
|
|
| // Now sync data comes in that says to disable good0. This should be
|
| // ignored.
|
| - extensions::ExtensionSyncData disable_good_crx(*extension, false, false);
|
| + extensions::ExtensionSyncData disable_good_crx(
|
| + *extension, false, false, false);
|
| syncer::SyncDataList sync_data;
|
| sync_data.push_back(disable_good_crx.GetSyncData());
|
| extension_sync_service_->MergeDataAndStartSyncing(
|
| @@ -6498,7 +6520,11 @@ class ExtensionSourcePriorityTest : public ExtensionServiceTest {
|
| // Fake a request from sync to install an extension.
|
| bool AddPendingSyncInstall() {
|
| return service_->pending_extension_manager()->AddFromSync(
|
| - crx_id_, GURL(kGoodUpdateURL), &IsExtension, kGoodInstallSilently);
|
| + crx_id_,
|
| + GURL(kGoodUpdateURL),
|
| + &IsExtension,
|
| + kGoodInstallSilently,
|
| + kGoodRemoteInstall);
|
| }
|
|
|
| // Fake a policy install.
|
|
|