Do not pop up Files app, when the recovery tool is running.

chrome/browser/chromeos/extensions/file_manager/event_router.cc

Index: chrome/browser/chromeos/extensions/file_manager/event_router.cc
diff --git a/chrome/browser/chromeos/extensions/file_manager/event_router.cc b/chrome/browser/chromeos/extensions/file_manager/event_router.cc
index 3d447debbfd51c227785c5cc613c9c0142a3d31e..fbaa9339228ed8c4a362303c8e68b36115a91c97 100644
--- a/chrome/browser/chromeos/extensions/file_manager/event_router.cc
+++ b/chrome/browser/chromeos/extensions/file_manager/event_router.cc
@@ -38,6 +38,7 @@
 #include "content/public/browser/notification_source.h"
 #include "content/public/browser/render_process_host.h"
 #include "extensions/browser/event_router.h"
+#include "extensions/browser/extension_prefs.h"
 #include "extensions/browser/extension_system.h"
 #include "webkit/common/fileapi/file_system_types.h"
 #include "webkit/common/fileapi/file_system_util.h"
@@ -120,6 +121,7 @@ void JobInfoToTransferStatus(
 }
 
 // Checks for availability of the Google+ Photos app.
+// TODO(mtomasz): Replace with crbug.com/341902 solution.
 bool IsGooglePhotosInstalled(Profile *profile) {
   ExtensionService* service =
       extensions::ExtensionSystem::Get(profile)->extension_service();
@@ -129,9 +131,9 @@ bool IsGooglePhotosInstalled(Profile *profile) {
   // Google+ Photos uses several ids for different channels. Therefore, all of
   // them should be checked.
   const std::string kGooglePlusPhotosIds[] = {
-    "ebpbnabdhheoknfklmpddcdijjkmklkp",  // G+ Photos staging
-    "efjnaogkjbogokcnohkmnjdojkikgobo",  // G+ Photos prod
-    "ejegoaikibpmikoejfephaneibodccma"   // G+ Photos dev
+      "ebpbnabdhheoknfklmpddcdijjkmklkp",  // G+ Photos staging
+      "efjnaogkjbogokcnohkmnjdojkikgobo",  // G+ Photos prod
+      "ejegoaikibpmikoejfephaneibodccma"   // G+ Photos dev
   };
 
   for (size_t i = 0; i < arraysize(kGooglePlusPhotosIds); ++i) {
@@ -143,6 +145,34 @@ bool IsGooglePhotosInstalled(Profile *profile) {
   return false;
 }
 
+// Checks if the Recovery Tool is running. This is a temporary solution.
+// TODO(mtomasz): Replace with crbug.com/341902 solution.
+bool IsRecoveryToolRunning(Profile* profile) {
+  ExtensionService* service =
+      extensions::ExtensionSystem::Get(profile)->extension_service();
+  if (!service)
+    return false;
+  extensions::ExtensionPrefs* extension_prefs = service->extension_prefs();
+  if (!extension_prefs)
+    return false;
+
+  const std::string kRecoveryToolIds[] = {
+      "kkebgepbbgbcmghedmmdfcbdcodlkngh",  // Recovert tool staging
+      "jndclpdbaamdhonoechobihbbiimdgai"   // Recovery tool prod
+  };
+
+  for (size_t i = 0; i < arraysize(kRecoveryToolIds); ++i) {
+    const std::string extension_id = kRecoveryToolIds[i];
+    if (service->GetExtensionById(extension_id, false /* include_disable */) !=
+        NULL) {
+      if (extension_prefs->IsExtensionRunning(extension_id))
+        return true;
+    }
+  }
+
+  return false;
+}
+
 // Sends an event named |event_name| with arguments |event_args| to extensions.
 void BroadcastEvent(Profile* profile,
                     const std::string& event_name,
@@ -690,18 +720,23 @@ void EventRouter::ShowRemovableDeviceInFileManager(
       profile_ != ProfileManager::GetActiveUserProfile())
     return;
 
+  // Do not pop-up the File Manager, if the recovery tool is running.
+  if (IsRecoveryToolRunning(profile_))
+    return;
+
   // According to DCF (Design rule of Camera File system) by JEITA / CP-3461
   // cameras should have pictures located in the DCIM root directory.
   const base::FilePath dcim_path = mount_path.Append(
       FILE_PATH_LITERAL("DCIM"));
 
-  // If there is no DCIM folder or an external photo importer is not available,
-  // then launch Files.app.
+  // If there is a DCIM folder and Google+ Photos is installed, then do not
+  // launch Files.app. Also, do not show Files app, if the recovery tool is
+  // running.

[kinaba, 2014/02/19 09:10:20]

The comment "Also, do not show Files app, if the recovery tool is running." is
already written few lines above.

[mtomasz, 2014/02/19 09:37:48]

Done.

   DirectoryExistsOnUIThread(
       dcim_path,
-      IsGooglePhotosInstalled(profile_) ?
-      base::Bind(&base::DoNothing) :
-      base::Bind(&util::OpenRemovableDrive, profile_, mount_path),
+      IsGooglePhotosInstalled(profile_)
+          ? base::Bind(&base::DoNothing)
+          : base::Bind(&util::OpenRemovableDrive, profile_, mount_path),
       base::Bind(&util::OpenRemovableDrive, profile_, mount_path));
 }