Index: chrome/browser/sync/glue/session_model_associator.cc |
diff --git a/chrome/browser/sync/glue/session_model_associator.cc b/chrome/browser/sync/glue/session_model_associator.cc |
index cbc143e9e1c084532579471d7d8e86dd96ab0232..739e1379a11209b602443c8b1c1714fa43276ca3 100644 |
--- a/chrome/browser/sync/glue/session_model_associator.cc |
+++ b/chrome/browser/sync/glue/session_model_associator.cc |
@@ -18,6 +18,7 @@ |
#include "chrome/browser/prefs/pref_service.h" |
#include "chrome/browser/profiles/profile.h" |
#include "chrome/browser/sessions/session_id.h" |
+#include "chrome/browser/sync/glue/device_info.h" |
#include "chrome/browser/sync/glue/synced_session.h" |
#include "chrome/browser/sync/glue/synced_tab_delegate.h" |
#include "chrome/browser/sync/glue/synced_window_delegate.h" |
@@ -70,32 +71,6 @@ static const int kMaxSyncNavigationCount = 6; |
// stale and becomes a candidate for garbage collection. |
static const size_t kDefaultStaleSessionThresholdDays = 14; // 2 weeks. |
-#if defined(OS_ANDROID) |
-bool IsTabletUI() { |
- return CommandLine::ForCurrentProcess()->HasSwitch(switches::kTabletUI); |
-} |
-#endif |
- |
-sync_pb::SessionHeader::DeviceType GetLocalDeviceType() { |
- // TODO(yfriedman): Refactor/combine with "DeviceInformation" code in |
- // sync_manager.cc[1060] |
-#if defined(OS_CHROMEOS) |
- return sync_pb::SessionHeader_DeviceType_TYPE_CROS; |
-#elif defined(OS_LINUX) |
- return sync_pb::SessionHeader_DeviceType_TYPE_LINUX; |
-#elif defined(OS_MACOSX) |
- return sync_pb::SessionHeader_DeviceType_TYPE_MAC; |
-#elif defined(OS_WIN) |
- return sync_pb::SessionHeader_DeviceType_TYPE_WIN; |
-#elif defined(OS_ANDROID) |
- return IsTabletUI() ? |
- sync_pb::SessionHeader_DeviceType_TYPE_TABLET : |
- sync_pb::SessionHeader_DeviceType_TYPE_PHONE; |
-#else |
- return sync_pb::SessionHeader_DeviceType_TYPE_OTHER; |
-#endif |
-} |
- |
} // namespace |
SessionModelAssociator::SessionModelAssociator(ProfileSyncService* sync_service, |
@@ -203,7 +178,7 @@ bool SessionModelAssociator::AssociateWindows(bool reload_tabs, |
synced_session_tracker_.GetSession(local_tag); |
current_session->modified_time = base::Time::Now(); |
header_s->set_client_name(current_session_name_); |
- header_s->set_device_type(GetLocalDeviceType()); |
+ header_s->set_device_type(DeviceInfo::GetLocalDeviceType()); |
synced_session_tracker_.ResetSessionTracking(local_tag); |
std::set<SyncedWindowDelegate*> windows = |
@@ -661,7 +636,7 @@ syncer::SyncError SessionModelAssociator::AssociateModels() { |
base_specifics.set_session_tag(current_machine_tag_); |
sync_pb::SessionHeader* header_s = base_specifics.mutable_header(); |
header_s->set_client_name(current_session_name_); |
- header_s->set_device_type(GetLocalDeviceType()); |
+ header_s->set_device_type(DeviceInfo::GetLocalDeviceType()); |
write_node.SetSessionSpecifics(base_specifics); |
local_session_syncid_ = write_node.GetId(); |
@@ -987,25 +962,25 @@ void SessionModelAssociator::PopulateSessionHeaderFromSpecifics( |
} |
if (header_specifics.has_device_type()) { |
switch (header_specifics.device_type()) { |
- case sync_pb::SessionHeader_DeviceType_TYPE_WIN: |
+ case sync_pb::SyncEnums_DeviceType_TYPE_WIN: |
session_header->device_type = SyncedSession::TYPE_WIN; |
break; |
- case sync_pb::SessionHeader_DeviceType_TYPE_MAC: |
+ case sync_pb::SyncEnums_DeviceType_TYPE_MAC: |
session_header->device_type = SyncedSession::TYPE_MACOSX; |
break; |
- case sync_pb::SessionHeader_DeviceType_TYPE_LINUX: |
+ case sync_pb::SyncEnums_DeviceType_TYPE_LINUX: |
session_header->device_type = SyncedSession::TYPE_LINUX; |
break; |
- case sync_pb::SessionHeader_DeviceType_TYPE_CROS: |
+ case sync_pb::SyncEnums_DeviceType_TYPE_CROS: |
session_header->device_type = SyncedSession::TYPE_CHROMEOS; |
break; |
- case sync_pb::SessionHeader_DeviceType_TYPE_PHONE: |
+ case sync_pb::SyncEnums_DeviceType_TYPE_PHONE: |
session_header->device_type = SyncedSession::TYPE_PHONE; |
break; |
- case sync_pb::SessionHeader_DeviceType_TYPE_TABLET: |
+ case sync_pb::SyncEnums_DeviceType_TYPE_TABLET: |
session_header->device_type = SyncedSession::TYPE_TABLET; |
break; |
- case sync_pb::SessionHeader_DeviceType_TYPE_OTHER: |
+ case sync_pb::SyncEnums_DeviceType_TYPE_OTHER: |
// Intentionally fall-through |
default: |
session_header->device_type = SyncedSession::TYPE_OTHER; |