WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
26365
/LayoutTests/editing/deleting/5390681.html doesn't test what it claims to test
https://bugs.webkit.org/show_bug.cgi?id=26365
Summary
/LayoutTests/editing/deleting/5390681.html doesn't test what it claims to test
Ryosuke Niwa
Reported
2009-06-12 18:00:58 PDT
The expected result of /LayoutTests/editing/deleting/5390681.html seems to be incorrect. We shouldn't be inserting a placeholder. The dumpAsText version of this test is: <p>This tests for a bug where expansion for smart delete would not consider editable boundaries. Only 'foo' should be deleted. You should see ' bar'. <b>There is a bug: while the non-editable space isn't deleted, deletion inserts a placeholder when it shouldn't.</b></p> <div contenteditable="true" id="div">foo<span contenteditable="false"> bar</span></div> <p id="console"></p> <script> if (window.layoutTestController) window.layoutTestController.dumpAsText(); function log(message) { var console = document.getElementById("console"); var text = document.createTextNode(message); console.appendChild(text); } var div = document.getElementById("div"); var sel = window.getSelection(); sel.setPosition(div, 0); sel.modify("extend", "forward", "word"); document.execCommand("Delete"); log(document.getElementById('div').innerHTML); </script>
Attachments
Add attachment
proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1
2012-05-25 17:11:56 PDT
The test has since been converted to a dump-as-markup test and
http://trac.webkit.org/changeset/95003
renamed the test.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug