Add DeviceTypeAsString to SyncedSession.

chrome/browser/sync/glue/synced_session.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_SYNC_GLUE_SYNCED_SESSION_H_
 #define CHROME_BROWSER_SYNC_GLUE_SYNCED_SESSION_H_
 
 #include <map>
 #include <string>
 
@@ skipping 75 matching lines @@
   // Type of device this session is from.
   DeviceType device_type;
 
   // Last time this session was modified remotely.
   base::Time modified_time;
 
   // Map of windows that make up this session. Windowws are owned by the session
   // itself and free'd on destruction.
   SyncedWindowMap windows;
 
+  // Converts the DeviceType enum value to a human readable string. This is used
+  // in the NTP handler for foreign sessions.
+  std::string DeviceTypeAsString() const {
+    switch (device_type) {
+      case SyncedSession::TYPE_WIN:
+        return "win";

[Nicolas Zea, 2012/07/24 18:44:24]

Not sure in what context these will be displayed, but should they have the first
letter capitalized?

[nyquist, 2012/07/24 20:18:38]

They are used for matching an icon style on the NTP towards laptop/phone/tablet,
and not visible other than in the NTP source code. I have clarified the comment.

+      case SyncedSession::TYPE_MACOSX:
+        return "macosx";
+      case SyncedSession::TYPE_LINUX:
+        return "linux";
+      case SyncedSession::TYPE_CHROMEOS:
+        return "chromeos";
+      case SyncedSession::TYPE_OTHER:
+        return "other";
+      case SyncedSession::TYPE_PHONE:
+        return "phone";
+      case SyncedSession::TYPE_TABLET:
+        return "tablet";
+      default:
+        return "";
+    }
+  }
+
  private:
   DISALLOW_COPY_AND_ASSIGN(SyncedSession);
 };
 
 // Control which foreign tabs we're interested in syncing/displaying. Checks
 // that the tab has navigations and contains at least one valid url.
 // Note: chrome:// and file:// are not considered valid urls (for syncing).
 bool ShouldSyncSessionTab(const SessionTab& tab);
 
 // Checks whether the window has tabs to sync. If no tabs to sync, it returns
 // true, false otherwise.
 bool SessionWindowHasNoTabsToSync(const SessionWindow& window);
 
 }  // namespace browser_sync
 
 #endif  // CHROME_BROWSER_SYNC_GLUE_SYNCED_SESSION_H_