From 7e496b1979e9891d2d0c987f7ca9818b4e99d5de Mon Sep 17 00:00:00 2001 From: "Jasper St. Pierre" Date: Wed, 19 Sep 2012 01:38:26 -0300 Subject: [PATCH] autorunManager: Clean up mount operations Add a processMount function that's shared between mount scanning and hotplug. https://bugzilla.gnome.org/show_bug.cgi?id=684093 --- js/ui/components/autorunManager.js | 30 ++++++++++++++---------------- 1 file changed, 14 insertions(+), 16 deletions(-) diff --git a/js/ui/components/autorunManager.js b/js/ui/components/autorunManager.js index 541c75ca7..942576ad3 100644 --- a/js/ui/components/autorunManager.js +++ b/js/ui/components/autorunManager.js @@ -175,16 +175,21 @@ const AutorunManager = new Lang.Class({ this._volumeMonitor.disconnect(this._mountRemovedId); }, + _processMount: function(mount, hotplug) { + let discoverer = new ContentTypeDiscoverer(Lang.bind(this, function(mount, apps, contentTypes) { + this._ensureResidentSource(); + this._residentSource.addMount(mount, apps); + + if (hotplug) + this._transDispatcher.addMount(mount, apps, contentTypes); + })); + discoverer.guessContentTypes(mount); + }, + _scanMounts: function() { let mounts = this._volumeMonitor.get_mounts(); - mounts.forEach(Lang.bind(this, function (mount) { - let discoverer = new ContentTypeDiscoverer(Lang.bind (this, - function (mount, apps) { - this._ensureResidentSource(); - this._residentSource.addMount(mount, apps); - })); - - discoverer.guessContentTypes(mount); + mounts.forEach(Lang.bind(this, function(mount) { + this._processMount(mount, false); })); }, @@ -194,14 +199,7 @@ const AutorunManager = new Lang.Class({ if (!this._loginManager.sessionActive) return; - let discoverer = new ContentTypeDiscoverer(Lang.bind (this, - function (mount, apps, contentTypes) { - this._transDispatcher.addMount(mount, apps, contentTypes); - this._ensureResidentSource(); - this._residentSource.addMount(mount, apps); - })); - - discoverer.guessContentTypes(mount); + this._processMount(mount, true); }, _onMountRemoved: function(monitor, mount) {