Revert 227477 "Refactored loading of applications / extensions i..."

trunk/src/chrome/browser/extensions/api/developer_private/developer_private_api.cc

 // Copyright (c) 2012 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/extensions/api/developer_private/developer_private_api.h"
 
 #include "apps/app_load_service.h"
 #include "apps/app_restore_service.h"
 #include "apps/saved_files_service.h"
 #include "apps/shell_window.h"
 #include "apps/shell_window_registry.h"
 #include "base/base64.h"
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/files/file_enumerator.h"
 #include "base/i18n/file_util_icu.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/devtools/devtools_window.h"
 #include "chrome/browser/extensions/api/developer_private/developer_private_api_factory.h"
 #include "chrome/browser/extensions/api/developer_private/entry_picker.h"
 #include "chrome/browser/extensions/api/extension_action/extension_action_api.h"
 #include "chrome/browser/extensions/devtools_util.h"
 #include "chrome/browser/extensions/extension_disabled_ui.h"
 #include "chrome/browser/extensions/extension_error_reporter.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_system.h"
-#include "chrome/browser/extensions/image_loader.h"
 #include "chrome/browser/extensions/management_policy.h"
 #include "chrome/browser/extensions/unpacked_installer.h"
 #include "chrome/browser/extensions/updater/extension_updater.h"
 #include "chrome/browser/platform_util.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/sync_file_system/drive_backend_v1/drive_file_sync_service.h"
 #include "chrome/browser/sync_file_system/syncable_file_system_util.h"
 #include "chrome/browser/ui/chrome_select_file_policy.h"
 #include "chrome/browser/ui/webui/extensions/extension_icon_source.h"
 #include "chrome/common/extensions/api/developer_private.h"
@@ skipping 38 matching lines @@
 
 namespace {
 
 const base::FilePath::CharType kUnpackedAppsFolder[]
     = FILE_PATH_LITERAL("apps_target");
 
 ExtensionUpdater* GetExtensionUpdater(Profile* profile) {
     return profile->GetExtensionService()->updater();
 }
 
+GURL GetImageURLFromData(std::string contents) {
+  std::string contents_base64;
+  if (!base::Base64Encode(contents, &contents_base64))
+    return GURL();
+
+  // TODO(dvh): make use of chrome::kDataScheme. Filed as crbug/297301.
+  const char kDataURLPrefix[] = "data:image;base64,";
+  return GURL(kDataURLPrefix + contents_base64);
+}
+
+GURL GetDefaultImageURL(developer_private::ItemType type) {
+  int icon_resource_id;
+  switch (type) {
+    case developer::ITEM_TYPE_LEGACY_PACKAGED_APP:
+    case developer::ITEM_TYPE_HOSTED_APP:
+    case developer::ITEM_TYPE_PACKAGED_APP:
+      icon_resource_id = IDR_APP_DEFAULT_ICON;
+      break;
+    default:
+      icon_resource_id = IDR_EXTENSION_DEFAULT_ICON;
+      break;
+  }
+
+  return GetImageURLFromData(
+      ResourceBundle::GetSharedInstance().GetRawDataResourceForScale(
+          icon_resource_id, ui::SCALE_FACTOR_100P).as_string());
+}
+
+// TODO(dvh): This code should be refactored and moved to
+// extensions::ImageLoader. Also a resize should be performed to avoid
+// potential huge URLs: crbug/297298.
+GURL ToDataURL(const base::FilePath& path, developer_private::ItemType type) {
+  std::string contents;
+  if (path.empty() || !base::ReadFileToString(path, &contents))
+    return GetDefaultImageURL(type);
+
+  return GetImageURLFromData(contents);
+}
+
 std::vector<base::FilePath> ListFolder(const base::FilePath path) {
   base::FileEnumerator files(path, false,
       base::FileEnumerator::DIRECTORIES | base::FileEnumerator::FILES);
   std::vector<base::FilePath> paths;
 
   for (base::FilePath current_path = files.Next(); !current_path.empty();
        current_path = files.Next()) {
     paths.push_back(current_path);
   }
   return paths;
@@ skipping 230 matching lines @@
   }
 
   info->may_disable = system->management_policy()->
       UserMayModifySettings(&item, NULL);
   info->is_app = item.is_app();
   info->views = GetInspectablePagesForExtension(&item, item_is_enabled);
 
   return info.Pass();
 }
 
