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

Unified Diff: chrome/browser/media_gallery/removable_device_notifications_linux.h

Issue 10919051: Move device notification impl out of chrome/browser/media_gallery (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: modify for chromeos and win Created 8 years, 3 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/media_gallery/removable_device_notifications_linux.h
diff --git a/chrome/browser/media_gallery/removable_device_notifications_linux.h b/chrome/browser/media_gallery/removable_device_notifications_linux.h
deleted file mode 100644
index 241ff928ae0c04217789056ae5c1a81d50e365e0..0000000000000000000000000000000000000000
--- a/chrome/browser/media_gallery/removable_device_notifications_linux.h
+++ /dev/null
@@ -1,143 +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.
-
-// RemovableDeviceNotificationsLinux listens for mount point changes, notifies
-// the SystemMonitor about the addition and deletion of media devices, and
-// answers queries about mounted devices.
-
-#ifndef CHROME_BROWSER_MEDIA_GALLERY_REMOVABLE_DEVICE_NOTIFICATIONS_LINUX_H_
-#define CHROME_BROWSER_MEDIA_GALLERY_REMOVABLE_DEVICE_NOTIFICATIONS_LINUX_H_
-
-#if defined(OS_CHROMEOS)
-#error "Use the ChromeOS-specific implementation instead."
-#endif
-
-#include <map>
-#include <set>
-#include <string>
-#include <utility>
-
-#include "base/basictypes.h"
-#include "base/compiler_specific.h"
-#include "base/files/file_path_watcher.h"
-#include "base/memory/ref_counted.h"
-#include "base/system_monitor/system_monitor.h"
-#include "content/public/browser/browser_thread.h"
-
-class FilePath;
-
-// Gets device information given a |device_path|. On success, returns true and
-// fills in |unique_id|, |name|, and |removable|.
-typedef bool (*GetDeviceInfoFunc)(const FilePath& device_path,
- std::string* unique_id, string16* name,
- bool* removable);
-
-namespace chrome {
-
-class RemovableDeviceNotificationsLinux
- : public base::RefCountedThreadSafe<RemovableDeviceNotificationsLinux,
- content::BrowserThread::DeleteOnFileThread> {
- public:
- // Should only be called by browser start up code. Use GetInstance() instead.
- explicit RemovableDeviceNotificationsLinux(const FilePath& path);
-
- static RemovableDeviceNotificationsLinux* GetInstance();
-
- // Must be called for RemovableDeviceNotificationsLinux to work.
- void Init();
-
- // Use |device_id| to find and return where the device is mounted.
- FilePath GetDeviceMountPoint(const std::string& device_id) const;
-
- // Finds the device that contains |path| and populates |device_info|.
- // Returns false if unable to find the device.
- bool GetDeviceInfoForPath(
- const FilePath& path,
- base::SystemMonitor::RemovableStorageInfo* device_info) const;
-
- protected:
- // Only for use in unit tests.
- RemovableDeviceNotificationsLinux(const FilePath& path,
- GetDeviceInfoFunc getDeviceInfo);
-
- // Avoids code deleting the object while there are references to it.
- // Aside from the base::RefCountedThreadSafe friend class, and derived
- // classes, any attempts to call this dtor will result in a compile-time
- // error.
- virtual ~RemovableDeviceNotificationsLinux();
-
- virtual void OnFilePathChanged(const FilePath& path, bool error);
-
- private:
- friend class base::RefCountedThreadSafe<RemovableDeviceNotificationsLinux>;
- friend class base::DeleteHelper<RemovableDeviceNotificationsLinux>;
- friend struct content::BrowserThread::DeleteOnThread<
- content::BrowserThread::FILE>;
-
- // Structure to save mounted device information such as device path and unique
- // identifier.
- struct MountPointInfo {
- MountPointInfo();
-
- FilePath mount_device;
- std::string device_id;
- string16 device_name;
- };
-
- // Mapping of mount points to MountPointInfo.
- typedef std::map<FilePath, MountPointInfo> MountMap;
-
- // (mount point, priority)
- // For devices that are mounted to multiple mount points, this helps us track
- // which one we've notified system monitor about.
- typedef std::map<FilePath, bool> ReferencedMountPoint;
-
- // (mount device, map of known mount points)
- // For each mount device, track the places it is mounted and which one (if
- // any) we have notified system monitor about.
- typedef std::map<FilePath, ReferencedMountPoint> MountPriorityMap;
-
- // Do initialization on the File Thread.
- void InitOnFileThread();
-
- // Parses mtab file and find all changes.
- void UpdateMtab();
-
- // Adds |mount_device| as mounted on |mount_point|. If the device is a new
- // device SystemMonitor is notified.
- void AddNewMount(const FilePath& mount_device, const FilePath& mount_point);
-
- // Whether Init() has been called or not.
- bool initialized_;
-
- // Mtab file that lists the mount points.
- const FilePath mtab_path_;
-
- // Watcher for |mtab_path_|.
- base::files::FilePathWatcher file_watcher_;
-
- // Set of known file systems that we care about.
- std::set<std::string> known_file_systems_;
-
- // Function handler to get device information. This is useful to set a mock
- // handler for unit testing.
- GetDeviceInfoFunc get_device_info_func_;
-
- // Mapping of relevant mount points and their corresponding mount devices.
- // Keep in mind on Linux, a device can be mounted at multiple mount points,
- // and multiple devices can be mounted at a mount point.
- MountMap mount_info_map_;
-
- // Because a device can be mounted to multiple places, we only want to
- // notify about one of them. If (and only if) that one is unmounted, we need
- // to notify about it's departure and notify about another one of it's mount
- // points.
- MountPriorityMap mount_priority_map_;
-
- DISALLOW_COPY_AND_ASSIGN(RemovableDeviceNotificationsLinux);
-};
-
-} // namespace chrome
-
-#endif // CHROME_BROWSER_MEDIA_GALLERY_REMOVABLE_DEVICE_NOTIFICATIONS_LINUX_H_

Powered by Google App Engine
This is Rietveld 408576698