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

[bugzilla-daemon at webkit.org]

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





--- Comment #27 from Alexey Proskuryakov <ap at webkit.org>  2010-12-30 15:53:52 PST ---
There are several aspects here (as in every other bug):
1) Is the patch moving in the right direction? In this case, it almost certainly is.
2) Are there any minor details we'd rather not overlook? Unclear - there was a Mozilla bug that was allegedly supposed to make Mozilla behave like WebKit, and it's marked as fixed, but Mozilla still doesn't behave like WebKit in your testing.
3) Does the patch accurately implement what it claims to? That's up to reviewer to assess.
4) Is regression testing adequate? Test coverage seems good here.
5) Is there a clear history trail to follow if this causes regressions, or if someone just claims that this was a bad idea? I think that there is much more of that now as a result of our discussion.

A reviewer who actually reads the patch can make more technical comments. My primary interest here is making sure that investigation was reasonably complete and that there is a history trail. In no way am I going to be a gatekeeper.

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