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

Unified Diff: chrome/browser/storage_monitor/media_device_notifications_utils.cc

Issue 12544005: Consolidate storage_monitor/MediaDeviceNotificationsUtils into MediaStorageUtil. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix mac apply patch Created 7 years, 9 months 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/storage_monitor/media_device_notifications_utils.cc
diff --git a/chrome/browser/storage_monitor/media_device_notifications_utils.cc b/chrome/browser/storage_monitor/media_device_notifications_utils.cc
deleted file mode 100644
index 67392d926e26048d027390c4c7eb495bab0ffaac..0000000000000000000000000000000000000000
--- a/chrome/browser/storage_monitor/media_device_notifications_utils.cc
+++ /dev/null
@@ -1,54 +0,0 @@
-// 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.
-
-#include "chrome/browser/storage_monitor/media_device_notifications_utils.h"
-
-#include "base/file_util.h"
-#include "base/string_util.h"
-#include "base/utf_string_conversions.h"
-#include "chrome/browser/storage_monitor/removable_device_constants.h"
-#include "content/public/browser/browser_thread.h"
-#include "ui/base/text/bytes_formatting.h"
-
-namespace chrome {
-
-bool IsMediaDevice(const base::FilePath::StringType& mount_point) {
- DCHECK(!content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
-
- base::FilePath dcim_path(mount_point);
- base::FilePath::StringType dcim_dir(kDCIMDirectoryName);
- if (!file_util::DirectoryExists(dcim_path.Append(dcim_dir))) {
- // Check for lowercase 'dcim' as well.
- base::FilePath dcim_path_lower(
- dcim_path.Append(StringToLowerASCII(dcim_dir)));
- if (!file_util::DirectoryExists(dcim_path_lower))
- return false;
- }
- return true;
-}
-
-string16 GetFullProductName(const std::string& vendor_name,
- const std::string& model_name) {
- if (vendor_name.empty() && model_name.empty())
- return string16();
-
- std::string product_name;
- if (vendor_name.empty())
- product_name = model_name;
- else if (model_name.empty())
- product_name = vendor_name;
- else
- product_name = vendor_name + ", " + model_name;
- return IsStringUTF8(product_name) ?
- UTF8ToUTF16("(" + product_name + ")") : string16();
-}
-
-string16 GetDisplayNameForDevice(uint64 storage_size_in_bytes,
- const string16& name) {
- DCHECK(!name.empty());
- return (storage_size_in_bytes == 0) ?
- name : ui::FormatBytes(storage_size_in_bytes) + ASCIIToUTF16(" ") + name;
-}
-
-} // namespace chrome

Powered by Google App Engine
This is Rietveld 408576698