[Webkit-unassigned] [Bug 8852] Scripts should not be executed before preceding stylesheets are loaded

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Dec 29 16:49:42 PST 2010


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





--- Comment #14 from Darin Adler <darin at apple.com>  2010-12-29 16:49:42 PST ---
(From update of attachment 77623)
View in context: https://bugs.webkit.org/attachment.cgi?id=77623&action=review

> WebCore/dom/PendingScript.h:95
> +    CachedResourceHandle<CachedScript> m_cachedScript; 

The patch adds a trailing space to this line, which made the source code control system think we modified it.

> WebCore/html/parser/HTMLScriptRunner.cpp:-310
> -            // FIXME: We do not block inline <script> tags on stylesheets to match the
> -            // old parser for now.  When we do, the ASSERT below should be added.
> -            // See https://bugs.webkit.org/show_bug.cgi?id=40047
> -            // ASSERT(document()->haveStylesheetsLoaded());

Why didn’t you add the assertion the way the comment said we would?

-- 
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