[Webkit-unassigned] [Bug 93643] Preserve styling elements in DeleteSelectionCommand
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Aug 14 16:41:58 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=93643
--- Comment #46 from Darin Adler <darin at apple.com> 2012-08-14 16:42:26 PST ---
(From update of attachment 158440)
View in context: https://bugs.webkit.org/attachment.cgi?id=158440&action=review
>> Source/WebCore/editing/AppendNodeCommand.cpp:46
>> + ASSERT(m_parent->rendererIsEditable() || !m_parent->attached() || (m_parent->hasTagName(HTMLNames::headTag) && ((HTMLElement*)m_parent.get())->contentEditable() == "true"));
>
> This logic is long enough that I think it should go into a helper function now, rather than being repeated three times.
>
> The style of that cast is wrong, but you won’t need a typecast at all. Something more like this:
>
> ASSERT(m_parent->rendererIsEditable() || !m_parent->attached() || (m_parent->hasTagName(HTMLNames::headTag) && m_parent->isEditable()));
>
> But also, you need a “why” comment. Why is the head element a special case?
I said isEditable() above where I meant isContentEditable().
>> Source/WebCore/editing/DeleteSelectionCommand.cpp:425
>> + return;
>
> The right way to do this is just:
>
> if (!head || !head->isContentEditable())
> return;
>
> You should use the contentEditable function that returns a string.
I meant: You should not have to use the contentEditable function that returns a string.
--
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