[Chromium] Update ThemeChromiumMac.mm for the recent changes of ThemeMac.mm
Created attachment 61017 [details] Patch
Comment on attachment 61017 [details] Patch I don't think there is much I can review here. Only really rubber-stamp. Assuming it works for Chromium, rs=me.
Comment on attachment 61017 [details] Patch > +++ b/WebCore/platform/chromium/ThemeChromiumMac.mm > + case OuterSpinButtonPart: { > + static const int stepperMargin[4] = { 0, 0, 0, 0}; Probably a negligible style issue. Missing a space at the end? "0 };"
Committed r63514: <http://trac.webkit.org/changeset/63514>
(In reply to comment #3) > (From update of attachment 61017 [details]) > > +++ b/WebCore/platform/chromium/ThemeChromiumMac.mm > > + case OuterSpinButtonPart: { > > + static const int stepperMargin[4] = { 0, 0, 0, 0}; > > Probably a negligible style issue. Missing a space at the end? "0 };" Thank you for pointing it out. I have fixed it and the same one in ThemeMac.mm by http://trac.webkit.org/changeset/63518
http://trac.webkit.org/changeset/63514 might have broken Chromium Win Release The following changes are on the blame list: http://trac.webkit.org/changeset/63514 http://trac.webkit.org/changeset/63515 http://trac.webkit.org/changeset/63516 http://trac.webkit.org/changeset/63517 http://trac.webkit.org/changeset/63518