// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
+#include "chrome/browser/apps/ephemeral_app_browsertest.h"
+
+#include <vector>
+
+#include "apps/app_restore_service.h"
#include "apps/saved_files_service.h"
#include "base/files/scoped_temp_dir.h"
+#include "base/scoped_observer.h"
#include "base/stl_util.h"
#include "chrome/browser/apps/app_browsertest_util.h"
#include "chrome/browser/apps/ephemeral_app_service.h"
#include "chrome/browser/extensions/api/file_system/file_system_api.h"
+#include "chrome/browser/extensions/app_sync_data.h"
#include "chrome/browser/extensions/extension_service.h"
-#include "chrome/browser/extensions/extension_test_message_listener.h"
+#include "chrome/browser/extensions/extension_sync_service.h"
#include "chrome/browser/extensions/extension_util.h"
#include "chrome/browser/notifications/desktop_notification_service.h"
#include "chrome/browser/notifications/desktop_notification_service_factory.h"
+#include "chrome/common/chrome_switches.h"
#include "chrome/common/extensions/api/alarms.h"
+#include "content/public/browser/power_save_blocker.h"
#include "content/public/test/browser_test.h"
#include "content/public/test/test_utils.h"
+#include "extensions/browser/api/power/power_api_manager.h"
+#include "extensions/browser/app_sorting.h"
#include "extensions/browser/event_router.h"
#include "extensions/browser/extension_prefs.h"
+#include "extensions/browser/extension_registry.h"
+#include "extensions/browser/extension_registry_observer.h"
#include "extensions/browser/extension_system.h"
+#include "extensions/browser/extension_util.h"
+#include "extensions/browser/notification_types.h"
#include "extensions/browser/process_manager.h"
+#include "extensions/browser/uninstall_reason.h"
+#include "extensions/common/extension.h"
#include "extensions/common/switches.h"
+#include "extensions/test/extension_test_message_listener.h"
+#include "extensions/test/result_catcher.h"
+#include "sync/api/fake_sync_change_processor.h"
+#include "sync/api/sync_change_processor_wrapper_for_test.h"
+#include "sync/api/sync_error_factory_mock.h"
#include "ui/message_center/message_center.h"
#include "ui/message_center/notifier_settings.h"
+using extensions::AppSyncData;
using extensions::Event;
using extensions::EventRouter;
using extensions::Extension;
-using extensions::ExtensionInfo;
using extensions::ExtensionPrefs;
+using extensions::ExtensionRegistry;
+using extensions::ExtensionRegistryObserver;
using extensions::ExtensionSystem;
using extensions::Manifest;
-using extensions::PlatformAppBrowserTest;
+using extensions::ResultCatcher;
namespace {
namespace alarms = extensions::api::alarms;
-const char kDispatchEventTestApp[] = "ephemeral_apps/dispatch_event";
-const char kMessagingReceiverApp[] = "ephemeral_apps/messaging_receiver";
-const char kMessagingReceiverAppV2[] = "ephemeral_apps/messaging_receiver2";
-const char kNotificationsTestApp[] = "ephemeral_apps/notification_settings";
-const char kFileSystemTestApp[] = "ephemeral_apps/filesystem_retain_entries";
-const char kRetainDataApp[] = "ephemeral_apps/retain_data";
+const char kPowerTestApp[] = "ephemeral_apps/power";
+
+// Enabling sync causes these tests to be flaky on Windows. Disable sync so that
+// everything else can be tested. See crbug.com/401028
+#if defined(OS_WIN)
+const bool kEnableSync = false;
+#else
+const bool kEnableSync = true;
+#endif
typedef std::vector<message_center::Notifier*> NotifierList;
return false;
}
-bool IsAppInExtensionsInfo(const ExtensionPrefs::ExtensionsInfo& ext_info,
- const std::string& extension_id) {
- for (size_t i = 0; i < ext_info.size(); ++i) {
- ExtensionInfo* info = ext_info.at(i).get();
- if (info->extension_id == extension_id)
- return true;
+// Saves some parameters from the extension installed notification in order
+// to verify them in tests.
+class InstallObserver : public ExtensionRegistryObserver {
+ public:
+ struct InstallParameters {
+ std::string id;
+ bool is_update;
+ bool from_ephemeral;
+
+ InstallParameters(
+ const std::string& id,
+ bool is_update,
+ bool from_ephemeral)
+ : id(id), is_update(is_update), from_ephemeral(from_ephemeral) {}
+ };
+
+ explicit InstallObserver(Profile* profile) : registry_observer_(this) {
+ registry_observer_.Add(ExtensionRegistry::Get(profile));
}
- return false;
-}
+ virtual ~InstallObserver() {}
-} // namespace
+ const InstallParameters& Last() {
+ CHECK(!install_params_.empty());
+ return install_params_.back();
+ }
-class EphemeralAppBrowserTest : public PlatformAppBrowserTest {
- protected:
- virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
- // Skip PlatformAppBrowserTest, which sets different values for the switches
- // below.
- ExtensionBrowserTest::SetUpCommandLine(command_line);
-
- // Make event pages get suspended immediately.
- command_line->AppendSwitchASCII(
- extensions::switches::kEventPageIdleTime, "10");
- command_line->AppendSwitchASCII(
- extensions::switches::kEventPageSuspendingTime, "10");
+ private:
+ virtual void OnExtensionWillBeInstalled(
+ content::BrowserContext* browser_context,
+ const Extension* extension,
+ bool is_update,
+ bool from_ephemeral,
+ const std::string& old_name) OVERRIDE {
+ install_params_.push_back(
+ InstallParameters(extension->id(), is_update, from_ephemeral));
}
- base::FilePath GetTestPath(const char* test_path) {
- return test_data_dir_.AppendASCII("platform_apps").AppendASCII(test_path);
+ std::vector<InstallParameters> install_params_;
+ ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver>
+ registry_observer_;
+};
+
+// Instead of actually changing the system power settings, tests will just
+// issue requests to this mock.
+class PowerSettingsMock {
+ public:
+ PowerSettingsMock() : keep_awake_count_(0) {}
+
+ void request_keep_awake() { ++keep_awake_count_; }
+
+ void release_keep_awake() {
+ --keep_awake_count_;
+ ASSERT_GE(keep_awake_count_, 0);
}
- const Extension* InstallEphemeralApp(const char* test_path,
- Manifest::Location manifest_location) {
- const Extension* extension =
- InstallExtensionWithSourceAndFlags(
- GetTestPath(test_path),
- 1,
- manifest_location,
- Extension::IS_EPHEMERAL);
- return extension;
+ int keep_awake_count() const { return keep_awake_count_; }
+
+ private:
+ int keep_awake_count_;
+
+ DISALLOW_COPY_AND_ASSIGN(PowerSettingsMock);
+};
+
+// Stub implementation of content::PowerSaveBlocker that updates the
+// PowerSettingsMock.
+class PowerSaveBlockerStub : public content::PowerSaveBlocker {
+ public:
+ explicit PowerSaveBlockerStub(PowerSettingsMock* power_settings)
+ : power_settings_(power_settings) {
+ power_settings_->request_keep_awake();
}
- const Extension* InstallEphemeralApp(const char* test_path) {
- return InstallEphemeralApp(test_path, Manifest::INTERNAL);
+ virtual ~PowerSaveBlockerStub() { power_settings_->release_keep_awake(); }
+
+ static scoped_ptr<PowerSaveBlocker> Create(PowerSettingsMock* power_settings,
+ PowerSaveBlockerType type,
+ const std::string& reason) {
+ return scoped_ptr<PowerSaveBlocker>(
+ new PowerSaveBlockerStub(power_settings));
}
- const Extension* InstallAndLaunchEphemeralApp(const char* test_path) {
- ExtensionTestMessageListener launched_listener("launched", false);
- const Extension* extension = InstallEphemeralApp(test_path);
- EXPECT_TRUE(extension);
- if (!extension)
- return NULL;
+ private:
+ PowerSettingsMock* power_settings_; // Not owned.
+
+ DISALLOW_COPY_AND_ASSIGN(PowerSaveBlockerStub);
+};
+
+} // namespace
+
+
+// EphemeralAppTestBase:
+
+const char EphemeralAppTestBase::kMessagingReceiverApp[] =
+ "ephemeral_apps/messaging_receiver";
+const char EphemeralAppTestBase::kMessagingReceiverAppV2[] =
+ "ephemeral_apps/messaging_receiver2";
+const char EphemeralAppTestBase::kDispatchEventTestApp[] =
+ "ephemeral_apps/dispatch_event";
+const char EphemeralAppTestBase::kNotificationsTestApp[] =
+ "ephemeral_apps/notification_settings";
+const char EphemeralAppTestBase::kFileSystemTestApp[] =
+ "ephemeral_apps/filesystem_retain_entries";
+
+EphemeralAppTestBase::EphemeralAppTestBase() {}
- LaunchPlatformApp(extension);
- bool wait_result = launched_listener.WaitUntilSatisfied();
- EXPECT_TRUE(wait_result);
- if (!wait_result)
- return NULL;
+EphemeralAppTestBase::~EphemeralAppTestBase() {}
- return extension;
+void EphemeralAppTestBase::SetUpCommandLine(base::CommandLine* command_line) {
+ // Skip PlatformAppBrowserTest, which sets different values for the switches
+ // below.
+ ExtensionBrowserTest::SetUpCommandLine(command_line);
+
+ // Make event pages get suspended immediately.
+ extensions::ProcessManager::SetEventPageIdleTimeForTesting(1);
+ extensions::ProcessManager::SetEventPageSuspendingTimeForTesting(1);
+
+ // Enable ephemeral apps flag.
+ command_line->AppendSwitch(switches::kEnableEphemeralApps);
+}
+
+void EphemeralAppTestBase::SetUpOnMainThread() {
+ PlatformAppBrowserTest::SetUpOnMainThread();
+
+ // Disable ephemeral apps immediately after they stop running in tests.
+ EphemeralAppService::Get(profile())->set_disable_delay_for_test(0);
+}
+
+base::FilePath EphemeralAppTestBase::GetTestPath(const char* test_path) {
+ return test_data_dir_.AppendASCII("platform_apps").AppendASCII(test_path);
+}
+
+const Extension* EphemeralAppTestBase::InstallEphemeralApp(
+ const char* test_path, Manifest::Location manifest_location) {
+ const Extension* extension = InstallEphemeralAppWithSourceAndFlags(
+ GetTestPath(test_path), 1, manifest_location, Extension::NO_FLAGS);
+ EXPECT_TRUE(extension);
+ if (extension)
+ EXPECT_TRUE(extensions::util::IsEphemeralApp(extension->id(), profile()));
+ return extension;
+}
+
+const Extension* EphemeralAppTestBase::InstallEphemeralApp(
+ const char* test_path) {
+ return InstallEphemeralApp(test_path, Manifest::INTERNAL);
+}
+
+const Extension* EphemeralAppTestBase::InstallAndLaunchEphemeralApp(
+ const char* test_path) {
+ ExtensionTestMessageListener launched_listener("launched", false);
+ const Extension* extension = InstallEphemeralApp(test_path);
+ EXPECT_TRUE(extension);
+ if (!extension)
+ return NULL;
+
+ LaunchPlatformApp(extension);
+ bool wait_result = launched_listener.WaitUntilSatisfied();
+ EXPECT_TRUE(wait_result);
+ if (!wait_result)
+ return NULL;
+
+ return extension;
+}
+
+const Extension* EphemeralAppTestBase::UpdateEphemeralApp(
+ const std::string& app_id,
+ const base::FilePath& test_dir,
+ const base::FilePath& pem_path) {
+ // Pack a new version of the app.
+ base::ScopedTempDir temp_dir;
+ EXPECT_TRUE(temp_dir.CreateUniqueTempDir());
+
+ base::FilePath crx_path = temp_dir.path().AppendASCII("temp.crx");
+ if (!base::DeleteFile(crx_path, false)) {
+ ADD_FAILURE() << "Failed to delete existing crx: " << crx_path.value();
+ return NULL;
}
+ base::FilePath app_v2_path = PackExtensionWithOptions(
+ test_dir, crx_path, pem_path, base::FilePath());
+ EXPECT_FALSE(app_v2_path.empty());
+
+ // Update the ephemeral app and wait for the update to finish.
+ extensions::CrxInstaller* crx_installer = NULL;
+ content::WindowedNotificationObserver windowed_observer(
+ extensions::NOTIFICATION_CRX_INSTALLER_DONE,
+ content::Source<extensions::CrxInstaller>(crx_installer));
+ ExtensionService* service =
+ ExtensionSystem::Get(profile())->extension_service();
+ EXPECT_TRUE(service->UpdateExtension(app_id, app_v2_path, true,
+ &crx_installer));
+ windowed_observer.Wait();
+
+ return ExtensionRegistry::Get(profile())
+ ->GetExtensionById(app_id, ExtensionRegistry::EVERYTHING);
+}
+
+void EphemeralAppTestBase::PromoteEphemeralApp(
+ const extensions::Extension* app) {
+ ExtensionService* extension_service =
+ ExtensionSystem::Get(profile())->extension_service();
+ ASSERT_TRUE(extension_service);
+ extension_service->PromoteEphemeralApp(app, false);
+}
+
+void EphemeralAppTestBase::DisableEphemeralApp(
+ const Extension* app,
+ Extension::DisableReason disable_reason) {
+ ExtensionPrefs* prefs = ExtensionPrefs::Get(profile());
+
+ // Disabling due to a permissions increase also involves setting the
+ // DidExtensionEscalatePermissions flag.
+ if (disable_reason == Extension::DISABLE_PERMISSIONS_INCREASE)
+ prefs->SetDidExtensionEscalatePermissions(app, true);
+
+ ExtensionSystem::Get(profile())->extension_service()->DisableExtension(
+ app->id(), disable_reason);
+
+ ASSERT_TRUE(ExtensionRegistry::Get(profile())->disabled_extensions().Contains(
+ app->id()));
+}
+
+void EphemeralAppTestBase::CloseApp(const std::string& app_id) {
+ EXPECT_EQ(1U, GetAppWindowCountForApp(app_id));
+ extensions::AppWindow* app_window = GetFirstAppWindowForApp(app_id);
+ ASSERT_TRUE(app_window);
+ CloseAppWindow(app_window);
+}
+
+void EphemeralAppTestBase::CloseAppWaitForUnload(const std::string& app_id) {
+ // Ephemeral apps are unloaded from extension system after they stop running.
+ content::WindowedNotificationObserver unloaded_signal(
+ extensions::NOTIFICATION_EXTENSION_UNLOADED_DEPRECATED,
+ content::Source<Profile>(profile()));
+ CloseApp(app_id);
+ unloaded_signal.Wait();
+}
+
+void EphemeralAppTestBase::EvictApp(const std::string& app_id) {
+ // Uninstall the app, which is what happens when ephemeral apps get evicted
+ // from the cache.
+ content::WindowedNotificationObserver uninstalled_signal(
+ extensions::NOTIFICATION_EXTENSION_UNINSTALLED_DEPRECATED,
+ content::Source<Profile>(profile()));
+
+ ExtensionService* service =
+ ExtensionSystem::Get(profile())->extension_service();
+ ASSERT_TRUE(service);
+ service->UninstallExtension(
+ app_id,
+ extensions::UNINSTALL_REASON_ORPHANED_EPHEMERAL_EXTENSION,
+ base::Bind(&base::DoNothing),
+ NULL);
+
+ uninstalled_signal.Wait();
+}
+
+// EphemeralAppBrowserTest:
+
+class EphemeralAppBrowserTest : public EphemeralAppTestBase {
+ protected:
bool LaunchAppAndRunTest(const Extension* app, const char* test_name) {
+ // Ephemeral apps are unloaded after they are closed. Ensure they are
+ // enabled before launch.
+ ExtensionService* service =
+ ExtensionSystem::Get(profile())->extension_service();
+ service->EnableExtension(app->id());
+
ExtensionTestMessageListener launched_listener("launched", true);
LaunchPlatformApp(app);
if (!launched_listener.WaitUntilSatisfied()) {
return false;
}
+ ResultCatcher catcher;
launched_listener.Reply(test_name);
- ResultCatcher catcher;
- if (!catcher.GetNextResult()) {
- message_ = catcher.message();
- return false;
- }
+ bool result = catcher.GetNextResult();
+ message_ = catcher.message();
- CloseApp(app->id());
- return true;
+ CloseAppWaitForUnload(app->id());
+ return result;
}
- void CloseApp(const std::string& app_id) {
- content::WindowedNotificationObserver event_page_destroyed_signal(
- chrome::NOTIFICATION_EXTENSION_HOST_DESTROYED,
- content::Source<Profile>(browser()->profile()));
-
- EXPECT_EQ(1U, GetAppWindowCountForApp(app_id));
- apps::AppWindow* app_window = GetFirstAppWindowForApp(app_id);
- ASSERT_TRUE(app_window);
- CloseAppWindow(app_window);
-
- event_page_destroyed_signal.Wait();
+ // Verify that the event page of the app has not been loaded.
+ void VerifyAppNotLoaded(const std::string& app_id) {
+ EXPECT_FALSE(ExtensionSystem::Get(profile())->
+ process_manager()->GetBackgroundHostForExtension(app_id));
}
- void EvictApp(const std::string& app_id) {
- // Uninstall the app, which is what happens when ephemeral apps get evicted
- // from the cache.
- content::WindowedNotificationObserver uninstalled_signal(
- chrome::NOTIFICATION_EXTENSION_UNINSTALLED,
- content::Source<Profile>(browser()->profile()));
+ // Verify properties of ephemeral apps.
+ void VerifyEphemeralApp(const std::string& app_id) {
+ EXPECT_TRUE(extensions::util::IsEphemeralApp(app_id, profile()));
- ExtensionService* service =
- ExtensionSystem::Get(browser()->profile())->extension_service();
- ASSERT_TRUE(service);
- service->UninstallExtension(app_id, false, NULL);
+ // Ephemeral apps should not be synced.
+ scoped_ptr<AppSyncData> sync_change = GetLastSyncChangeForApp(app_id);
+ EXPECT_FALSE(sync_change.get());
- uninstalled_signal.Wait();
+ // Ephemeral apps should not be assigned ordinals.
+ extensions::AppSorting* app_sorting =
+ ExtensionPrefs::Get(profile())->app_sorting();
+ EXPECT_FALSE(app_sorting->GetAppLaunchOrdinal(app_id).IsValid());
+ EXPECT_FALSE(app_sorting->GetPageOrdinal(app_id).IsValid());
}
- void VerifyAppNotLoaded(const std::string& app_id) {
- EXPECT_FALSE(ExtensionSystem::Get(browser()->profile())->
- process_manager()->GetBackgroundHostForExtension(app_id));
+ // Verify that after ephemeral apps stop running, they reside in extension
+ // system in a disabled and unloaded state.
+ void VerifyInactiveEphemeralApp(const std::string& app_id) {
+ EXPECT_TRUE(
+ ExtensionRegistry::Get(profile())->disabled_extensions().Contains(
+ app_id));
+
+ ExtensionPrefs* prefs = ExtensionPrefs::Get(profile());
+ EXPECT_TRUE(prefs->IsExtensionDisabled(app_id));
+ EXPECT_NE(0,
+ prefs->GetDisableReasons(app_id) &
+ Extension::DISABLE_INACTIVE_EPHEMERAL_APP);
+ }
+
+ // Verify the state of an app that has been promoted from an ephemeral to a
+ // fully installed app.
+ void VerifyPromotedApp(const std::string& app_id,
+ ExtensionRegistry::IncludeFlag expected_set) {
+ const Extension* app = ExtensionRegistry::Get(profile())
+ ->GetExtensionById(app_id, expected_set);
+ ASSERT_TRUE(app) << "App not found in expected set: " << expected_set;
+
+ // The app should not be ephemeral.
+ ExtensionPrefs* prefs = ExtensionPrefs::Get(profile());
+ ASSERT_TRUE(prefs);
+ EXPECT_FALSE(prefs->IsEphemeralApp(app_id));
+ EXPECT_EQ(0,
+ prefs->GetDisableReasons(app_id) &
+ Extension::DISABLE_INACTIVE_EPHEMERAL_APP);
+
+ // Check sort ordinals.
+ extensions::AppSorting* app_sorting = prefs->app_sorting();
+ EXPECT_TRUE(app_sorting->GetAppLaunchOrdinal(app_id).IsValid());
+ EXPECT_TRUE(app_sorting->GetPageOrdinal(app_id).IsValid());
}
+ // Dispatch a fake alarm event to the app.
void DispatchAlarmEvent(EventRouter* event_router,
const std::string& app_id) {
alarms::Alarm dummy_alarm;
event_router->DispatchEventToExtension(app_id, event.Pass());
}
- void GarbageCollectData() {
- EphemeralAppService* service =
- EphemeralAppService::Get(browser()->profile());
- ASSERT_TRUE(service);
- service->GarbageCollectData();
+ // Simulates the scenario where an app is installed, via the normal
+ // installation route, on top of an ephemeral app. This can occur due to race
+ // conditions.
+ const Extension* ReplaceEphemeralApp(const std::string& app_id,
+ const char* test_path,
+ int expected_enabled_change) {
+ return UpdateExtensionWaitForIdle(
+ app_id, GetTestPath(test_path), expected_enabled_change);
+ }
+
+ void PromoteEphemeralAppAndVerify(
+ const Extension* app,
+ ExtensionRegistry::IncludeFlag expected_set) {
+ ASSERT_TRUE(app);
+
+ // Ephemeral apps should not be synced.
+ scoped_ptr<AppSyncData> sync_change = GetLastSyncChangeForApp(app->id());
+ EXPECT_FALSE(sync_change.get());
+
+ // Promote the app to a regular installed app.
+ InstallObserver installed_observer(profile());
+ PromoteEphemeralApp(app);
+ VerifyPromotedApp(app->id(), expected_set);
+
+ // Check the notification parameters.
+ const InstallObserver::InstallParameters& params =
+ installed_observer.Last();
+ EXPECT_EQ(app->id(), params.id);
+ EXPECT_TRUE(params.is_update);
+ EXPECT_TRUE(params.from_ephemeral);
+
+ // The installation should now be synced.
+ sync_change = GetLastSyncChangeForApp(app->id());
+ VerifySyncChange(sync_change.get(),
+ expected_set == ExtensionRegistry::ENABLED);
+ }
+
+ void PromoteEphemeralAppFromSyncAndVerify(
+ const Extension* app,
+ bool enable_from_sync,
+ ExtensionRegistry::IncludeFlag expected_set) {
+ ASSERT_TRUE(app);
+
+ // Simulate an install from sync.
+ const syncer::StringOrdinal kAppLaunchOrdinal("x");
+ const syncer::StringOrdinal kPageOrdinal("y");
+ AppSyncData app_sync_data(*app,
+ enable_from_sync,
+ false /* incognito enabled */,
+ false /* remote install */,
+ kAppLaunchOrdinal,
+ kPageOrdinal,
+ extensions::LAUNCH_TYPE_REGULAR);
+
+ std::string app_id = app->id();
+ app = NULL;
+
+ ExtensionSyncService* sync_service = ExtensionSyncService::Get(profile());
+ sync_service->ProcessAppSyncData(app_sync_data);
+
+ // Verify the installation.
+ VerifyPromotedApp(app_id, expected_set);
+
+ // The sort ordinals from sync should not be overridden.
+ ExtensionPrefs* prefs = ExtensionPrefs::Get(profile());
+ extensions::AppSorting* app_sorting = prefs->app_sorting();
+ EXPECT_TRUE(
+ app_sorting->GetAppLaunchOrdinal(app_id).Equals(kAppLaunchOrdinal));
+ EXPECT_TRUE(app_sorting->GetPageOrdinal(app_id).Equals(kPageOrdinal));
+ }
+
+ void InitSyncService() {
+ if (!kEnableSync)
+ return;
+
+ ExtensionSyncService* sync_service = ExtensionSyncService::Get(profile());
+ sync_service->MergeDataAndStartSyncing(
+ syncer::APPS,
+ syncer::SyncDataList(),
+ scoped_ptr<syncer::SyncChangeProcessor>(
+ new syncer::SyncChangeProcessorWrapperForTest(
+ &mock_sync_processor_)),
+ scoped_ptr<syncer::SyncErrorFactory>(
+ new syncer::SyncErrorFactoryMock()));
+ }
+
+ scoped_ptr<AppSyncData> GetLastSyncChangeForApp(const std::string& id) {
+ scoped_ptr<AppSyncData> sync_data;
+ for (syncer::SyncChangeList::iterator it =
+ mock_sync_processor_.changes().begin();
+ it != mock_sync_processor_.changes().end(); ++it) {
+ scoped_ptr<AppSyncData> data(new AppSyncData(*it));
+ if (data->id() == id)
+ sync_data.reset(data.release());
+ }
+
+ return sync_data.Pass();
+ }
+
+ void VerifySyncChange(const AppSyncData* sync_change, bool expect_enabled) {
+ if (!kEnableSync)
+ return;
+
+ ASSERT_TRUE(sync_change);
+ EXPECT_TRUE(sync_change->page_ordinal().IsValid());
+ EXPECT_TRUE(sync_change->app_launch_ordinal().IsValid());
+ EXPECT_FALSE(sync_change->uninstalled());
+ EXPECT_EQ(expect_enabled, sync_change->extension_sync_data().enabled());
+ }
+
+ void TestInstallEvent(bool close_app) {
+ ExtensionTestMessageListener first_msg_listener(false);
+ const Extension* app = InstallAndLaunchEphemeralApp(kDispatchEventTestApp);
+ ASSERT_TRUE(app);
+
+ // When an ephemeral app is first added, it should not receive the
+ // onInstalled event, hence the first message received from the test should
+ // be "launched" and not "installed".
+ ASSERT_TRUE(first_msg_listener.WaitUntilSatisfied());
+ EXPECT_EQ(std::string("launched"), first_msg_listener.message());
+
+ if (close_app)
+ CloseAppWaitForUnload(app->id());
+
+ // When installed permanently, the app should receive the onInstalled event.
+ ExtensionTestMessageListener install_listener("installed", false);
+ PromoteEphemeralApp(app);
+ ASSERT_TRUE(install_listener.WaitUntilSatisfied());
}
+
+ private:
+ syncer::FakeSyncChangeProcessor mock_sync_processor_;
};
// Verify that ephemeral apps can be launched and receive system events when
// Send a fake alarm event to the app and verify that a response is
// received.
- EventRouter* event_router =
- ExtensionSystem::Get(browser()->profile())->event_router();
+ EventRouter* event_router = EventRouter::Get(profile());
ASSERT_TRUE(event_router);
ExtensionTestMessageListener alarm_received_listener("alarm_received", false);
DispatchAlarmEvent(event_router, extension->id());
ASSERT_TRUE(alarm_received_listener.WaitUntilSatisfied());
- CloseApp(extension->id());
-
- // The app needs to be launched once in order to have the onAlarm() event
- // registered.
- ASSERT_TRUE(event_router->ExtensionHasEventListener(
- extension->id(), alarms::OnAlarm::kEventName));
+ CloseAppWaitForUnload(extension->id());
// Dispatch the alarm event again and verify that the event page did not get
// loaded for the app.
}
// Verify that ephemeral apps will receive messages while they are running.
-IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest, ReceiveMessagesWhenLaunched) {
+// Flaky test: crbug.com/394426
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest,
+ DISABLED_ReceiveMessagesWhenLaunched) {
const Extension* receiver =
InstallAndLaunchEphemeralApp(kMessagingReceiverApp);
ASSERT_TRUE(receiver);
// Verify that messages are received while the app is running.
- ExtensionApiTest::ResultCatcher result_catcher;
- LoadAndLaunchPlatformApp("ephemeral_apps/messaging_sender_success");
+ ResultCatcher result_catcher;
+ LoadAndLaunchPlatformApp("ephemeral_apps/messaging_sender_success",
+ "Launched");
EXPECT_TRUE(result_catcher.GetNextResult());
- CloseApp(receiver->id());
+ CloseAppWaitForUnload(receiver->id());
// Verify that messages are not received while the app is inactive.
- LoadAndLaunchPlatformApp("ephemeral_apps/messaging_sender_fail");
+ LoadAndLaunchPlatformApp("ephemeral_apps/messaging_sender_fail", "Launched");
EXPECT_TRUE(result_catcher.GetNextResult());
}
+// Verifies that the chrome.runtime.onInstalled() event is received by a running
+// ephemeral app only when it is promoted.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest,
+ InstallEventReceivedWhileRunning) {
+ TestInstallEvent(false /* close app */);
+}
+
+// Verifies that when an idle ephemeral app is promoted, it will be loaded to
+// receive the chrome.runtime.onInstalled() event.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest, InstallEventReceivedWhileIdle) {
+ TestInstallEvent(true /* close app */);
+}
+
+// Verifies that the chrome.runtime.onRestarted() event is received by an
+// ephemeral app.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest, RestartEventReceived) {
+ const Extension* app = InstallAndLaunchEphemeralApp(kDispatchEventTestApp);
+ ASSERT_TRUE(app);
+ CloseAppWaitForUnload(app->id());
+
+ // Fake ephemeral app running before restart.
+ ExtensionSystem::Get(profile())->extension_service()->EnableExtension(
+ app->id());
+ ASSERT_TRUE(ExtensionRegistry::Get(profile())->enabled_extensions().Contains(
+ app->id()));
+ ExtensionPrefs::Get(profile())->SetExtensionRunning(app->id(), true);
+
+ ExtensionTestMessageListener restart_listener("restarted", false);
+ apps::AppRestoreService::Get(profile())->HandleStartup(true);
+ EXPECT_TRUE(restart_listener.WaitUntilSatisfied());
+}
+
// Verify that an updated ephemeral app will still have its ephemeral flag
// enabled.
IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest, UpdateEphemeralApp) {
- const Extension* app_v1 = InstallEphemeralApp(kMessagingReceiverApp);
+ InitSyncService();
+
+ const Extension* app_v1 = InstallAndLaunchEphemeralApp(kMessagingReceiverApp);
ASSERT_TRUE(app_v1);
- ASSERT_TRUE(app_v1->is_ephemeral());
+ VerifyEphemeralApp(app_v1->id());
+ CloseAppWaitForUnload(app_v1->id());
+ VerifyInactiveEphemeralApp(app_v1->id());
+
std::string app_id = app_v1->id();
base::Version app_original_version = *app_v1->version();
- app_v1 = NULL; // The extension object will be destroyed during update.
-
- // Pack version 2 of the app.
- base::ScopedTempDir temp_dir;
- ASSERT_TRUE(temp_dir.CreateUniqueTempDir());
-
- base::FilePath crx_path = temp_dir.path().AppendASCII("temp.crx");
- if (!base::DeleteFile(crx_path, false)) {
- ADD_FAILURE() << "Failed to delete crx: " << crx_path.value();
- return;
- }
-
- base::FilePath app_v2_path = PackExtensionWithOptions(
- GetTestPath(kMessagingReceiverAppV2),
- crx_path,
- GetTestPath(kMessagingReceiverApp).ReplaceExtension(
- FILE_PATH_LITERAL(".pem")),
- base::FilePath());
- ASSERT_FALSE(app_v2_path.empty());
- // Update the ephemeral app and wait for the update to finish.
- extensions::CrxInstaller* crx_installer = NULL;
- content::WindowedNotificationObserver windowed_observer(
- chrome::NOTIFICATION_CRX_INSTALLER_DONE,
- content::Source<extensions::CrxInstaller>(crx_installer));
- ExtensionService* service =
- ExtensionSystem::Get(browser()->profile())->extension_service();
- EXPECT_TRUE(service->UpdateExtension(app_id, app_v2_path, true, GURL(),
- &crx_installer));
- windowed_observer.Wait();
-
- const Extension* app_v2 = service->GetExtensionById(app_id, false);
+ // Update to version 2 of the app.
+ app_v1 = NULL; // The extension object will be destroyed during update.
+ InstallObserver installed_observer(profile());
+ const Extension* app_v2 =
+ UpdateEphemeralApp(app_id,
+ GetTestPath(kMessagingReceiverAppV2),
+ GetTestPath(kMessagingReceiverApp)
+ .ReplaceExtension(FILE_PATH_LITERAL(".pem")));
+
+ // Check the notification parameters.
+ const InstallObserver::InstallParameters& params = installed_observer.Last();
+ EXPECT_EQ(app_id, params.id);
+ EXPECT_TRUE(params.is_update);
+ EXPECT_FALSE(params.from_ephemeral);
+
+ // The ephemeral flag should still be set.
ASSERT_TRUE(app_v2);
- EXPECT_TRUE(app_v2->version()->CompareTo(app_original_version) > 0);
- EXPECT_TRUE(app_v2->is_ephemeral());
+ EXPECT_GT(app_v2->version()->CompareTo(app_original_version), 0);
+ VerifyEphemeralApp(app_id);
+
+ // The app should still be disabled in extension system.
+ VerifyInactiveEphemeralApp(app_id);
}
// Verify that if notifications have been disabled for an ephemeral app, it will
// remain disabled even after being evicted from the cache.
IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest, StickyNotificationSettings) {
- const Extension* app = InstallEphemeralApp(kNotificationsTestApp);
+ const Extension* app = InstallAndLaunchEphemeralApp(kNotificationsTestApp);
ASSERT_TRUE(app);
// Disable notifications for this app.
DesktopNotificationService* notification_service =
- DesktopNotificationServiceFactory::GetForProfile(browser()->profile());
+ DesktopNotificationServiceFactory::GetForProfile(profile());
ASSERT_TRUE(notification_service);
message_center::NotifierId notifier_id(
EXPECT_FALSE(notification_service->IsNotifierEnabled(notifier_id));
// Remove the app.
+ CloseAppWaitForUnload(app->id());
EvictApp(app->id());
// Reinstall the ephemeral app and verify that notifications remain disabled.
IncludeRunningEphemeralAppsInNotifiers) {
message_center::NotifierSettingsProvider* settings_provider =
message_center::MessageCenter::Get()->GetNotifierSettingsProvider();
- // TODO(tmdiep): Remove once notifications settings are supported across
- // all platforms. This test will fail for Linux GTK.
- if (!settings_provider)
- return;
+ DCHECK(settings_provider);
const Extension* app = InstallAndLaunchEphemeralApp(kNotificationsTestApp);
ASSERT_TRUE(app);
STLDeleteElements(¬ifiers);
// Close the ephemeral app.
- CloseApp(app->id());
+ CloseAppWaitForUnload(app->id());
// Inactive ephemeral apps should not be included in the list of notifiers to
// show in the UI.
// Verify that after the app has been closed, all retained entries are
// flushed.
std::vector<apps::SavedFileEntry> file_entries =
- apps::SavedFilesService::Get(browser()->profile())
+ apps::SavedFilesService::Get(profile())
->GetAllFileEntries(app->id());
EXPECT_TRUE(file_entries.empty());
ASSERT_TRUE(LaunchAppAndRunTest(app, "RestoreRetainedFile")) << message_;
}
-// Verify that once evicted from the cache, the data of ephemeral apps will not
-// be deleted.
-IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest, RetainData) {
- // Phase 1 - Install the ephemeral app and write data to various storage.
- const Extension* app = InstallEphemeralApp(kRetainDataApp);
+// Checks the process of launching an ephemeral app and then promoting the app
+// while it is running.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest, PromoteAppWhileRunning) {
+ InitSyncService();
+
+ const Extension* app = InstallAndLaunchEphemeralApp(kNotificationsTestApp);
ASSERT_TRUE(app);
- ASSERT_TRUE(LaunchAppAndRunTest(app, "WriteData")) << message_;
- // Sanity check to ensure that the ReadData tests should pass before the app
- // is removed.
- ASSERT_TRUE(LaunchAppAndRunTest(app, "ReadData")) << message_;
+ PromoteEphemeralAppAndVerify(app, ExtensionRegistry::ENABLED);
- // Remove the app.
- const std::string app_id = app->id();
- EvictApp(app->id());
- app = NULL;
+ // Ensure that the app is not unloaded and disabled after it is closed.
+ CloseApp(app->id());
+ VerifyPromotedApp(app->id(), ExtensionRegistry::ENABLED);
+}
- // The app should be in the list of evicted apps.
- ExtensionPrefs* prefs = ExtensionPrefs::Get(browser()->profile());
- ASSERT_TRUE(prefs);
- scoped_ptr<ExtensionPrefs::ExtensionsInfo> extensions_info(
- prefs->GetEvictedEphemeralAppsInfo());
- EXPECT_TRUE(IsAppInExtensionsInfo(*extensions_info, app_id));
-
- // The app should not be in the list of installed extensions.
- extensions_info = prefs->GetInstalledExtensionsInfo();
- EXPECT_FALSE(IsAppInExtensionsInfo(*extensions_info, app_id));
-
- // Ensure the evicted app is considered to have isolated storage. This will
- // prevent its data from getting garbage collected by
- // ExtensionService::GarbageCollectIsolatedStorage().
- GURL site_url = extensions::util::GetSiteForExtensionId(
- app_id, browser()->profile());
- EXPECT_TRUE(extensions::util::SiteHasIsolatedStorage(
- site_url, browser()->profile()));
-
- // Phase 2 - Reinstall the ephemeral app and verify that data still exists
- // in the storage.
- app = InstallEphemeralApp(kRetainDataApp);
+// Checks the process of launching an ephemeral app and then promoting the app
+// while it is idle.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest, PromoteAppWhileIdle) {
+ InitSyncService();
+
+ const Extension* app = InstallAndLaunchEphemeralApp(kNotificationsTestApp);
ASSERT_TRUE(app);
- EXPECT_TRUE(LaunchAppAndRunTest(app, "ReadData")) << message_;
+ CloseAppWaitForUnload(app->id());
+ VerifyInactiveEphemeralApp(app->id());
- // The app should now be in the list of installed extensions, but not in the
- // list of evicted apps.
- extensions_info = prefs->GetInstalledExtensionsInfo();
- EXPECT_TRUE(IsAppInExtensionsInfo(*extensions_info, app_id));
- extensions_info = prefs->GetEvictedEphemeralAppsInfo();
- EXPECT_FALSE(IsAppInExtensionsInfo(*extensions_info, app_id));
+ PromoteEphemeralAppAndVerify(app, ExtensionRegistry::ENABLED);
}
-// Verify that the data of regular installed apps are deleted on uninstall.
-IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest, RemoveInstalledData) {
- // Install the ephemeral app and write data to various storage.
- const Extension* app = InstallPlatformApp(kRetainDataApp);
+// Verifies that promoting an ephemeral app that was disabled due to a
+// permissions increase will enable it.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest, PromoteAppAndGrantPermissions) {
+ InitSyncService();
+
+ const Extension* app = InstallAndLaunchEphemeralApp(kNotificationsTestApp);
ASSERT_TRUE(app);
- ASSERT_TRUE(LaunchAppAndRunTest(app, "WriteData")) << message_;
+ CloseAppWaitForUnload(app->id());
+ DisableEphemeralApp(app, Extension::DISABLE_PERMISSIONS_INCREASE);
- // Remove the app.
- const std::string app_id = app->id();
- EvictApp(app->id());
+ PromoteEphemeralAppAndVerify(app, ExtensionRegistry::ENABLED);
+ EXPECT_FALSE(ExtensionPrefs::Get(profile())
+ ->DidExtensionEscalatePermissions(app->id()));
+}
+
+// Verifies that promoting an ephemeral app that has unsupported requirements
+// will not enable it.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest,
+ PromoteUnsupportedEphemeralApp) {
+ InitSyncService();
+
+ const Extension* app = InstallAndLaunchEphemeralApp(kNotificationsTestApp);
+ ASSERT_TRUE(app);
+ CloseAppWaitForUnload(app->id());
+ DisableEphemeralApp(app, Extension::DISABLE_UNSUPPORTED_REQUIREMENT);
+
+ // When promoted to a regular installed app, it should remain disabled.
+ PromoteEphemeralAppAndVerify(app, ExtensionRegistry::DISABLED);
+}
+
+// Verifies that promoting an ephemeral app that is blacklisted will not enable
+// it.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest,
+ PromoteBlacklistedEphemeralApp) {
+ InitSyncService();
+
+ const Extension* app = InstallAndLaunchEphemeralApp(kNotificationsTestApp);
+ ASSERT_TRUE(app);
+ CloseAppWaitForUnload(app->id());
+
+ ExtensionService* service =
+ ExtensionSystem::Get(profile())->extension_service();
+ service->BlacklistExtensionForTest(app->id());
+ ASSERT_TRUE(
+ ExtensionRegistry::Get(profile())->blacklisted_extensions().Contains(
+ app->id()));
+
+ // When promoted to a regular installed app, it should remain blacklisted.
+ PromoteEphemeralAppAndVerify(app, ExtensionRegistry::BLACKLISTED);
+
+ // The app should be synced, but disabled.
+ scoped_ptr<AppSyncData> sync_change = GetLastSyncChangeForApp(app->id());
+ VerifySyncChange(sync_change.get(), false);
+}
+
+// Checks the process of promoting an ephemeral app from sync while the app is
+// running.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest,
+ PromoteAppFromSyncWhileRunning) {
+ InitSyncService();
+
+ const Extension* app = InstallAndLaunchEphemeralApp(kNotificationsTestApp);
+ ASSERT_TRUE(app);
+
+ PromoteEphemeralAppFromSyncAndVerify(app, true, ExtensionRegistry::ENABLED);
+
+ // Ensure that the app is not unloaded and disabled after it is closed.
+ CloseApp(app->id());
+ VerifyPromotedApp(app->id(), ExtensionRegistry::ENABLED);
+}
+
+// Checks the process of promoting an ephemeral app from sync while the app is
+// idle.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest, PromoteAppFromSyncWhileIdle) {
+ InitSyncService();
+
+ const Extension* app = InstallAndLaunchEphemeralApp(kNotificationsTestApp);
+ ASSERT_TRUE(app);
+ CloseAppWaitForUnload(app->id());
+ VerifyInactiveEphemeralApp(app->id());
+
+ PromoteEphemeralAppFromSyncAndVerify(app, true, ExtensionRegistry::ENABLED);
+}
+
+// Checks the process of promoting an ephemeral app from sync, where the app
+// from sync is disabled, and the ephemeral app is running.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest,
+ PromoteDisabledAppFromSyncWhileRunning) {
+ InitSyncService();
+
+ const Extension* app = InstallAndLaunchEphemeralApp(kNotificationsTestApp);
+ ASSERT_TRUE(app);
+
+ PromoteEphemeralAppFromSyncAndVerify(app, false, ExtensionRegistry::DISABLED);
+}
+
+// Checks the process of promoting an ephemeral app from sync, where the app
+// from sync is disabled, and the ephemeral app is idle.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest,
+ PromoteDisabledAppFromSyncWhileIdle) {
+ InitSyncService();
+
+ const Extension* app = InstallAndLaunchEphemeralApp(kNotificationsTestApp);
+ ASSERT_TRUE(app);
+ CloseAppWaitForUnload(app->id());
+ VerifyInactiveEphemeralApp(app->id());
+
+ PromoteEphemeralAppFromSyncAndVerify(app, false, ExtensionRegistry::DISABLED);
+}
+
+// In most cases, ExtensionService::PromoteEphemeralApp() will be called to
+// permanently install an ephemeral app. However, there may be cases where an
+// install occurs through the usual route of installing from the Web Store (due
+// to race conditions). Ensure that the app is still installed correctly.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest,
+ ReplaceEphemeralAppWithInstalledApp) {
+ InitSyncService();
+
+ const Extension* app = InstallAndLaunchEphemeralApp(kNotificationsTestApp);
+ ASSERT_TRUE(app);
+ CloseAppWaitForUnload(app->id());
+ std::string app_id = app->id();
+ app = NULL;
+
+ InstallObserver installed_observer(profile());
+ ReplaceEphemeralApp(app_id, kNotificationsTestApp, 1);
+ VerifyPromotedApp(app_id, ExtensionRegistry::ENABLED);
+
+ // Check the notification parameters.
+ const InstallObserver::InstallParameters& params = installed_observer.Last();
+ EXPECT_EQ(app_id, params.id);
+ EXPECT_TRUE(params.is_update);
+ EXPECT_TRUE(params.from_ephemeral);
+}
+
+// This is similar to ReplaceEphemeralAppWithInstalledApp, but installs will
+// be delayed until the app is idle.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest,
+ ReplaceEphemeralAppWithDelayedInstalledApp) {
+ InitSyncService();
+ const Extension* app = InstallAndLaunchEphemeralApp(kNotificationsTestApp);
+ ASSERT_TRUE(app);
+ std::string app_id = app->id();
+ app = NULL;
+
+ // Initiate install.
+ ReplaceEphemeralApp(app_id, kNotificationsTestApp, 0);
+
+ // The delayed installation will occur when the ephemeral app is closed.
+ content::WindowedNotificationObserver installed_signal(
+ extensions::NOTIFICATION_EXTENSION_WILL_BE_INSTALLED_DEPRECATED,
+ content::Source<Profile>(profile()));
+ InstallObserver installed_observer(profile());
+ CloseAppWaitForUnload(app_id);
+ installed_signal.Wait();
+ VerifyPromotedApp(app_id, ExtensionRegistry::ENABLED);
+
+ // Check the notification parameters.
+ const InstallObserver::InstallParameters& params = installed_observer.Last();
+ EXPECT_EQ(app_id, params.id);
+ EXPECT_TRUE(params.is_update);
+ EXPECT_TRUE(params.from_ephemeral);
+}
+
+// Verifies that an installed app cannot turn into an ephemeral app as result of
+// race conditions, i.e. an ephemeral app can be promoted to an installed app,
+// but not vice versa.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest,
+ ReplaceInstalledAppWithEphemeralApp) {
+ const Extension* app = InstallPlatformApp(kNotificationsTestApp);
+ ASSERT_TRUE(app);
+ std::string app_id = app->id();
app = NULL;
- // The app should not be in the preferences.
- ExtensionPrefs* prefs = ExtensionPrefs::Get(browser()->profile());
- ASSERT_TRUE(prefs);
- scoped_ptr<ExtensionPrefs::ExtensionsInfo> extensions_info(
- prefs->GetEvictedEphemeralAppsInfo());
- EXPECT_FALSE(IsAppInExtensionsInfo(*extensions_info, app_id));
- extensions_info = prefs->GetInstalledExtensionsInfo();
- EXPECT_FALSE(IsAppInExtensionsInfo(*extensions_info, app_id));
-
- // Reinstall the app and verify that all data has been reset.
- app = InstallPlatformApp(kRetainDataApp);
- ASSERT_TRUE(LaunchAppAndRunTest(app, "DataReset")) << message_;
-}
-
-// Verify that once evicted from the cache, ephemeral apps will remain in
-// extension prefs, but marked as evicted. After garbage collection of data,
-// both their data and preferences should be removed.
-IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest, GarbageCollectData) {
- // Create two apps. Both will be evicted from the cache, but the data of
- // one will be garbage collected.
- const Extension* evict_app = InstallEphemeralApp(kRetainDataApp);
- ASSERT_TRUE(evict_app);
- ASSERT_TRUE(LaunchAppAndRunTest(evict_app, "WriteData")) << message_;
- std::string evict_app_id = evict_app->id();
- EvictApp(evict_app_id);
- evict_app = NULL;
-
- const Extension* retain_app = InstallEphemeralApp(kDispatchEventTestApp);
- ASSERT_TRUE(retain_app);
- std::string retain_app_id = retain_app->id();
- EvictApp(retain_app_id);
- retain_app = NULL;
-
- ExtensionPrefs* prefs = ExtensionPrefs::Get(browser()->profile());
- ASSERT_TRUE(prefs);
-
- // Both apps should be in the list of evicted apps.
- scoped_ptr<ExtensionPrefs::ExtensionsInfo> extensions_info(
- prefs->GetEvictedEphemeralAppsInfo());
- EXPECT_TRUE(IsAppInExtensionsInfo(*extensions_info, retain_app_id));
- EXPECT_TRUE(IsAppInExtensionsInfo(*extensions_info, evict_app_id));
-
- // Set a fake last launch time so that the ephemeral app's data will be
- // garbage collected.
- base::Time launch_time =
- base::Time::Now() - base::TimeDelta::FromDays(
- EphemeralAppService::kDataInactiveThreshold + 1);
- prefs->SetLastLaunchTime(evict_app_id, launch_time);
- prefs->SetLastLaunchTime(retain_app_id, base::Time::Now());
-
- // Garbage collect data.
- GarbageCollectData();
-
- // The garbage collected app should no longer be in the preferences.
- extensions_info = prefs->GetEvictedEphemeralAppsInfo();
- EXPECT_TRUE(IsAppInExtensionsInfo(*extensions_info, retain_app_id));
- ASSERT_FALSE(IsAppInExtensionsInfo(*extensions_info, evict_app_id));
-
- // Reinstall the app and verify that all data has been reset.
- evict_app = InstallEphemeralApp(kRetainDataApp);
- ASSERT_TRUE(LaunchAppAndRunTest(evict_app, "DataReset")) << message_;
+ EXPECT_FALSE(extensions::util::IsEphemeralApp(app_id, profile()));
+ app =
+ InstallEphemeralAppWithSourceAndFlags(GetTestPath(kNotificationsTestApp),
+ 0,
+ Manifest::INTERNAL,
+ Extension::NO_FLAGS);
+ EXPECT_FALSE(extensions::util::IsEphemeralApp(app_id, profile()));
+}
+
+// Ephemerality was previously encoded by the Extension::IS_EPHEMERAL creation
+// flag. This was changed to an "ephemeral_app" property. Check that the prefs
+// are handled correctly.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest,
+ ExtensionPrefBackcompatibility) {
+ // Ensure that apps with the old prefs are recognized as ephemeral.
+ const Extension* app =
+ InstallExtensionWithSourceAndFlags(GetTestPath(kNotificationsTestApp),
+ 1,
+ Manifest::INTERNAL,
+ Extension::IS_EPHEMERAL);
+ ASSERT_TRUE(app);
+ EXPECT_TRUE(extensions::util::IsEphemeralApp(app->id(), profile()));
+
+ // Ensure that when the app is promoted to an installed app, the bit in the
+ // creation flags is cleared.
+ PromoteEphemeralApp(app);
+ EXPECT_FALSE(extensions::util::IsEphemeralApp(app->id(), profile()));
+
+ int creation_flags =
+ ExtensionPrefs::Get(profile())->GetCreationFlags(app->id());
+ EXPECT_EQ(0, creation_flags & Extension::IS_EPHEMERAL);
+}
+
+// Verifies that the power keep awake will be automatically released for
+// ephemeral apps that stop running. Well behaved apps should actually call
+// chrome.power.releaseKeepAwake() themselves.
+IN_PROC_BROWSER_TEST_F(EphemeralAppBrowserTest, ReleasePowerKeepAwake) {
+ PowerSettingsMock power_settings;
+ extensions::PowerApiManager* power_manager =
+ extensions::PowerApiManager::Get(profile());
+ power_manager->SetCreateBlockerFunctionForTesting(
+ base::Bind(&PowerSaveBlockerStub::Create, &power_settings));
+
+ const Extension* app = InstallAndLaunchEphemeralApp(kPowerTestApp);
+ ASSERT_TRUE(app);
+ EXPECT_EQ(1, power_settings.keep_awake_count());
+
+ CloseAppWaitForUnload(app->id());
+
+ EXPECT_EQ(0, power_settings.keep_awake_count());
}