[Webkit-unassigned] [Bug 48493] Rendered <noscript> tag if XHTMLMP feature is enable.

[bugzilla-daemon at webkit.org]

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





--- Comment #7 from kyounga <kyounga.ra at gmail.com>  2010-11-08 17:35:16 PST ---
(From update of attachment 73314)
>Index: WebCore/ChangeLog
>===================================================================
>--- WebCore/ChangeLog	(revision 71478)
>+++ WebCore/ChangeLog	(working copy)
>@@ -1,3 +1,13 @@
>+2010-11-07  kyounga  <kyounga.ra at gmail.com>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        <noscript> is rendered with enabled XHTMLMP.
>+        https://bugs.webkit.org/show_bug.cgi?id=48493
>+
>+        * html/HTMLElement.cpp:
>+        (WebCore::HTMLElement::rendererIsNeeded):
>+
> 2010-11-06  Pavel Feldman  <pfeldman at chromium.org>
> 
>         Reviewed by Simon Fraser.
>Index: WebCore/html/HTMLElement.cpp
>===================================================================
>--- WebCore/html/HTMLElement.cpp	(revision 71020)
>+++ WebCore/html/HTMLElement.cpp	(working copy)
>@@ -778,14 +778,16 @@ PassRefPtr<HTMLCollection> HTMLElement::
> 
> bool HTMLElement::rendererIsNeeded(RenderStyle *style)
> {
>-#if !ENABLE(XHTMLMP)
>     if (hasLocalName(noscriptTag)) {
>         Frame* frame = document()->frame();
>+#if ENABLE(XHTMLMP)
>+        if (!document()->shouldProcessNoscriptElement())
>+            return false;
>+#else
>         if (frame && frame->script()->canExecuteScripts(NotAboutToExecuteScript))
>             return false;
>-    } else
>-#endif
>-    if (hasLocalName(noembedTag)) {
>+#endif        
>+    } else if (hasLocalName(noembedTag)) {
>         Frame* frame = document()->frame();
>         if (frame && frame->loader()->subframeLoader()->allowPlugins(NotAboutToInstantiatePlugin))
>             return false;
>

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