Revert "Separate logic for starting network monitor"

This reverts commit e60953e.

Author: itsmeichigo

WooCommerce/Classes/Tools/Connectivity/ConnectivityObserver.swift

@@ -3,11 +3,8 @@ import Foundation
 /// Interface for the observing connectivity
 ///
 protocol ConnectivityObserver {
-    /// Starts the connectivity observer.
-    func startObserving()
-
-    /// Updates callback block for when connectivity changes.
-    func updateListener(_ listener: @escaping (ConnectivityStatus) -> Void)
+    /// Starts the observer with a listener.
+    func startObserving(listener: @escaping (ConnectivityStatus) -> Void)
 
     /// Stops the observer.
     func stopObserving()

WooCommerce/Classes/Tools/Connectivity/DefaultConnectivityObserver.swift

@@ -12,21 +12,15 @@ final class DefaultConnectivityObserver: ConnectivityObserver {
         self.networkMonitor = networkMonitor
     }
 
-    func startObserving() {
-        networkMonitor.start(queue: observingQueue)
-    }
-
-    func updateListener(_ listener: @escaping (ConnectivityStatus) -> Void) {
-        if networkMonitor.pathUpdateHandler == nil {
-            startObserving()
-        }
+    func startObserving(listener: @escaping (ConnectivityStatus) -> Void) {
         networkMonitor.pathUpdateHandler = { [weak self] path in
             guard let self = self else { return }
             let connectivityStatus = self.connectivityStatus(from: path)
             DispatchQueue.main.async {
                 listener(connectivityStatus)
             }
         }
+        networkMonitor.start(queue: observingQueue)
     }
 
     func stopObserving() {