Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(513)

Unified Diff: chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc

Issue 2335213004: [Extensions] Convert some ChromeSyncExtensionFunctions (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc
diff --git a/chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc b/chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc
index 98c233eda7c13a2e50060a0e93e6966348feeb1b..24577c811c2736a3eefe7a41274c2d5e8f8ee870 100644
--- a/chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc
+++ b/chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc
@@ -11,6 +11,7 @@
#include "base/sys_info.h"
#include "chrome/browser/browser_process.h"
#include "chrome/browser/chromeos/file_manager/open_with_browser.h"
+#include "chrome/browser/profiles/profile.h"
#include "chrome/grit/generated_resources.h"
#include "chromeos/system/statistics_provider.h"
#include "extensions/common/extension_l10n_util.h"
@@ -347,7 +348,7 @@ FileManagerPrivateGetStringsFunction::FileManagerPrivateGetStringsFunction() {
FileManagerPrivateGetStringsFunction::~FileManagerPrivateGetStringsFunction() {
}
-bool FileManagerPrivateGetStringsFunction::RunSync() {
+ExtensionFunction::ResponseAction FileManagerPrivateGetStringsFunction::Run() {
std::unique_ptr<base::DictionaryValue> dict(new base::DictionaryValue());
AddStringsForDrive(dict.get());
@@ -676,10 +677,12 @@ bool FileManagerPrivateGetStringsFunction::RunSync() {
dict->SetBoolean("PDF_VIEW_ENABLED",
file_manager::util::ShouldBeOpenedWithPlugin(
- GetProfile(), FILE_PATH_LITERAL(".pdf")));
+ Profile::FromBrowserContext(browser_context()),
+ FILE_PATH_LITERAL(".pdf")));
dict->SetBoolean("SWF_VIEW_ENABLED",
file_manager::util::ShouldBeOpenedWithPlugin(
- GetProfile(), FILE_PATH_LITERAL(".swf")));
+ Profile::FromBrowserContext(browser_context()),
+ FILE_PATH_LITERAL(".swf")));
dict->SetString("CHROMEOS_RELEASE_BOARD",
base::SysInfo::GetLsbReleaseBoard());
dict->SetString(
@@ -703,9 +706,8 @@ bool FileManagerPrivateGetStringsFunction::RunSync() {
const std::string& app_locale = g_browser_process->GetApplicationLocale();
webui::SetLoadTimeDataDefaults(app_locale, dict.get());
- SetResult(std::move(dict));
- return true;
+ return RespondNow(OneArgument(std::move(dict)));
}
} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698