Reviewed by Chris
authorkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Aug 2004 17:00:08 +0000 (17:00 +0000)
committerkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Aug 2004 17:00:08 +0000 (17:00 +0000)
commit99b6cd931711b1cdbabbdeb68e200df1525a389b
tree5ff91d33e628a1fd755d447e0c115af8a9028cfc
parenta2e6fdbd9c8a045b4697b592fb6a90f75f0977ee
    Reviewed by Chris

        Fix for this bug:

        <rdar://problem/3773564> REGRESSION (125-159): Code to remove HTML styles before applying new HTML styles is broken

        * khtml/editing/htmlediting_impl.cpp:
        (khtml::ApplyStyleCommandImpl::doApply): Unrelated change to constrain downstream position of selection
        start to block boundaries. This is a new feature of the downstream function and I missed this usage
        when adding the feature.
        (khtml::ApplyStyleCommandImpl::removeCSSStyle): There once was code to remove style attributes from spans
        which became emptied as a result of removing CSS properties, but I do not see such code in the
        tree any more. A quick review of the ChangeLog did not reveal anything. I do not remember making such
        a change myself...

        In any case, I have restored logic to prune out styling spans we insert if the process of
        removing styles caused a span-plus-styles node to become emptied of markup which changes the style
        of its contents. This fixes the bug.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@7335 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebCore/ChangeLog-2005-08-23
WebCore/khtml/editing/htmlediting_impl.cpp