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

chrome/browser/system_monitor/media_storage_util.cc

Index: chrome/browser/system_monitor/media_storage_util.cc
diff --git a/chrome/browser/system_monitor/media_storage_util.cc b/chrome/browser/system_monitor/media_storage_util.cc
index e7f39cdc5bd66f4605835222ed0ac442caed5a2d..54535aa607229406f0e3aac42c1ba14d2fadf745 100644
--- a/chrome/browser/system_monitor/media_storage_util.cc
+++ b/chrome/browser/system_monitor/media_storage_util.cc
@@ -254,22 +254,30 @@ bool MediaStorageUtil::GetDeviceInfoFromPath(const FilePath& path,
 #endif
 
   if (found_device && IsRemovableDevice(device_info.device_id)) {
-    if (device_id)
-      *device_id = device_info.device_id;
-    if (device_name)
-      *device_name = device_info.name;
-    if (relative_path) {
-      *relative_path = FilePath();
-      FilePath mount_point(device_info.location);
-      bool success = mount_point.AppendRelativePath(path, relative_path);
-#if defined(OS_POSIX)
-      if (!relative_path->value().empty() && device_name) {
+    FilePath sub_folder_path;
+    if (device_name || relative_path) {
+      bool success = FilePath(device_info.location)
+          .AppendRelativePath(path, &sub_folder_path);
+      DCHECK(success);
+    }
+
+#if defined(OS_CHROMEOS)
+    if (device_name) {
+      *device_name = notifier->GetStorageSizeInfo(device_info.location) +

[Lei Zhang, 2012/11/08 07:42:44]

Consider localizing this as well?

Is this going to stay CrOS only?

[kmadhusu, 2012/11/08 18:50:35]

Modified code to use FormatBytes() and it localizes the byte block units.
No. I will update windows, linux and mac code in a separate CL.

+          ASCIIToUTF16(" ");
+      if (!sub_folder_path.value().empty()) {
         *device_name += ASCIIToUTF16(" (") +
-            relative_path->BaseName().LossyDisplayName() + ASCIIToUTF16(")");
+            sub_folder_path.BaseName().LossyDisplayName() + ASCIIToUTF16(") ");
       }
-#endif
-      DCHECK(success);
     }
+#endif
+
+    if (device_id)
+      *device_id = device_info.device_id;
+    if (device_name)
+      *device_name += device_info.name;
+    if (relative_path)
+      *relative_path = sub_folder_path;
     return true;
   }