Have select element respect current appearance
Created attachment 336087 [details] Patch
<rdar://problem/38644433>
Comment on attachment 336087 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=336087&action=review > Source/WebCore/rendering/RenderThemeMac.mm:1348 > + style.setColor(e ? !e->isDisabledFormControl() ? systemColor(CSSValueButtontext, e->document().page()->useSystemAppearance()) : systemColor(CSSValueGraytext, e->document().page()->useSystemAppearance()) : Color::darkGray); The logic here (nested ternary operators) is incredibly hard to follow. Can we pop it out into a few lines instead?
Also I can't actually follow if you don't dereference e in the !e case
Created attachment 336088 [details] Patch
Comment on attachment 336088 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=336088&action=review > Source/WebCore/rendering/RenderThemeMac.mm:1350 > + c = !e->isDisabledFormControl() ? systemColor(CSSValueButtontext, e->document().page()->useSystemAppearance()) : systemColor(CSSValueGraytext, e->document().page()->useSystemAppearance()); Pop useSystemAppearance into a local
Created attachment 336090 [details] Patch
https://trac.webkit.org/changeset/229730/webkit
Rebaselined failing tests in https://trac.webkit.org/changeset/229762/webkit