[Webkit-unassigned] [Bug 55229] New: REGRESSION (r66062) - set(Inner/Outer)Text on many types of elements throws an exception, and they didn't before

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=55229

           Summary: REGRESSION (r66062) - set(Inner/Outer)Text on many
                    types of elements throws an exception, and they didn't
                    before
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: HTML DOM
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: beidson at apple.com


REGRESSION (r66062) - set(Inner/Outer)Text on many types of elements throws an exception, and they didn't before.

Link to the change:
http://trac.webkit.org/changeset/66057
Link to the original bug:
https://bugs.webkit.org/show_bug.cgi?id=44626

ChangeLog says "this is just moving code and should have no functional change."

But by replacing the usage of "HTMLElement::endTagRequirement()" with "HTMLElement::ieForbidsInsertHTML()" in 3 key places in HTMLElement, it had functional change.

Before this change, HTMLElement::deprecatedCreateContextualFragment would only fail to return a fragment for <wbr> elements.  Now it fails for ~20 tag names listed in HTMLElement::ieForbidsInsertHTML()

Before this change, HTMLElement::setInnerText and HTMLElement::setOuterText only threw exceptions for 10 tag names they knew about, and <wbr>.
Now they throw exceptions for the 10 tag names they knew about, and the ~20 listed in HTMLElement::ieForbidsInsertHTML()

This broke compatibility with other browsers like Firefox and Opera.  It also *broke* compatibility with IE while purporting to match IE's rules, as noted in the FIXME comment in ieForbidsInsertHTML()

Test case coming soon...

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.