Bug 50229

Summary: Cleanup SVGPropertyTraits
Product: WebKit Reporter: Nikolas Zimmermann <zimmermann>
Component: SVGAssignee: Nikolas Zimmermann <zimmermann>
Status: RESOLVED FIXED    
Severity: Normal CC: krit, mdelaney7, zimmermann
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 47905    
Attachments:
Description Flags
Patch krit: review+

Description Nikolas Zimmermann 2010-11-30 05:51:51 PST
SVGPropertyTraits includes several SVG* files, just to declare the template specializations. We can just move the specializations into the SVG* files, and remove all SVG*.h includes from SVGPropertyTraits.h
Comment 1 Nikolas Zimmermann 2010-11-30 05:55:23 PST
Created attachment 75130 [details]
Patch
Comment 2 Dirk Schulze 2010-11-30 09:41:14 PST
Comment on attachment 75130 [details]
Patch

LGTM. r=me
Comment 3 Nikolas Zimmermann 2010-12-01 00:51:39 PST
Fixed all style issues locally, wondering why the style bot didn't post a link here.
Chromium EWS does nothing unfortunately, so I'm just risking to land...
Comment 4 Dirk Schulze 2010-12-01 00:56:37 PST
Saw the same problem on other patches. Seems to be a bug in the Style-bot.
Comment 5 Nikolas Zimmermann 2010-12-01 01:03:43 PST
Landed in r73011, closing bug when it builds everywhere.
Comment 6 Nikolas Zimmermann 2010-12-01 02:04:36 PST
Builds just fine, closing bug, marking as blocks 47905, what I forgot before.