Bug 1387143 followup. Remove assertion that does not seem to hold and causes a CLOSED TREE. r=bzbarsky
authorBoris Zbarsky <bzbarsky@mit.edu>
Tue, 08 May 2018 15:04:53 -0400
changeset 471685 d8c40353781d712f0e32e5c4458552680d8377b1
parent 471684 93f3dc3cecc3ce364736bc777e259bd273fcc9be
child 471686 b17ded2f29626043dbf28bd4cd76c7bdcf91a40c
push id9374
push userjlund@mozilla.com
push dateMon, 18 Jun 2018 21:43:20 +0000
treeherdermozilla-beta@160e085dfb0b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbzbarsky
bugs1387143
milestone62.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1387143 followup. Remove assertion that does not seem to hold and causes a CLOSED TREE. r=bzbarsky
dom/base/Selection.cpp
--- a/dom/base/Selection.cpp
+++ b/dom/base/Selection.cpp
@@ -3342,19 +3342,17 @@ Selection::AddSelectionListener(nsISelec
 {
   MOZ_ASSERT(aNewListener);
   mSelectionListeners.AppendElement(aNewListener); // AddRefs
 }
 
 void
 Selection::RemoveSelectionListener(nsISelectionListener* aListenerToRemove)
 {
-  DebugOnly<bool> found =
-    mSelectionListeners.RemoveElement(aListenerToRemove); // Releases
-  MOZ_ASSERT(found, "Removing a nonexistent listener?");
+  mSelectionListeners.RemoveElement(aListenerToRemove); // Releases
 }
 
 Element*
 Selection::GetCommonEditingHostForAllRanges()
 {
   Element* editingHost = nullptr;
   for (RangeData& rangeData : mRanges) {
     nsRange* range = rangeData.mRange;