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; |
} |