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

chrome/browser/system_monitor/media_device_notifications_utils.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.
 
 // Common code shared between MediaDeviceNotifications{Win,Linux}.
 
 #ifndef CHROME_BROWSER_SYSTEM_MONITOR_MEDIA_DEVICE_NOTIFICATIONS_UTILS_H_
 #define CHROME_BROWSER_SYSTEM_MONITOR_MEDIA_DEVICE_NOTIFICATIONS_UTILS_H_
 
 #include "base/file_path.h"
 
 namespace chrome {
 
 // Check if the file system at the passed mount point looks like a media
 // device using the existence of DCIM directory.
 // Returns true, if it is a media device otherwise return false.
 // Mac OS X behaves similarly, but this is not the only heuristic it uses.
 // TODO(vandebo) Try to figure out how Mac OS X decides this.
 bool IsMediaDevice(const FilePath::StringType& mount_point);
 
+// Constructs and returns a device manufacturer name from |vendor_name|
+// and |model_name|.
+std::string GetDeviceManufacturerName(const std::string& vendor_name,

[Lei Zhang, 2012/11/12 07:46:56]

This name is confusing. For computer products, "manufacturer name" means the
same thing as "vendor name". How about GetFullProductName instead?

[kmadhusu, 2012/11/12 19:56:56]

Done.

+                                      const std::string& model_name);
+
+// Constructs and returns a display name for device from |device_partition_size|
+// and |name|.
+string16 GetDisplayNameForDevice(const string16& device_partition_size,

[Lei Zhang, 2012/11/12 07:46:56]

You need to #include string16.h in the header.

[kmadhusu, 2012/11/12 19:56:56]

Done.

+                                 const string16& name);
+
 }  // namespace chrome
 
 #endif  // CHROME_BROWSER_SYSTEM_MONITOR_MEDIA_DEVICE_NOTIFICATIONS_UTILS_H_