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

Unified Diff: chrome/browser/ui/webui/chromeos/login/eula_screen_handler.cc

Issue 14063018: Implemented argument wrapping for CallJS. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Get rid of macro. Created 7 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/ui/webui/chromeos/login/eula_screen_handler.cc
diff --git a/chrome/browser/ui/webui/chromeos/login/eula_screen_handler.cc b/chrome/browser/ui/webui/chromeos/login/eula_screen_handler.cc
index a595e1af18a8a91edfd5567a440ade996809653b..9a4199b6f762de57b3ef6c1e7bfb563ad9dfc661 100644
--- a/chrome/browser/ui/webui/chromeos/login/eula_screen_handler.cc
+++ b/chrome/browser/ui/webui/chromeos/login/eula_screen_handler.cc
@@ -83,14 +83,13 @@ void EulaScreenHandler::Initialize() {
if (!page_is_ready() || !delegate_)
return;
- base::FundamentalValue checked(delegate_->IsUsageStatsEnabled());
- CallJS("cr.ui.Oobe.setUsageStats", checked);
+ CallJS("cr.ui.Oobe.setUsageStats", delegate_->IsUsageStatsEnabled());
// This OEM EULA is a file:// URL which we're unable to load in iframe.
// Instead if it's defined we use chrome://terms/oem that will load same file.
if (!delegate_->GetOemEulaUrl().is_empty()) {
- StringValue oem_eula_url(chrome::kChromeUITermsOemURL);
- CallJS("cr.ui.Oobe.setOemEulaUrl", oem_eula_url);
+ CallJS("cr.ui.Oobe.setOemEulaUrl",
+ std::string(chrome::kChromeUITermsOemURL));
}
if (show_on_init_) {
@@ -107,8 +106,7 @@ void EulaScreenHandler::RegisterMessages() {
}
void EulaScreenHandler::OnPasswordFetched(const std::string& tpm_password) {
- StringValue tpm_password_value(tpm_password);
- CallJS("cr.ui.Oobe.setTpmPassword", tpm_password_value);
+ CallJS("cr.ui.Oobe.setTpmPassword", tpm_password);
}
void EulaScreenHandler::HandleOnExit(bool accepted, bool usage_stats_enabled) {

Powered by Google App Engine
This is Rietveld 408576698