Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(577)

Unified Diff: chrome/browser/system_monitor/media_device_notifications_utils.h

Issue 11366144: [Media Gallery][ChromeOS] Improve device media gallery names. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed review comments Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..f9fbd6632391b830949fec7394262c4d08607a54 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,16 @@ namespace chrome {
// TODO(vandebo) Try to figure out how Mac OS X decides this.
bool IsMediaDevice(const FilePath::StringType& mount_point);
+// Constructs and returns the device product name from |vendor_name| and
Lei Zhang 2012/11/12 23:20:52 nit: you don't need to say "and returns", it's imp
kmadhusu 2012/11/13 01:03:58 Done.
+// |model_name|.
+std::string GetFullProductName(const std::string& vendor_name,
+ const std::string& model_name);
+
+// Constructs and returns a 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_

Powered by Google App Engine
This is Rietveld 408576698