<html>
    <head>
      <base href="https://bugs.webkit.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [GTK] Rework scrollbars theming code for GTK+ 3.20"
   href="https://bugs.webkit.org/show_bug.cgi?id=156462#c3">Comment # 3</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [GTK] Rework scrollbars theming code for GTK+ 3.20"
   href="https://bugs.webkit.org/show_bug.cgi?id=156462">bug 156462</a>
              from <span class="vcard"><a class="email" href="mailto:cgarcia&#64;igalia.com" title="Carlos Garcia Campos &lt;cgarcia&#64;igalia.com&gt;"> <span class="fn">Carlos Garcia Campos</span></a>
</span></b>
        <pre>Comment on <span class=""><a href="attachment.cgi?id=276138&amp;action=diff" name="attach_276138" title="Patch">attachment 276138</a> <a href="attachment.cgi?id=276138&amp;action=edit" title="Patch">[details]</a></span>
Patch

View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=276138&amp;action=review">https://bugs.webkit.org/attachment.cgi?id=276138&amp;action=review</a>

<span class="quote">&gt;&gt; Source/WebCore/platform/gtk/RenderThemeGadget.cpp:70
&gt;&gt; +    // Scrollbars need to use its GType to be able to get non-CSS style properties.
&gt; 
&gt; Wow, I thought the type was ignored....</span >

Me too, but otherwise gtk_style_context_get_style() fails when trying to get has-*-steppers properties.

<span class="quote">&gt;&gt; Source/WebCore/platform/gtk/ScrollbarThemeGtk.cpp:309
&gt;&gt; +    // painting the thumb can be skipped. We don't have to be exact here.
&gt; 
&gt; I would name the method something like ScrollbarThemeGtk::probablyHasThumb.</span >

This is inherited, it's pure virtual of ScrollbarThemeComposite

<span class="quote">&gt;&gt; Source/WebCore/platform/gtk/ScrollbarThemeGtk.h:65
&gt;&gt; +    bool m_hasForwardButtonStartPart : 1;
&gt; 
&gt; I don't pretend to know if using a bitfield here would improve performance because due to better cache locality, or hurt performance due to the nature of bitfields (seems more likely to me?), but it only saves three bytes, and it's not like this structure is going to be allocated frequently (should be just once, right?) so I would personally use normal bools here.</span >

This is a singleton. We didn't use bool nor bitfield before because we used the member addresses directly in gtk_style_context_get_style(), but now that with newer GTK+ we have a different approach and we have to set the values individually, I thought it wouldn't hurt to use bool : 1 for these values, but I agree this doesn't make a huge difference.</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>