+void DeveloperPrivateGetItemsInfoFunction::GetIconsOnFileThread(
+    ItemInfoList item_list,
+    const std::map<std::string, ExtensionResource> idToIcon) {
+  for (ItemInfoList::iterator iter = item_list.begin();
+       iter != item_list.end(); ++iter) {
+    developer_private::ItemInfo* info = iter->get();
+    std::map<std::string, ExtensionResource>::const_iterator resource_ptr
+        = idToIcon.find(info->id);
+    if (resource_ptr != idToIcon.end()) {
+      info->icon_url =
+          ToDataURL(resource_ptr->second.GetFilePath(), info->type).spec();
+    }
+  }
+
+  results_ = developer::GetItemsInfo::Results::Create(item_list);
+  content::BrowserThread::PostTask(content::BrowserThread::UI, FROM_HERE,
+      base::Bind(&DeveloperPrivateGetItemsInfoFunction::SendResponse,
+                 this,
+                 true));
+}
+
 void DeveloperPrivateGetItemsInfoFunction::
     GetInspectablePagesForExtensionProcess(
         const Extension* extension,
         const std::set<content::RenderViewHost*>& views,
         ItemInspectViewList* result) {
   bool has_generated_background_page =
       BackgroundInfo::HasGeneratedBackgroundPage(extension);
   for (std::set<content::RenderViewHost*>::const_iterator iter = views.begin();
        iter != views.end(); ++iter) {
     content::RenderViewHost* host = *iter;
@@ skipping 139 matching lines @@
   items.InsertAll(*service->extensions());
 
   if (include_disabled) {
     items.InsertAll(*service->disabled_extensions());
   }
 
   if (include_terminated) {
     items.InsertAll(*service->terminated_extensions());
   }
 
+  std::map<std::string, ExtensionResource> id_to_icon;
+  ItemInfoList item_list;
+
   for (ExtensionSet::const_iterator iter = items.begin();
        iter != items.end(); ++iter) {
     const Extension& item = *iter->get();
 
+    ExtensionResource item_resource =
+        IconsInfo::GetIconResource(&item,
+                                   extension_misc::EXTENSION_ICON_MEDIUM,
+                                   ExtensionIconSet::MATCH_BIGGER);
+    id_to_icon[item.id()] = item_resource;
+
     // Don't show component extensions and invisible apps.
     if (item.ShouldNotBeVisible())
       continue;
 
-    item_list_.push_back(make_linked_ptr<developer::ItemInfo>(
+    item_list.push_back(make_linked_ptr<developer::ItemInfo>(
         CreateItemInfo(
             item, service->IsExtensionEnabled(item.id())).release()));
   }
 
-  // Iterate over |item_list_| to request all the icons.
-  RequestNextIcon();
+  content::BrowserThread::PostTask(content::BrowserThread::FILE, FROM_HERE,
+      base::Bind(&DeveloperPrivateGetItemsInfoFunction::GetIconsOnFileThread,
+                 this,
+                 item_list,
+                 id_to_icon));
 
   return true;
 }
 
-void DeveloperPrivateGetItemsInfoFunction::RequestNextIcon() {
-  developer::ItemInfo* info = item_list_[icon_to_load_].get();
-  ExtensionService* service = profile()->GetExtensionService();
-  ImageLoader::Get(profile())->LoadExtensionIconDataURLAsync(
-      service->GetExtensionById(info->id, true),
-      extension_misc::EXTENSION_ICON_MEDIUM,
-      ExtensionIconSet::MATCH_BIGGER,
-      false,
-      base::Bind(&DeveloperPrivateGetItemsInfoFunction::LoadIconFinished,
-                 this));
-}
-
-void DeveloperPrivateGetItemsInfoFunction::LoadIconFinished(const GURL& url) {
-  // Store the result.
-  developer::ItemInfo* info = item_list_[icon_to_load_].get();
-  info->icon_url = url.spec();
-  ++icon_to_load_;
-  if (icon_to_load_ < static_cast<int>(item_list_.size())) {
-    RequestNextIcon();
-  } else {
-    // All icons have been loaded: send the result.
-    results_ = developer::GetItemsInfo::Results::Create(item_list_);
-    SendResponse(true);
-  }
-}
-
-DeveloperPrivateGetItemsInfoFunction::DeveloperPrivateGetItemsInfoFunction()
-    : icon_to_load_(0) {
-}
-
 DeveloperPrivateGetItemsInfoFunction::~DeveloperPrivateGetItemsInfoFunction() {}
 
 bool DeveloperPrivateAllowFileAccessFunction::RunImpl() {
   scoped_ptr<AllowFileAccess::Params> params(
       AllowFileAccess::Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params.get());
 
   EXTENSION_FUNCTION_VALIDATE(user_gesture_);
 
   ExtensionSystem* system = ExtensionSystem::Get(profile());
@@ skipping 743 matching lines @@
   return true;
 }
 
 DeveloperPrivateIsProfileManagedFunction::
     ~DeveloperPrivateIsProfileManagedFunction() {
 }
 
 } // namespace api
 
 } // namespace extensions