Bug 70331

Summary: Move default stylesheet loading to a separate function
Product: WebKit Reporter: Antti Koivisto <koivisto>
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch kling: review+

Antti Koivisto
Reported 2011-10-18 08:27:08 PDT
From CSSStyleSelector::styleForElement()
Attachments
patch (9.67 KB, patch)
2011-10-18 08:39 PDT, Antti Koivisto 		kling: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Antti Koivisto
Comment 1 2011-10-18 08:39:20 PDT
Created attachment 111447 [details] patch
Andreas Kling
Comment 2 2011-10-18 08:43:27 PDT
Comment on attachment 111447 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=111447&action=review LGTM! > Source/WebCore/css/CSSStyleSelector.cpp:1121 > - > + Did you performance test this?
Antti Koivisto
Comment 3 2011-10-18 08:48:40 PDT
http://trac.webkit.org/changeset/97758
Antti Koivisto
Comment 4 2011-10-18 08:49:38 PDT
> Did you performance test this? Nope. In an unlikely event it managed to slow down something I'm sure I'll notice in the future.
Note You need to log in before you can comment on or make changes to this bug.