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

Unified Diff: chrome/browser/ui/webui/sync_file_system_internals/sync_file_system_internals_handler.cc

Issue 2664753002: Remove base::StringValue (Closed)
Patch Set: Rebase Created 3 years, 9 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/sync_file_system_internals/sync_file_system_internals_handler.cc
diff --git a/chrome/browser/ui/webui/sync_file_system_internals/sync_file_system_internals_handler.cc b/chrome/browser/ui/webui/sync_file_system_internals/sync_file_system_internals_handler.cc
index 57872aaca994ccba7f1328e214c70c337a802f16..2bd61444adbb9a5a4c893beb566aba55e9d69d98 100644
--- a/chrome/browser/ui/webui/sync_file_system_internals/sync_file_system_internals_handler.cc
+++ b/chrome/browser/ui/webui/sync_file_system_internals/sync_file_system_internals_handler.cc
@@ -86,7 +86,7 @@ void SyncFileSystemInternalsHandler::OnSyncStateUpdated(
// TODO(calvinlo): OnSyncStateUpdated should be updated to also provide the
// notification mechanism (XMPP or Polling).
web_ui()->CallJavascriptFunctionUnsafe("SyncService.onGetServiceStatus",
- base::StringValue(state_string));
+ base::Value(state_string));
}
void SyncFileSystemInternalsHandler::OnFileSynced(
@@ -121,7 +121,7 @@ void SyncFileSystemInternalsHandler::GetServiceStatus(
const std::string state_string = extensions::api::sync_file_system::ToString(
extensions::SyncServiceStateToExtensionEnum(state_enum));
web_ui()->CallJavascriptFunctionUnsafe("SyncService.onGetServiceStatus",
- base::StringValue(state_string));
+ base::Value(state_string));
}
void SyncFileSystemInternalsHandler::GetNotificationSource(
@@ -132,9 +132,8 @@ void SyncFileSystemInternalsHandler::GetNotificationSource(
return;
bool xmpp_enabled = drive_notification_manager->push_notification_enabled();
std::string notification_source = xmpp_enabled ? "XMPP" : "Polling";
- web_ui()->CallJavascriptFunctionUnsafe(
- "SyncService.onGetNotificationSource",
- base::StringValue(notification_source));
+ web_ui()->CallJavascriptFunctionUnsafe("SyncService.onGetNotificationSource",
+ base::Value(notification_source));
}
void SyncFileSystemInternalsHandler::GetLog(

Powered by Google App Engine
This is Rietveld 408576698