Merge pull request #510 from tonyhallett/remove-event-handler

remove event handler

Author: tonyhallett

FineCodeCoverageTests/Editor/DynamicCoverage/BufferLineCoverage_Tests.cs

@@ -111,7 +111,7 @@ public void Should_Stop_Listening_When_TextView_Closed()
 
             autoMoqer.Verify<IEventAggregator>(eventAggregator => eventAggregator.RemoveListener(bufferLineCoverage));
             mockTextView.VerifyRemove(textView => textView.Closed -= It.IsAny<EventHandler>(), Times.Once);
-            mockTextBuffer.VerifyRemove(textBuffer => textBuffer.Changed -= It.IsAny<EventHandler<TextContentChangedEventArgs>>(), Times.Once);
+            mockTextBuffer.VerifyRemove(textBuffer => textBuffer.ChangedOnBackground -= It.IsAny<EventHandler<TextContentChangedEventArgs>>(), Times.Once);
             var mockAppOptionsProvider = autoMoqer.GetMock<IAppOptionsProvider>();
             mockAppOptionsProvider.VerifyRemove(appOptionsProvider => appOptionsProvider.OptionsChanged -= It.IsAny<Action<IAppOptions>>(), Times.Once);
         }

SharedProject/Editor/DynamicCoverage/Management/BufferLineCoverage.cs

@@ -90,7 +90,7 @@ void AppOptionsChanged(IAppOptions appOptions)
             void textViewClosedHandler(object s, EventArgs e)
             {
                 this.UpdateDynamicCoverageStore((s as ITextView).TextSnapshot);
-                this.textBuffer.Changed -= this.TextBuffer_ChangedOnBackground;
+                this.textBuffer.ChangedOnBackground -= this.TextBuffer_ChangedOnBackground;
                 this.textBuffer.ContentTypeChanged -= this.ContentTypeChanged;
                 textInfo.TextView.Closed -= textViewClosedHandler;
                 appOptionsProvider.OptionsChanged -= AppOptionsChanged;