Review of attachment 170791 [details] [review]:

Looks pretty good - I have a few nitpicks.

::: src/Core/Banshee.Services/Banshee.Networking/DotNetNetworking.cs
@@ +35,3 @@
+    public class DotNetNetworking : INetworkAvailabilityService
+    {
+        public static bool Available { get { try { return Type.GetType ("Mono.Runtime") == null; } catch { return true; } } }

Might as well call this ManagerPresent like the others do, no?

@@ +40,3 @@
+
+        public State State
+        {

put this { up on the previous line (for properties)

@@ +59,3 @@
+        void OnNetworkAvailabilityChanged (object o, NetworkAvailabilityEventArgs args)
+        {
+            StateChangeHandler sc = StateChange;

rewrite this as
var handler = StateChanged;
if (handler != null) {
...
}

Created attachment 170850 [details] [review]
Updated patch with fixes

Updated based on feedback.

Review of attachment 170850 [details] [review]:

Committed, thanks!