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

Unified Diff: components/sync/driver/about_sync_util.cc

Issue 2388673002: Revert of [Sync] Move //components/sync to the syncer namespace. (patchset #5 id:40001 of https://co (Closed)
Patch Set: Created 4 years, 2 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
« no previous file with comments | « components/sync/driver/about_sync_util.h ('k') | components/sync/driver/about_sync_util_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/sync/driver/about_sync_util.cc
diff --git a/components/sync/driver/about_sync_util.cc b/components/sync/driver/about_sync_util.cc
index 3b5d38f548367120f4c8c3128ff12f322509e657..9a57372a1c517cd401c1e371c0b91ae7db8fdecd 100644
--- a/components/sync/driver/about_sync_util.cc
+++ b/components/sync/driver/about_sync_util.cc
@@ -23,7 +23,7 @@
using base::DictionaryValue;
using base::ListValue;
-namespace syncer {
+namespace sync_driver {
namespace sync_ui_util {
@@ -205,27 +205,28 @@ std::string GetTimeStr(base::Time time, const std::string& default_msg) {
if (time.is_null())
time_str = default_msg;
else
- time_str = GetTimeDebugString(time);
+ time_str = syncer::GetTimeDebugString(time);
return time_str;
}
-std::string GetConnectionStatus(const SyncService::SyncTokenStatus& status) {
+std::string GetConnectionStatus(
+ const sync_driver::SyncService::SyncTokenStatus& status) {
std::string message;
switch (status.connection_status) {
- case CONNECTION_NOT_ATTEMPTED:
+ case syncer::CONNECTION_NOT_ATTEMPTED:
base::StringAppendF(&message, "not attempted");
break;
- case CONNECTION_OK:
+ case syncer::CONNECTION_OK:
base::StringAppendF(
&message, "OK since %s",
GetTimeStr(status.connection_status_update_time, "n/a").c_str());
break;
- case CONNECTION_AUTH_ERROR:
+ case syncer::CONNECTION_AUTH_ERROR:
base::StringAppendF(
&message, "auth error since %s",
GetTimeStr(status.connection_status_update_time, "n/a").c_str());
break;
- case CONNECTION_SERVER_ERROR:
+ case syncer::CONNECTION_SERVER_ERROR:
base::StringAppendF(
&message, "server error since %s",
GetTimeStr(status.connection_status_update_time, "n/a").c_str());
@@ -243,7 +244,7 @@ std::string GetConnectionStatus(const SyncService::SyncTokenStatus& status) {
// which are grouped into sections and populated with the help of the SyncStat
// classes defined above.
std::unique_ptr<base::DictionaryValue> ConstructAboutInformation(
- SyncService* service,
+ sync_driver::SyncService* service,
SigninManagerBase* signin,
version_info::Channel channel) {
std::unique_ptr<base::DictionaryValue> about_info(
@@ -357,10 +358,10 @@ std::unique_ptr<base::DictionaryValue> ConstructAboutInformation(
return about_info;
}
- SyncStatus full_status;
+ syncer::SyncStatus full_status;
bool is_status_valid = service->QueryDetailedSyncStatus(&full_status);
bool sync_active = service->IsSyncActive();
- const SyncCycleSnapshot& snapshot = service->GetLastCycleSnapshot();
+ const syncer::SyncCycleSnapshot& snapshot = service->GetLastCycleSnapshot();
if (is_status_valid)
summary_string.SetValue(service->QuerySyncStatusSummaryString());
@@ -374,7 +375,7 @@ std::unique_ptr<base::DictionaryValue> ConstructAboutInformation(
if (signin)
username.SetValue(signin->GetAuthenticatedAccountInfo().email);
- const SyncService::SyncTokenStatus& token_status =
+ const sync_driver::SyncService::SyncTokenStatus& token_status =
service->GetSyncTokenStatus();
server_connection.SetValue(GetConnectionStatus(token_status));
request_token_time.SetValue(
@@ -424,7 +425,7 @@ std::unique_ptr<base::DictionaryValue> ConstructAboutInformation(
if (snapshot.legacy_updates_source() !=
sync_pb::GetUpdatesCallerInfo::UNKNOWN) {
session_source.SetValue(
- GetUpdatesSourceString(snapshot.legacy_updates_source()));
+ syncer::GetUpdatesSourceString(snapshot.legacy_updates_source()));
}
get_key_result.SetValue(GetSyncerErrorString(
snapshot.model_neutral_state().last_get_key_result));
@@ -474,8 +475,8 @@ std::unique_ptr<base::DictionaryValue> ConstructAboutInformation(
// full_status.sync_protocol_error is exported directly from the
// ProfileSyncService, even if the backend doesn't exist.
const bool actionable_error_detected =
- full_status.sync_protocol_error.error_type != UNKNOWN_ERROR &&
- full_status.sync_protocol_error.error_type != SYNC_SUCCESS;
+ full_status.sync_protocol_error.error_type != syncer::UNKNOWN_ERROR &&
+ full_status.sync_protocol_error.error_type != syncer::SYNC_SUCCESS;
about_info->SetBoolean("actionable_error_detected",
actionable_error_detected);
@@ -492,10 +493,10 @@ std::unique_ptr<base::DictionaryValue> ConstructAboutInformation(
StringSyncStat description(actionable_error, "Error Description");
if (actionable_error_detected) {
- error_type.SetValue(
- GetSyncErrorTypeString(full_status.sync_protocol_error.error_type));
+ error_type.SetValue(syncer::GetSyncErrorTypeString(
+ full_status.sync_protocol_error.error_type));
action.SetValue(
- GetClientActionString(full_status.sync_protocol_error.action));
+ syncer::GetClientActionString(full_status.sync_protocol_error.action));
url.SetValue(full_status.sync_protocol_error.url);
description.SetValue(full_status.sync_protocol_error.error_description);
}
@@ -521,4 +522,4 @@ std::unique_ptr<base::DictionaryValue> ConstructAboutInformation(
} // namespace sync_ui_util
-} // namespace syncer
+} // namespace sync_driver
« no previous file with comments | « components/sync/driver/about_sync_util.h ('k') | components/sync/driver/about_sync_util_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698