Rewritten MountLibrary to work with non-blocking mount API calls in libcros....

chrome/browser/chromeos/usb_mount_observer.cc

 // Copyright (c) 2011 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/chromeos/usb_mount_observer.h"
 
 #include "base/command_line.h"
+#include "base/json/json_writer.h"
 #include "base/singleton.h"
+#include "chrome/browser/chromeos/cros/cros_library.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/webui/filebrowse_ui.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/url_constants.h"
 #include "content/browser/tab_contents/tab_contents.h"
 
+
 namespace chromeos {
 
 const char* kFilebrowseURLHash = "chrome://filebrowse#";
 const char* kFilebrowseScanning = "scanningdevice";
 const int kPopupLeft = 0;
 const int kPopupTop = 0;
 const int kPopupWidth = 250;
 const int kPopupHeight = 300;
 
 // static
@@ skipping 10 matching lines @@
     if (Source<Browser>(source).ptr() == i->browser) {
       i->browser = NULL;
       registrar_.Remove(this,
                         NotificationType::BROWSER_CLOSED,
                         source);
       return;
     }
   }
 }
 
-void USBMountObserver::ScanForDevices(chromeos::MountLibrary* obj) {
-  const chromeos::MountLibrary::DiskVector& disks = obj->disks();
-  for (size_t i = 0; i < disks.size(); ++i) {
-    chromeos::MountLibrary::Disk disk = disks[i];
-    if (!disk.is_parent && !disk.device_path.empty()) {
-      obj->MountPath(disk.device_path.c_str());
-    }
-  }
-}
-
 void USBMountObserver::OpenFileBrowse(const std::string& url,
                                       const std::string& device_path,
                                       bool small) {
   Browser* browser;
   Profile* profile;
   browser =  BrowserList::GetLastActive();
   if (browser == NULL) {
     return;
   }
   profile = browser->profile();
@@ skipping 19 matching lines @@
   if (iter == browsers_.end()) {
     BrowserWithPath new_browser;
     new_browser.browser = browser;
     new_browser.device_path = device_path;
     browsers_.push_back(new_browser);
   } else {
     iter->browser = browser;
   }
 }
 
