[Media Gallery][ChromeOS] Improve device media gallery names.

chrome/browser/system_monitor/media_device_notifications_utils.h

Index: chrome/browser/system_monitor/media_device_notifications_utils.h
diff --git a/chrome/browser/system_monitor/media_device_notifications_utils.h b/chrome/browser/system_monitor/media_device_notifications_utils.h
index 46980b185b9e1d0a32d3f37133a3060f3dacaff0..fad77e3bcd8f66dc0fdbef5ac61e264476170bff 100644
--- a/chrome/browser/system_monitor/media_device_notifications_utils.h
+++ b/chrome/browser/system_monitor/media_device_notifications_utils.h
@@ -8,6 +8,7 @@
 #define CHROME_BROWSER_SYSTEM_MONITOR_MEDIA_DEVICE_NOTIFICATIONS_UTILS_H_
 
 #include "base/file_path.h"
+#include "base/string16.h"
 
 namespace chrome {
 
@@ -18,6 +19,15 @@ namespace chrome {
 // TODO(vandebo) Try to figure out how Mac OS X decides this.
 bool IsMediaDevice(const FilePath::StringType& mount_point);
 
+// Constructs the device product name from |vendor_name| and |model_name|.
+std::string GetFullProductName(const std::string& vendor_name,

[Lei Zhang, 2012/11/13 01:13:07]

Also, have you considered moving the IsStringUTF8() call into this function as
well? That way callers do no have to worry if the returned string is UTF-8 or
not.

[kmadhusu, 2012/11/13 01:37:32]

As we discussed, all the callers of this function will eventually convert the
result of this function to string16. Therefore, modified this function to return
a string16.

+                               const std::string& model_name);
+
+// Constructs the display name for device from |storage_size_in_bytes| and
+// |name|.
+string16 GetDisplayNameForDevice(uint64 storage_size_in_bytes,
+                                 const string16& name);
+
 }  // namespace chrome
 
 #endif  // CHROME_BROWSER_SYSTEM_MONITOR_MEDIA_DEVICE_NOTIFICATIONS_UTILS_H_