[SyncFS] Make GetOriginStatusMap asynchronous

chrome/browser/ui/webui/sync_file_system_internals/extension_statuses_handler.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/ui/webui/sync_file_system_internals/extension_statuses_handler.h"
 
 #include <map>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/values.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/sync_file_system/sync_file_system_service.h"
 #include "chrome/browser/sync_file_system/sync_file_system_service_factory.h"
 #include "content/public/browser/web_ui.h"
 #include "content/public/browser/web_ui_data_source.h"
 #include "extensions/browser/extension_system.h"
 #include "extensions/common/extension.h"
 #include "grit/sync_file_system_internals_resources.h"
 
 using sync_file_system::SyncFileSystemServiceFactory;
 using sync_file_system::SyncServiceState;
 
 namespace syncfs_internals {
 
+namespace {
+
+void ConvertExtensionStatusToDictionary(
+    ExtensionService* extension_service,
+    const base::Callback<void(const base::ListValue&)>& callback,
+    const std::map<GURL, std::string>& status_map) {
+  DCHECK(!extension_service);
+
+  base::ListValue list;
+  for (std::map<GURL, std::string>::const_iterator itr = status_map.begin();
+       itr != status_map.end();
+       ++itr) {
+    std::string extension_id = itr->first.HostNoBrackets();
+
+    // Join with human readable extension name.
+    const extensions::Extension* extension =
+        extension_service->GetExtensionById(extension_id, true);
+    if (!extension)
+      continue;
+
+    base::DictionaryValue* dict = new base::DictionaryValue;
+    dict->SetString("extensionID", extension_id);
+    dict->SetString("extensionName", extension->name());
+    dict->SetString("status", itr->second);
+    list.Append(dict);
+  }
+
+  callback.Run(list);
+}
+
+}  // namespace
+
 ExtensionStatusesHandler::ExtensionStatusesHandler(Profile* profile)
-    : profile_(profile) {}
+    : profile_(profile),
+      weak_ptr_factory_(this) {}
 
 ExtensionStatusesHandler::~ExtensionStatusesHandler() {}
 
 void ExtensionStatusesHandler::RegisterMessages() {
   web_ui()->RegisterMessageCallback(
       "getExtensionStatuses",
       base::Bind(&ExtensionStatusesHandler::GetExtensionStatuses,
                  base::Unretained(this)));
 }
 
 // static
 void ExtensionStatusesHandler::GetExtensionStatusesAsDictionary(
     Profile* profile,
-    base::ListValue* values) {
+    const base::Callback<void(const base::ListValue&)>& callback) {
   DCHECK(profile);
-  DCHECK(values);
+
   sync_file_system::SyncFileSystemService* sync_service =
       SyncFileSystemServiceFactory::GetForProfile(profile);
+  if (!sync_service)
+    return;
+
   ExtensionService* extension_service =
       extensions::ExtensionSystem::Get(profile)->extension_service();
-  if (!sync_service || !extension_service)
+  if (!extension_service)
     return;
 
-  std::map<GURL, std::string> status_map;
-  sync_service->GetExtensionStatusMap(&status_map);
-  for (std::map<GURL, std::string>::const_iterator itr = status_map.begin();
-       itr != status_map.end();
-       ++itr) {
-    std::string extension_id = itr->first.HostNoBrackets();
-
-    // Join with human readable extension name.
-    const extensions::Extension* extension =
-        extension_service->GetExtensionById(extension_id, true);
-    if (!extension)
-      continue;
-
-    base::DictionaryValue* dict = new base::DictionaryValue;
-    dict->SetString("extensionID", extension_id);
-    dict->SetString("extensionName", extension->name());
-    dict->SetString("status", itr->second);
-    values->Append(dict);
-  }
+  sync_service->GetExtensionStatusMap(base::Bind(
+      &ConvertExtensionStatusToDictionary,
+      base::Unretained(extension_service), callback));

[tzik, 2014/05/21 10:52:33]

extension_service->AsWeakPtr() and check it in
ConvertExtensionStatusToDictionary? (just in case)

[peria, 2014/05/22 02:25:21]

Done.
There already is a DCHECK(!extension_service).
Should I add other checks (for release builds)?

 }
 
 void ExtensionStatusesHandler::GetExtensionStatuses(
     const base::ListValue* args) {
   DCHECK(args);
-  base::ListValue list;
-  GetExtensionStatusesAsDictionary(profile_, &list);
+  GetExtensionStatusesAsDictionary(
+      profile_,
+      base::Bind(&ExtensionStatusesHandler::DidGetExtensionStatuses,
+                 weak_ptr_factory_.GetWeakPtr()));
+}
+
+void ExtensionStatusesHandler::DidGetExtensionStatuses(
+    const base::ListValue& list) {
   web_ui()->CallJavascriptFunction("ExtensionStatuses.onGetExtensionStatuses",
                                    list);
 }
 
 }  // namespace syncfs_internals