[GTK] Add support for rendering slider ticks

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

Reviewed by Adrian Perez de Castro.

.:

Add a test case for sliders with ticks.

* ManualTests/gtk/theme.html:

Source/WebCore:

* rendering/RenderThemeAdwaita.cpp:
(WebCore::RenderThemeAdwaita::paintSliderTrack):
(WebCore::RenderThemeAdwaita::sliderTickSize const):
(WebCore::RenderThemeAdwaita::sliderTickOffsetFromTrackCenter const):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@257300 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Author: [email protected]

ChangeLog

@@ -1,3 +1,14 @@
+2020-02-25  Carlos Garcia Campos  <[email protected]>
+
+        [GTK] Add support for rendering slider ticks
+        https://bugs.webkit.org/show_bug.cgi?id=208132
+
+        Reviewed by Adrian Perez de Castro.
+
+        Add a test case for sliders with ticks.
+
+        * ManualTests/gtk/theme.html:
+
 2020-02-25  Carlos Garcia Campos  <[email protected]>
 
         [GTK] Stop using gtk foreign drawing API to style form controls

ManualTests/gtk/theme.html

@@ -133,7 +133,18 @@ <h1>Range selector</h1>
       <td><input type="range" value="50" max="100" disabled></input></td>
       <td><input type="range" value="50" max="100" style="-webkit-appearance: slider-vertical" disabled></input></td>
     </tr>
+    <tr>
+      <td><input type="range" min="0" max="100" step="25" list="steplist"></input></td>
+      <td><input type="range" min="0" max="100" step="25" list="steplist" style="-webkit-appearance: slider-vertical"></input></td>
+    </tr>
   </table>
+  <datalist id="steplist">
+    <option>0</option>
+    <option>25</option>
+    <option>50</option>
+    <option>75</option>
+    <option>100</option>
+  </datalist>
 
   <h1>Iframe scrollbars</h1>
   <iframe width="200" height="100" scrolling="yes" src="data:text/html,

Source/WebCore/ChangeLog

@@ -1,3 +1,15 @@
+2020-02-25  Carlos Garcia Campos  <[email protected]>
+
+        [GTK] Add support for rendering slider ticks
+        https://bugs.webkit.org/show_bug.cgi?id=208132
+
+        Reviewed by Adrian Perez de Castro.
+
+        * rendering/RenderThemeAdwaita.cpp:
+        (WebCore::RenderThemeAdwaita::paintSliderTrack):
+        (WebCore::RenderThemeAdwaita::sliderTickSize const):
+        (WebCore::RenderThemeAdwaita::sliderTickOffsetFromTrackCenter const):
+
 2020-02-25  Carlos Garcia Campos  <[email protected]>
 
         [GTK] Stop using gtk foreign drawing API to style form controls

Source/WebCore/rendering/RenderThemeAdwaita.cpp

@@ -433,6 +433,10 @@ bool RenderThemeAdwaita::paintSliderTrack(const RenderObject& renderObject, cons
     graphicsContext.setFillColor(activeSelectionBackgroundColor({ }));
     graphicsContext.fillPath(path);
 
+#if ENABLE(DATALIST_ELEMENT)
+    paintSliderTicks(renderObject, paintInfo, rect);
+#endif
+
     if (isFocused(renderObject))
         ThemeAdwaita::paintFocus(graphicsContext, fieldRect, sliderTrackFocusOffset);
 
@@ -565,14 +569,12 @@ bool RenderThemeAdwaita::paintMediaVolumeSliderTrack(const RenderObject& renderO
 #if ENABLE(DATALIST_ELEMENT)
 IntSize RenderThemeAdwaita::sliderTickSize() const
 {
-    // FIXME: We need to set this to the size of one tick mark.
-    return IntSize(0, 0);
+    return { 1, 7 };
 }
 
 int RenderThemeAdwaita::sliderTickOffsetFromTrackCenter() const
 {
-    // FIXME: We need to set this to the position of the tick marks.
-    return 0;
+    return -16;
 }
 
 void RenderThemeAdwaita::adjustListButtonStyle(RenderStyle& style, const Element*) const