[Webkit-unassigned] [Bug 37204] RenderThemeChromiumMac.mm should share code with RenderThemeMac.mm

[bugzilla-daemon at webkit.org]

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





--- Comment #7 from MORITA Hajime <morrita at google.com>  2010-04-09 22:29:26 PST ---
Hi Eric, Thank you for reviewing!

> OK.  This is WAY better than what we have now.  The only worry I have is burned
> on the Mac folks and/or regressions to the Mac code.
I agree your concern. 
Fortunately changes on RenderThemeMac.mm is small and is (or should be) trivial
-
It just added empty hooks, extract a part of code blocks to methods.
I hope these changes would be acceptable...

> I think the new methods you're adding to RenderThemeMac.h should be
> commented/documented in the header.
Added comments to explain their purpose.

>This guard seems unecessary:
>60 #if PLATFORM(CHROMIUM)
>15961 PassRefPtr<RenderTheme> RenderTheme::themeForPage(Page*)
>16062 {
Removed.

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