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

Unified Diff: chrome/browser/extensions/api/input_ime/input_ime_api_nonchromeos.cc

Issue 2476493003: Remove FundamentalValue
Patch Set: Fix Created 4 years, 1 month 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/input_ime/input_ime_api_nonchromeos.cc
diff --git a/chrome/browser/extensions/api/input_ime/input_ime_api_nonchromeos.cc b/chrome/browser/extensions/api/input_ime/input_ime_api_nonchromeos.cc
index 8d8778b812a0e9407f50384fa36dda1f59a20514..df3bf959fd006d40b49e20fa123dfb11d50b8c2c 100644
--- a/chrome/browser/extensions/api/input_ime/input_ime_api_nonchromeos.cc
+++ b/chrome/browser/extensions/api/input_ime/input_ime_api_nonchromeos.cc
@@ -138,7 +138,7 @@ void InputImeAPI::OnExtensionLoaded(content::BrowserContext* browser_context,
Profile* profile = Profile::FromBrowserContext(browser_context);
ExtensionPrefs::Get(profile)->UpdateExtensionPref(
extension->id(), kPrefNeverActivatedSinceLoaded,
- new base::FundamentalValue(true));
+ new base::Value(true));
}
void InputImeAPI::OnExtensionUnloaded(content::BrowserContext* browser_context,
@@ -150,7 +150,7 @@ void InputImeAPI::OnExtensionUnloaded(content::BrowserContext* browser_context,
// Records the extension is not the last active IME engine.
ExtensionPrefs::Get(Profile::FromBrowserContext(browser_context))
->UpdateExtensionPref(extension->id(), kPrefLastActiveEngine,
- new base::FundamentalValue(false));
+ new base::Value(false));
event_router->DeleteInputMethodEngine(extension->id());
}
}
@@ -178,7 +178,7 @@ void InputImeEventRouter::SetActiveEngine(const std::string& extension_id) {
// Records the extension is the last active IME engine.
ExtensionPrefs::Get(GetProfile())
->UpdateExtensionPref(extension_id, kPrefLastActiveEngine,
- new base::FundamentalValue(true));
+ new base::Value(true));
if (active_engine_) {
if (active_engine_->GetExtensionId() == extension_id) {
active_engine_->Enable(std::string());
@@ -189,7 +189,7 @@ void InputImeEventRouter::SetActiveEngine(const std::string& extension_id) {
ExtensionPrefs::Get(GetProfile())
->UpdateExtensionPref(active_engine_->GetExtensionId(),
kPrefLastActiveEngine,
- new base::FundamentalValue(false));
+ new base::Value(false));
DeleteInputMethodEngine(active_engine_->GetExtensionId());
}
@@ -241,13 +241,13 @@ ExtensionFunction::ResponseAction InputImeActivateFunction::Run() {
event_router->SetActiveEngine(extension_id());
ExtensionPrefs::Get(profile)->UpdateExtensionPref(
extension_id(), kPrefNeverActivatedSinceLoaded,
- new base::FundamentalValue(false));
+ new base::Value(false));
return RespondNow(NoArguments());
}
// The API has already been called at least once.
ExtensionPrefs::Get(profile)->UpdateExtensionPref(
extension_id(), kPrefNeverActivatedSinceLoaded,
- new base::FundamentalValue(false));
+ new base::Value(false));
// Otherwise, this API is only allowed to be called from a user action.
if (!user_gesture())
@@ -309,7 +309,7 @@ void InputImeActivateFunction::OnPermissionBubbleFinished(
// again' check box. So we can activate the extension directly next time.
ExtensionPrefs::Get(profile)->UpdateExtensionPref(
extension_id(), kPrefWarningBubbleNeverShow,
- new base::FundamentalValue(true));
+ new base::Value(true));
}
Respond(NoArguments());
@@ -367,7 +367,7 @@ ExtensionFunction::ResponseAction InputImeCreateWindowFunction::Run() {
return RespondNow(Error(error));
std::unique_ptr<base::DictionaryValue> result(new base::DictionaryValue());
- result->Set("frameId", new base::FundamentalValue(frame_id));
+ result->Set("frameId", new base::Value(frame_id));
return RespondNow(OneArgument(std::move(result)));
}

Powered by Google App Engine
This is Rietveld 408576698