[gnome-shell/hotplug: 6/14] automount: only autorun volumes marked as allowed

[Cosimo Cecchi <cosimoc src gnome org>]

commit bb782a69ba1f538b1da652293ca6e819f71d7660
Author: Cosimo Cecchi <cosimoc gnome org>
Date:   Wed Jun 22 09:45:03 2011 -0400

    automount: only autorun volumes marked as allowed
    
    Port code from g-s-d to mark volumes as allowed for autorun, and check
    for it when running the notification.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=653520

 js/ui/automountManager.js |   35 +++++++++++++++++++++++++++--------
 js/ui/autorunManager.js   |    2 +-
 2 files changed, 28 insertions(+), 9 deletions(-)
---
diff --git a/js/ui/automountManager.js b/js/ui/automountManager.js
index d6dcadc..337eb9d 100644
--- a/js/ui/automountManager.js
+++ b/js/ui/automountManager.js
@@ -200,13 +200,6 @@ AutomountManager.prototype = {
     },
 
     _checkAndMountVolume: function(volume) {
-        if (!this._settings.get_boolean(SETTING_ENABLE_AUTOMOUNT))
-            return;
-
-        if (!volume.should_automount() ||
-            !volume.can_mount())
-            return;
-
         // if we're not in the current ConsoleKit session,
         // don't attempt automount
         if (!this.ckListener.sessionActive)
@@ -219,6 +212,19 @@ AutomountManager.prototype = {
             return;
         }
 
+        if (!this._settings.get_boolean(SETTING_ENABLE_AUTOMOUNT) ||
+            !volume.should_automount() ||
+            !volume.can_mount()) {
+	    // allow the autorun to run anyway; this can happen if the
+            // mount gets added programmatically later, even if 
+            // should_automount() or can_mount() are false, like for
+            // blank optical media.
+            this._allowAutorun(volume);
+            this._allowAutorunExpire(volume);
+
+            return;
+        }
+
         // TODO: mount op
         this._mountVolume(volume, null);
     },
@@ -229,6 +235,8 @@ AutomountManager.prototype = {
     },
 
     _onVolumeMounted: function(volume, res) {
+        this._allowAutorunExpire(volume);
+
         try {
             volume.mount_finish(res);
         } catch (e) {
@@ -243,5 +251,16 @@ AutomountManager.prototype = {
             this._volumeQueue.filter(function(element) {
                 return (element != volume);
             });
+    },
+
+    _allowAutorun: function(volume) {
+        volume.allowAutorun = true;
+    },
+
+    _allowAutorunExpire: function(volume) {
+        Mainloop.timeout_add_seconds(10, function() {
+            volume.allowAutorun = false;
+            return false;
+        });
     }
-}
\ No newline at end of file
+}
diff --git a/js/ui/autorunManager.js b/js/ui/autorunManager.js
index 39ac9f3..224a14d 100644
--- a/js/ui/autorunManager.js
+++ b/js/ui/autorunManager.js
@@ -23,7 +23,7 @@ function ignoreAutorunForMount(mount) {
     let volume = mount.get_volume();
 
     if ((root.is_native() && !isMountRootHidden(root)) ||
-        (volume && volume.should_automount()))
+        (volume && volume.allowAutorun && volume.should_automount()))
         return false;
 
     return true;