[Webkit-unassigned] [Bug 23452] Add XHTMLMP support to Webkit

[bugzilla-daemon at webkit.org]

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





------- Comment #45 from simon.fraser at apple.com  2009-05-19 21:46 PDT -------
(From update of attachment 30492)
> Index: WebCore/dom/Node.cpp
> ===================================================================

> +#if ENABLE(XHTMLMP)
> +        if (localName() == HTMLNames::noscriptTag.localName())
> +            return document()->styleSelector()->styleForElement(static_cast<HTMLNoScriptElement*>(this), 0, false);
> +#endif
>          return document()->styleSelector()->styleForElement(static_cast<Element*>(this));

This would be cleaner as:

bool allowSharing = true;
#if ENABLE(XHTMLMP)
  if (localName() == HTMLNames::noscriptTag.localName())
    allowSharing = false; // comment explaining why noscript is so special
#endif
  return
document()->styleSelector()->styleForElement(static_cast<Element*>(this), 0,
allowSharing);


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