[Webkit-unassigned] [Bug 71207] An extra line break is inserted when pasting into a font element
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Mar 28 08:02:20 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=71207
--- Comment #23 from yi shen <yi.4.shen at nokia.com> 2012-03-28 08:02:20 PST ---
Thanks for the review :)
(In reply to comment #21)
> (From update of attachment 134055 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=134055&action=review
>
> > Source/WebCore/ChangeLog:1
> > +02012-03-27 Yi Shen <yi.4.shen at nokia.com>, Ryosuke Niwa <rniwa at webkit.org>
>
> Oops, you need to fix this line :( Please delete my name.
>
> > LayoutTests/editing/inserting/insert-text-into-font.html:1
> > +<body contentEditable="true">
>
> No DOCTYPE?
>
> > LayoutTests/editing/inserting/insert-text-into-font.html:39
> > + // Inserting HTML to replace the "line" in the "Second line" should not insert an extra div (line break).
> > + var targetDiv = document.getElementById("secondDiv");
> > + var targetText = targetDiv.firstChild;
> > + execSetSelectionCommand(targetText, 8, targetText, targetText.textContent.length);
> > + document.execCommand("inserthtml", false, "<span id='red' style='color:red'>line</span>");
> > +
> > + // Verify that the font still only has one div.
> > + var font = document.getElementById("fonts");
> > + var divs = font.querySelectorAll("div");
> > + if (divs.length != 1)
> > + fail("An extra div is inserted which is incorrect. There are " + divs.length + " divs inside font element.");
> > +
> > + // Inserting HTML into "Second line" should not insert an extra div (line break).
> > + execSetSelectionCommand(targetText, 8, targetText, 8);
> > + document.execCommand("inserthtml", false, "<span id='green' style='color:green'>green</span>");
> > +
> > + // Verify that the font still only has one div.
> > + var font = document.getElementById("fonts");
> > + var divs = font.querySelectorAll("div");
> > + if (divs.length != 1)
> > + fail("An extra div is inserted which is incorrect. There are " + divs.length + " divs inside font element.");
>
> It's probably better to use dump-as-markup.js here.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list