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

Unified Diff: chrome/browser/extensions/api/font_settings/font_settings_api.cc

Issue 252653002: Rename (Chrome)SyncExtensionFunction::RunImpl to RunSync so that the RunImpl (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix bookmarks Created 6 years, 8 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/extensions/api/font_settings/font_settings_api.cc
diff --git a/chrome/browser/extensions/api/font_settings/font_settings_api.cc b/chrome/browser/extensions/api/font_settings/font_settings_api.cc
index befb3b3d6256ae71412ba5b52e486380d7caaf67..2031cf74618e5ac7e60c45442c1fec6d52da16a1 100644
--- a/chrome/browser/extensions/api/font_settings/font_settings_api.cc
+++ b/chrome/browser/extensions/api/font_settings/font_settings_api.cc
@@ -220,7 +220,7 @@ FontSettingsAPI::GetFactoryInstance() {
return g_factory.Pointer();
}
-bool FontSettingsClearFontFunction::RunImpl() {
+bool FontSettingsClearFontFunction::RunSync() {
if (GetProfile()->IsOffTheRecord()) {
error_ = kSetFromIncognitoError;
return false;
@@ -242,7 +242,7 @@ bool FontSettingsClearFontFunction::RunImpl() {
return true;
}
-bool FontSettingsGetFontFunction::RunImpl() {
+bool FontSettingsGetFontFunction::RunSync() {
scoped_ptr<fonts::GetFont::Params> params(
fonts::GetFont::Params::Create(*args_));
EXTENSION_FUNCTION_VALIDATE(params.get());
@@ -273,7 +273,7 @@ bool FontSettingsGetFontFunction::RunImpl() {
return true;
}
-bool FontSettingsSetFontFunction::RunImpl() {
+bool FontSettingsSetFontFunction::RunSync() {
if (GetProfile()->IsOffTheRecord()) {
error_ = kSetFromIncognitoError;
return false;
@@ -343,7 +343,7 @@ bool FontSettingsGetFontListFunction::CopyFontsToResult(
return true;
}
-bool ClearFontPrefExtensionFunction::RunImpl() {
+bool ClearFontPrefExtensionFunction::RunSync() {
if (GetProfile()->IsOffTheRecord()) {
error_ = kSetFromIncognitoError;
return false;
@@ -354,7 +354,7 @@ bool ClearFontPrefExtensionFunction::RunImpl() {
return true;
}
-bool GetFontPrefExtensionFunction::RunImpl() {
+bool GetFontPrefExtensionFunction::RunSync() {
PrefService* prefs = GetProfile()->GetPrefs();
const PrefService::Preference* pref = prefs->FindPreference(GetPrefName());
EXTENSION_FUNCTION_VALIDATE(pref);
@@ -374,7 +374,7 @@ bool GetFontPrefExtensionFunction::RunImpl() {
return true;
}
-bool SetFontPrefExtensionFunction::RunImpl() {
+bool SetFontPrefExtensionFunction::RunSync() {
if (GetProfile()->IsOffTheRecord()) {
error_ = kSetFromIncognitoError;
return false;
« no previous file with comments | « chrome/browser/extensions/api/font_settings/font_settings_api.h ('k') | chrome/browser/extensions/api/history/history_api.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698