900 instances of "NS_ENSURE_TRUE(selection->RangeCount()) failed" emitted from editor/libeditor/nsHTMLEditRules.cpp during linux64 debug testing

RESOLVED FIXED in Firefox 43

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: erahm, Assigned: Ehsan)

Tracking

(Blocks 1 bug)

unspecified
mozilla43
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox43 fixed)

Details

> 921 [NNNNN] WARNING: NS_ENSURE_TRUE(selection->RangeCount()) failed: file editor/libeditor/nsHTMLEditRules.cpp, line 320

Given the widespread distribution of this warning it feels like we should probably just remove it.

This warning [1] shows up in the following test suites:

> mozilla-central_ubuntu64_vm-debug_test-mochitest-4-bm54-tests1-linux64-build4.txt:173
> mozilla-central_ubuntu64_vm-debug_test-crashtest-bm54-tests1-linux64-build7.txt:171
> mozilla-central_ubuntu64_vm-debug_test-mochitest-e10s-4-bm115-tests1-linux64-build34.txt:125
> mozilla-central_ubuntu64_vm-debug_test-mochitest-5-bm117-tests1-linux64-build5.txt:80
> mozilla-central_ubuntu64_vm-debug_test-mochitest-other-bm52-tests1-linux64-build8.txt:72
> mozilla-central_ubuntu64_vm-debug_test-reftest-e10s-2-bm120-tests1-linux64-build0.txt:58
> mozilla-central_ubuntu64_vm-debug_test-reftest-4-bm122-tests1-linux64-build20.txt:56
> mozilla-central_ubuntu64_vm-debug_test-reftest-e10s-1-bm120-tests1-linux64-build1.txt:33
> mozilla-central_ubuntu64_vm-debug_test-reftest-2-bm117-tests1-linux64-build12.txt:30
> mozilla-central_ubuntu64_vm-debug_test-mochitest-2-bm51-tests1-linux64-build9.txt:28
> mozilla-central_ubuntu64_vm-debug_test-mochitest-e10s-2-bm113-tests1-linux64-build0.txt:27
> mozilla-central_ubuntu64_vm-debug_test-mochitest-1-bm54-tests1-linux64-build0.txt:22
> mozilla-central_ubuntu64_vm-debug_test-mochitest-e10s-1-bm122-tests1-linux64-build8.txt:13
> mozilla-central_ubuntu64_vm-debug_test-mochitest-e10s-5-bm117-tests1-linux64-build17.txt:12
> mozilla-central_ubuntu64_vm-debug_test-mochitest-e10s-browser-chrome-1-bm122-tests1-linux64-build7.txt:5
> mozilla-central_ubuntu64_vm-debug_test-mochitest-browser-chrome-1-bm51-tests1-linux64-build6.txt:5
> mozilla-central_ubuntu64_vm-debug_test-reftest-1-bm116-tests1-linux64-build43.txt:3
> mozilla-central_ubuntu64_vm-debug_test-mochitest-e10s-browser-chrome-2-bm54-tests1-linux64-build15.txt:3
> mozilla-central_ubuntu64_vm-debug_test-mochitest-browser-chrome-2-bm118-tests1-linux64-build16.txt:3
> mozilla-central_ubuntu64_vm-debug_test-reftest-3-bm54-tests1-linux64-build16.txt:2

It shows up in 369 tests. A few of the most prevalent:

> 67 - layout/base/tests/test_reftests_with_caret.html
> 52 - editor/libeditor/tests/test_bug611182.html
> 47 - file:///builds/slave/test/build/tests/reftest/tests/editor/libeditor/crashtests/759748.html
> 32 - editor/libeditor/tests/test_bug520189.html
> 28 - editor/libeditor/tests/test_root_element_replacement.html
> 19 - dom/html/test/test_bug446483.html
> 12 - file:///builds/slave/test/build/tests/reftest/tests/editor/reftests/824080-4.html
> 10 - editor/libeditor/tests/test_dom_input_event_on_htmleditor.html
> 10 - editor/libeditor/tests/test_bug455992.html
> 8 - file:///builds/slave/test/build/tests/reftest/tests/layout/reftests/bugs/613433-1.html

[1] https://hg.mozilla.org/mozilla-central/annotate/49683d4e9ebd/editor/libeditor/nsHTMLEditRules.cpp#l320
Assignee: nobody → ehsan
https://hg.mozilla.org/mozilla-central/rev/4f222c9dca5b
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in before you can comment on or make changes to this bug.