Clear the ephemeral app cache from Clear Browsing Data

chrome/browser/apps/ephemeral_app_service.cc

 // Copyright 2013 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 "chrome/browser/apps/ephemeral_app_service.h"
 
 #include "base/command_line.h"
 #include "chrome/browser/apps/ephemeral_app_service_factory.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/extensions/extension_service.h"
@@ skipping 53 matching lines @@
       extensions::ExtensionRegistry::Get(profile_));
   registrar_.Add(this, chrome::NOTIFICATION_EXTENSIONS_READY,
                  content::Source<Profile>(profile_));
   registrar_.Add(this, chrome::NOTIFICATION_PROFILE_DESTROYED,
                  content::Source<Profile>(profile_));
 }
 
 EphemeralAppService::~EphemeralAppService() {
 }
 
+void EphemeralAppService::ClearCachedApps() {
+  // Cancel any pending garbage collects.
+  garbage_collect_apps_timer_.Stop();
+
+  scoped_ptr<ExtensionSet> extensions =
+      extensions::ExtensionRegistry::Get(profile_)
+          ->GenerateInstalledExtensionsSet();
+  ExtensionPrefs* prefs = ExtensionPrefs::Get(profile_);
+  DCHECK(prefs);
+
+  ExtensionService* service =
+      ExtensionSystem::Get(profile_)->extension_service();
+  DCHECK(service);
+
+  for (ExtensionSet::const_iterator it = extensions->begin();
+       it != extensions->end();
+       ++it) {
+    std::string extension_id = (*it)->id();
+    if (!prefs->IsEphemeralApp(extension_id) ||
+        !service->GetInstalledExtension(extension_id)) {

[tapted, 2014/07/11 01:16:36]

I think GetInstalledExtension(..) should return true for everything in
GenerateInstalledExtensionsSet - maybe DCHECK? (also note GetInstalledExtension
is marked deprecated)

unless uninstalling one extension can cascade and trigger an uninstall of
another..

[tmdiep, 2014/07/11 02:28:31]

There is a CHECK in ExtensionService::UninstallExtension() which makes me
nervous, so this is just a sanity check. If prefs->IsEphemeralApp() returns
true, there *should* be an extension in the registry, so I made this a DCHECK.

SharedModuleService::OnExtensionUninstalled() is a current case of cascading
uninstalls. That shouldn't apply here, but just in case there will be
dependencies between ephemeral apps in the future...

+      continue;
+    }
+
+    // Do not remove apps that are running.
+    if (!extensions::util::IsExtensionIdle(extension_id, profile_))
+      continue;
+
+    service->UninstallExtension(
+        extension_id,
+        ExtensionService::UNINSTALL_REASON_ORPHANED_EPHEMERAL_EXTENSION,
+        NULL);
+  }
+}
+
 void EphemeralAppService::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
   switch (type) {
     case chrome::NOTIFICATION_EXTENSIONS_READY: {
       Init();
       break;
     }
     case chrome::NOTIFICATION_PROFILE_DESTROYED: {
@@ skipping 98 matching lines @@
     // the app will be removed.
     if (last_launch_time.is_null())
       last_launch_time = prefs->GetInstallTime(extension->id());
 
     app_launch_times.insert(std::make_pair(last_launch_time, extension->id()));
   }
 
   ExtensionService* service =
       ExtensionSystem::Get(profile_)->extension_service();
   DCHECK(service);
-  // Execute the replacement policies and remove apps marked for deletion.
+  // Execute the eviction policies and remove apps marked for deletion.
   if (!app_launch_times.empty()) {
     GetAppsToRemove(app_count, app_launch_times, &remove_app_ids);
     for (std::set<std::string>::const_iterator id = remove_app_ids.begin();
          id != remove_app_ids.end(); ++id) {
+      if (!service->GetInstalledExtension(*id))

[tapted, 2014/07/11 01:16:36]

I don't think I've wrapped my head around this bit. Can you add a comment?  My
interpretation: since IsExtensionIdle returns `false` for extensions that it
can't find we can end up here with an extension that's been uninstalled. But
then I'm not sure why you don't need to clear out something else, or do
`--app_count` before continuing. And I'm not sure why it would have appeared in
the set generated by GenerateInstalledExtensionsSet in the first place.

Also would this apply to things uninstalled by sync as well as clear browsing
data?

(also GetInstalledExtension is marked deprecated - use the alternative?)

[tmdiep, 2014/07/11 02:28:31]

Same reason as above - in case of cascading uninstalls.

Good point about the app count. I don't really need to maintain it here, 
OnExtensionUninstalled() will handle it. 
Ephemeral apps don't really get synced. But suppose you have an ephemeral app on
device A, you installed and uninstalled it on device B.
- If both the install and uninstall messages got through - the app gets promoted
and uninstalled (this case is half tested in browser tests - the install from
sync part).
- If only the uninstall message got through for some reason, I think the
ephemeral app would be removed, which is fine. I'll need to add a browser test
for this.
Done.

+        continue;
+
       if (service->UninstallExtension(
               *id,
               ExtensionService::UNINSTALL_REASON_ORPHANED_EPHEMERAL_EXTENSION,
               NULL)) {
         --app_count;
       }
     }
   }
 
   ephemeral_app_count_ = app_count;
@@ skipping 21 matching lines @@
     // Remove ephemeral apps that have been inactive for a while or if the cache
     // is larger than the desired size.
     if (it->first < inactive_threshold || app_count > kMaxEphemeralAppsCount) {
       remove_app_ids->insert(it->second);
       --app_count;
     } else {
       break;
     }
   }
 }