[Webkit-unassigned] [Bug 70931] Matched declaration cache

[bugzilla-daemon at webkit.org]

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





--- Comment #7 from Dave Hyatt <hyatt at apple.com>  2011-10-26 10:59:38 PST ---
(From update of attachment 112550)
View in context: https://bugs.webkit.org/attachment.cgi?id=112550&action=review

Looks good. One thing I think we should be doing is sampling the Web to try to find out how to break up our data structs these days. I think we may get a big win from having more smaller structs (especially with the gigantic RareXXX structs we've got going on now).

> Source/WebCore/css/CSSProperty.cpp:381
> +    default:

What about removing this default and instead listing all the other properties? Then you'll get a compile error that the case isn't handled by the switch for any new props added.

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