Move All Delegate Calls Outside Layout Pass (#57)

- Moves a few more delegate calls or delegate triggering calls outside the protected area in `layoutLines` to prevent crashes. These are safe to call outside the intended protected area.
- Documents an important layout method.
- Fixes a flaking test by sorting selection ranges after deleting.

Author: thecoolwinter

Sources/CodeEditTextView/TextLayoutManager/TextLayoutManager.swift

@@ -266,6 +266,13 @@ public class TextLayoutManager: NSObject {
         // Update the visible lines with the new set.
         visibleLineIds = newVisibleLines
 
+        #if DEBUG
+        isInLayout = false
+        #endif
+
+        // These are fine to update outside of `isInLayout` as our internal data structures are finalized at this point
+        // so laying out again won't break our line storage or visible line.
+
         if maxFoundLineWidth > maxLineWidth {
             maxLineWidth = maxFoundLineWidth
         }
@@ -274,15 +281,11 @@ public class TextLayoutManager: NSObject {
             delegate?.layoutManagerYAdjustment(yContentAdjustment)
         }
 
-        #if DEBUG
-        isInLayout = false
-        #endif
-        needsLayout = false
-
-        // This needs to happen after ``needsLayout`` is toggled. Things can be triggered by frame changes.
         if originalHeight != lineStorage.height || layoutView?.frame.size.height != lineStorage.height {
             delegate?.layoutManagerHeightDidUpdate(newHeight: lineStorage.height)
         }
+
+        needsLayout = false
     }
 
     /// Lays out a single text line.

Sources/CodeEditTextView/TextView/TextView+Delete.swift

@@ -60,6 +60,7 @@ extension TextView {
             guard extendedRange.location >= 0 else { continue }
             textSelection.range.formUnion(extendedRange)
         }
+        selectionManager.textSelections.sort(by: { $0.range.location < $1.range.location })
         KillRing.shared.kill(
             strings: selectionManager.textSelections.map(\.range).compactMap({ textStorage.substring(from: $0) })
         )

Sources/CodeEditTextView/TextView/TextView+Layout.swift

@@ -58,6 +58,8 @@ extension TextView {
         inputContext?.invalidateCharacterCoordinates()
     }
 
+    /// Updates the view's frame if needed depending on wrapping lines, a new maximum width, or changed available size.
+    /// - Returns: Whether or not the view was updated.
     @discardableResult
     public func updateFrameIfNeeded() -> Bool {
         var availableSize = scrollView?.contentSize ?? .zero