<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [GTK] Correct behavior for dark themes"
   href="https://bugs.webkit.org/show_bug.cgi?id=186219#c3">Comment # 3</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [GTK] Correct behavior for dark themes"
   href="https://bugs.webkit.org/show_bug.cgi?id=186219">bug 186219</a>
              from <span class="vcard"><a class="email" href="mailto:cadubentzen@gmail.com" title="Carlos Eduardo Ramalho <cadubentzen@gmail.com>"> <span class="fn">Carlos Eduardo Ramalho</span></a>
</span></b>
        <pre>(In reply to Michael Catanzaro from <a href="show_bug.cgi?id=186219#c1">comment #1</a>)
<span class="quote">> Carlos Garcia's comment in <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - [GTK] Menu list button doesn't use the text color from the theme"
   href="show_bug.cgi?id=118234">bug #118234</a> might be relevant:

> (In reply to Carlos Garcia Campos from <a href="show_bug.cgi?id=186219#c12">comment #12</a>)
> > I think we need to implement RenderTheme::isControlStyled() properly to
> > decide whether to apply native style or not.</span >

I've been doing a few tests on this and here some notes:

The return of isControlStyled() impacts whether the adjustXyzStyle() methods are called AND paintXyz() (though it's not coded in RenderTheme, but in RenderBox checking for style.appearance()).

The adjustXyzStyle() basically set theme styles for the foreground (which is always drawn by the engine) and the paintXyz() methods draw the background directly to the cairo context via GTK foreign drawing (meaning basically background-color and rounded borders from GTK)

Checking if the color was modified in isControlStyled results in the background also not being drawn. This already happens if the background is changed via CSS as we don't the rounded border in some tests.

The difference is that if one sets the color now, we don't customize the background either. This is expected, expect I don't like having the inputs not rounded when the color is set :(</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>