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

Unified Diff: chrome/browser/sync/glue/device_info.cc

Issue 22706006: Implementation of the DeviceInfo get API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: For try runs Created 7 years, 4 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/sync/glue/device_info.cc
diff --git a/chrome/browser/sync/glue/device_info.cc b/chrome/browser/sync/glue/device_info.cc
index 486c8bb5e2752d906686ba65e954f2db114ee35f..dc2119032a61957ead6b3e7a66536a9d7ab6b3ed 100644
--- a/chrome/browser/sync/glue/device_info.cc
+++ b/chrome/browser/sync/glue/device_info.cc
@@ -44,22 +44,36 @@ std::string ChannelToString(chrome::VersionInfo::Channel channel) {
std::string DeviceTypeToString(sync_pb::SyncEnums::DeviceType device_type) {
switch (device_type) {
case sync_pb::SyncEnums_DeviceType_TYPE_WIN:
- return "WIN";
case sync_pb::SyncEnums_DeviceType_TYPE_MAC:
- return "MAC";
case sync_pb::SyncEnums_DeviceType_TYPE_LINUX:
- return "LINUX";
case sync_pb::SyncEnums_DeviceType_TYPE_CROS:
- return "CHROME OS";
- case sync_pb::SyncEnums_DeviceType_TYPE_OTHER:
- return "OTHER";
+ return "desktop_or_laptop";
case sync_pb::SyncEnums_DeviceType_TYPE_PHONE:
- return "PHONE";
+ return "phone";
case sync_pb::SyncEnums_DeviceType_TYPE_TABLET:
- return "TABLET";
+ return "tablet";
default:
- NOTREACHED();
- return "UNKNOWN";
+ return "unknown";
+ }
+}
+
+std::string GetOS(sync_pb::SyncEnums::DeviceType device_type) {
+ switch (device_type) {
+ case sync_pb::SyncEnums_DeviceType_TYPE_WIN:
+ return "win";
+ case sync_pb::SyncEnums_DeviceType_TYPE_MAC:
+ return "mac";
+ case sync_pb::SyncEnums_DeviceType_TYPE_LINUX:
+ return "linux";
+ case sync_pb::SyncEnums_DeviceType_TYPE_CROS:
+ return "chrome_os";
+ case sync_pb::SyncEnums_DeviceType_TYPE_PHONE:
+ case sync_pb::SyncEnums_DeviceType_TYPE_TABLET:
+ // TODO(lipalani): crbug.com/170375. Add support for ios
+ // phones and tablets.
+ return "android";
+ default:
+ return "unknown";
}
}
@@ -172,10 +186,10 @@ std::string DeviceInfo::MakeUserAgentForSyncApi(
base::DictionaryValue* DeviceInfo::ToValue() {
base::DictionaryValue* value = new base::DictionaryValue();
- value->SetString("Id", public_id_);
- value->SetString("Client Name", client_name_);
- value->SetString("Chrome Version", chrome_version_);
- value->SetString("Sync User Agent", sync_user_agent_);
+ value->SetString("id", public_id_);
+ value->SetString("name", client_name_);
+ value->SetString("chromeVersion", chrome_version_);
+ value->SetString("os", GetOS(device_type_));
value->SetString("Device Type", DeviceTypeToString(device_type_));
return value;
}

Powered by Google App Engine
This is Rietveld 408576698