-void USBMountObserver::MountChanged(chromeos::MountLibrary* obj,
-                                    chromeos::MountEventType evt,
-                                    const std::string& path) {
-  if (evt == chromeos::DISK_ADDED) {
-    const chromeos::MountLibrary::DiskVector& disks = obj->disks();
-    for (size_t i = 0; i < disks.size(); ++i) {
-      chromeos::MountLibrary::Disk disk = disks[i];
-      if (disk.device_path == path) {
-        if (disk.is_parent) {
-          if (!disk.has_media) {
-            RemoveBrowserFromVector(disk.system_path);
-          }
-        } else if (!obj->MountPath(path.c_str())) {
-          RemoveBrowserFromVector(disk.system_path);
-        }
-      }
+void USBMountObserver::DiskChanged(MountLibraryEventType event,
+                                   const MountLibrary::Disk* disk) {
+  if (event == MOUNT_DISK_ADDED) {
+     OnDiskAdded(disk);
+  } else if (event == MOUNT_DISK_REMOVED) {
+    OnDiskRemoved(disk);
+  } else if (event == MOUNT_DISK_CHANGED) {
+    OnDiskChanged(disk);
+  }
+}
+
+void USBMountObserver::DeviceChanged(MountLibraryEventType event,
+                                     const std::string& device_path) {
+  if (event == MOUNT_DEVICE_ADDED) {
+    OnDeviceAdded(device_path);
+  }else if (event == MOUNT_DEVICE_REMOVED) {
+    OnDeviceRemoved(device_path);
+  } else if (event == MOUNT_DEVICE_SCANNED) {
+    OnDeviceScanned(device_path);
+  }
+}
+
+void USBMountObserver::FireFileSystemChanged(
+    const std::string& web_path) {
+  // TODO(zelidrag): Send message to all extensions that file system has
+  // changed.
+  return;
+}
+
+void USBMountObserver::OnDiskAdded(const MountLibrary::Disk* disk) {
+  LOG(WARNING) << "Disk added: " << disk->device_path();

[stevenjb, 2011/03/16 22:52:55]

-> VLOG(1)

[zel, 2011/03/17 01:00:30]

Done.

+  if (disk->device_path().empty()) {
+    LOG(WARNING) << "Empty system path for " << disk->device_path();
+    return;
+  }
+  if (disk->is_parent()) {
+    if (!disk->has_media()) {
+      RemoveBrowserFromVector(disk->system_path());
     }

[stevenjb, 2011/03/16 22:52:55]

nit: unneeded {}

[zel, 2011/03/17 01:00:30]

Done.

-    VLOG(1) << "Got added mount: " << path;
-  } else if (evt == chromeos::DISK_REMOVED ||
-             evt == chromeos::DEVICE_REMOVED) {
-    RemoveBrowserFromVector(path);
-  } else if (evt == chromeos::DISK_CHANGED) {
-    BrowserIterator iter = FindBrowserForPath(path);
-    VLOG(1) << "Got changed mount: " << path;
-    if (iter == browsers_.end()) {
-      // We don't currently have this one, so it must have been
-      // mounted
-      const chromeos::MountLibrary::DiskVector& disks = obj->disks();
-      for (size_t i = 0; i < disks.size(); ++i) {
-        if (disks[i].device_path == path) {
-          if (!disks[i].mount_path.empty()) {
-            // Doing second search to see if the current disk has already
-            // been popped up due to its parent device being plugged in.
-            iter = FindBrowserForPath(disks[i].system_path);
-            if (iter != browsers_.end() && iter->browser) {
-              std::string url = kFilebrowseURLHash;
-              url += disks[i].mount_path;
-              TabContents* tab = iter->browser->GetSelectedTabContents();
-              iter->browser->window()->SetBounds(gfx::Rect(
-                  0, 0, FileBrowseUI::kPopupWidth, FileBrowseUI::kPopupHeight));
-              tab->OpenURL(GURL(url), GURL(), CURRENT_TAB,
-                  PageTransition::LINK);
-              tab->NavigateToPendingEntry(NavigationController::RELOAD);
-              iter->device_path = path;
-              iter->mount_path = disks[i].mount_path;
-            } else {
-              OpenFileBrowse(disks[i].mount_path, disks[i].device_path, false);
-            }
-          }
-          return;
-        }
-      }
+    return;
+  }
+
+  // If disk is not mounted yet, give it a try.
+  if (disk->mount_path().empty()) {
+    // Initiate disk mount operation.
+    chromeos::MountLibrary* lib =
+        chromeos::CrosLibrary::Get()->GetMountLibrary();
+    lib->MountPath(disk->device_path().c_str());
+  }
+}
+
+void USBMountObserver::OnDiskRemoved(const MountLibrary::Disk* disk) {
+  LOG(WARNING) << "Disk removed: " << disk->device_path();

[stevenjb, 2011/03/16 22:52:55]

VLOG(1)

[zel, 2011/03/17 01:00:30]

Done.

+  RemoveBrowserFromVector(disk->device_path());
+  MountPointMap::iterator iter = mounted_devices_.find(disk->device_path());
+  if (iter == mounted_devices_.end())
+    return;
+
+  chromeos::MountLibrary* lib =
+      chromeos::CrosLibrary::Get()->GetMountLibrary();
+  // TODO(zelidrag): This for some reason does not work as advertized.
+  // we might need to clean up mount directory on FILE thread here as well.
+  lib->UnmountPath(disk->device_path().c_str());
+
+  FireFileSystemChanged(iter->second);
+  mounted_devices_.erase(iter);
+}
+
+void USBMountObserver::OnDiskChanged(const MountLibrary::Disk* disk) {
+  LOG(WARNING) << "Disk changed : " << disk->device_path();

[stevenjb, 2011/03/16 22:52:55]

VLOG(1)

[zel, 2011/03/17 01:00:30]

Done.

+  if (!disk->mount_path().empty()) {
+    // Remember this mount point.
+    mounted_devices_.insert(
+        std::pair<std::string, std::string>(disk->device_path(),
+                                            disk->mount_path()));
+    FireFileSystemChanged(disk->mount_path());
+
+    // TODO(zelidrag): We should remove old file browser stuff later.
+    // Doing second search to see if the current disk has already
+    // been popped up due to its parent device being plugged in.
+    BrowserIterator iter = FindBrowserForPath(disk->system_path());
+    if (iter != browsers_.end() && iter->browser) {
+      std::string url = kFilebrowseURLHash;
+      url += disk->mount_path();
+      TabContents* tab = iter->browser->GetSelectedTabContents();
+      iter->browser->window()->SetBounds(gfx::Rect(
+          0, 0, FileBrowseUI::kPopupWidth, FileBrowseUI::kPopupHeight));
+      tab->OpenURL(GURL(url), GURL(), CURRENT_TAB,
+          PageTransition::LINK);
+      tab->NavigateToPendingEntry(NavigationController::RELOAD);
+      iter->device_path = disk->device_path();
+      iter->mount_path = disk->mount_path();
+    } else {
+      OpenFileBrowse(disk->mount_path(), disk->device_path(), false);
     }
-  } else if (evt == chromeos::DEVICE_ADDED) {
-    VLOG(1) << "Got device added: " << path;
-    OpenFileBrowse(kFilebrowseScanning, path, true);
-  } else if (evt == chromeos::DEVICE_SCANNED) {
-    VLOG(1) << "Got device scanned: " << path;
   }
 }
 
+void USBMountObserver::OnDeviceAdded(const std::string& device_path) {
+  LOG(WARNING) << "Device added : " << device_path;

[stevenjb, 2011/03/16 22:52:55]

VLOG(1)

[zel, 2011/03/17 01:00:30]

Done.

+  OpenFileBrowse(kFilebrowseScanning, device_path, true);
+}
+
+void USBMountObserver::OnDeviceRemoved(const std::string& device_path) {
+  // New device is added, initiate disk rescan.
+  RemoveBrowserFromVector(device_path);
+}
+
+void USBMountObserver::OnDeviceScanned(const std::string& device_path) {
+  LOG(WARNING) << "Device scanned : " << device_path;

[stevenjb, 2011/03/16 22:52:55]

VLOG(1)

[zel, 2011/03/17 01:00:30]

Done.

+}
+
 USBMountObserver::BrowserIterator USBMountObserver::FindBrowserForPath(
     const std::string& path) {
   for (BrowserIterator i = browsers_.begin();i != browsers_.end();
        ++i) {
     // Doing a substring match so that we find if this new one is a subdevice
     // of another already inserted device.
     if (path.find(i->device_path) != std::string::npos) {
       return i;
     }
   }
@@ skipping 27 matching lines @@
     }
   }
   for (size_t x = 0; x < close_these.size(); x++) {
     if (close_these[x]->window()) {
       close_these[x]->window()->Close();
     }
   }
 }
 
 } // namespace chromeos