wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 502259. Make delete at end of paragraph work. r=cpearce

Browse Source
This commit is contained in:
liucougar@gmail.com 2009-08-04 09:46:48 +12:00
parent 152f21c553
commit cc07460944
2 changed files with 11 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
editor/libeditor/text/nsPlaintextEditor.cpp
View File

@ -683,7 +683,7 @@ nsPlaintextEditor::ExtendSelectionForDelete(nsISelection *aSelection,
break;
case eNext:
result = selCont->CharacterExtendForDelete();
*aAction = eNone;
// Don't set aAction to eNone (see Bug 502259)
break;
case ePrevious:
/* FIXME: extend selection over UTF-16 surrogates for Bug #332636

13
layout/generic/test/test_backspace_delete.xul
View File

@ -76,12 +76,15 @@ function execTests() {
is(selRange.endOffset, endOffset, selErrString("Word right"));
}
function testDelete(node, offset, text) {
function testDelete(node, offset, text, richtext) {
doCommand("cmd_deleteCharForward");
var msg = "Delete broken in \"" + editor.innerHTML + "\", offset " + offset;
if(typeof node == 'function'){
node = node();
}
is(sel.anchorNode, node, msg);
is(sel.anchorOffset, offset, msg);
is(editor.textContent, text, msg);
is(richtext?editor.innerHTML:editor.textContent, text, msg);
}
function testBackspace(node, offset, text) {
@ -177,7 +180,7 @@ function execTests() {
// Tests for Bug 419217
setupTest("foo<div>bar</div>", 3);
testDelete(editor.firstChild, 3, "foobar");
testDelete(function(){return editor.firstChild;}, 3, "foobar", true);
// Tests for Bug 419406
var s = "helloשלום";
@ -202,6 +205,10 @@ function execTests() {
testDeleteNextWord(editor.firstChild, 0, "\u00A0text.");
// testDeleteNextWord(editor, 0, "");
// Tests for Bug 502259
setupTest("<p>Bug</p>\n<p>502259</p>", 1);
testDelete(function(){return editor.firstChild.firstChild;}, 3, "<p>Bug502259</p>", true);
SimpleTest.finish();
}