Backed out changeset 3aff5bcb5816 (bug 1402196) for misconfig in editor/libeditor/crashtests/crashtests.list line 91: unknown test type 1402196.html. r=backout
authorSebastian Hengst <archaeopteryx@coole-files.de>
Tue, 24 Oct 2017 09:42:17 +0200
changeset 685708 51a6e6d8ce203daf6e1bea0fe0e2fc5e2640661a
parent 685707 2d9463ac3b6024350c5622b6dd1bbff8ac3d9b64
child 685709 ba502bdf9d9c69a5830eae00d90512f53dc3ada2
push id86016
push userkgupta@mozilla.com
push dateWed, 25 Oct 2017 01:53:44 +0000
reviewersbackout
bugs1402196
milestone58.0a1
backs out3aff5bcb58168c55c71957eac068d2176d1f24a3
Backed out changeset 3aff5bcb5816 (bug 1402196) for misconfig in editor/libeditor/crashtests/crashtests.list line 91: unknown test type 1402196.html. r=backout
editor/libeditor/HTMLEditRules.cpp
--- a/editor/libeditor/HTMLEditRules.cpp
+++ b/editor/libeditor/HTMLEditRules.cpp
@@ -5888,20 +5888,17 @@ HTMLEditRules::GetNodesForOperation(
       rv = BustUpInlinesAtRangeEndpoints(*item);
       if (NS_FAILED(rv)) {
         break;
       }
     }
     // Then unregister the ranges
     for (auto& item : rangeItemArray) {
       htmlEditor->mRangeUpdater.DropRangeItem(item);
-      RefPtr<nsRange> range = item->GetRange();
-      if (range) {
-        aArrayOfRanges.AppendElement(range);
-      }
+      aArrayOfRanges.AppendElement(item->GetRange());
     }
     NS_ENSURE_SUCCESS(rv, rv);
   }
   // Gather up a list of all the nodes
   for (auto& range : aArrayOfRanges) {
     DOMSubtreeIterator iter;
     nsresult rv = iter.Init(*range);
     NS_ENSURE_SUCCESS(rv, rv);