Title: [188206] trunk/Source/WebCore
Revision
188206
Author
commit-qu...@webkit.org
Date
2015-08-09 23:50:56 -0700 (Sun, 09 Aug 2015)

Log Message

[EFL] Use the non-overlay scrollbar
https://bugs.webkit.org/show_bug.cgi?id=147725

Patch by Hunseop Jeong <hs85.je...@samsung.com> on 2015-08-09
Reviewed by Gyuyoung Kim.

No new tests because there is no behavior change.

* platform/efl/ScrollbarThemeEfl.cpp:
(WebCore::ScrollbarThemeEfl::usesOverlayScrollbars):
Changed the condition of the 'usesOverlayScrollbars' to use the
setting value.
* platform/efl/ScrollbarThemeEfl.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (188205 => 188206)


--- trunk/Source/WebCore/ChangeLog	2015-08-10 06:41:20 UTC (rev 188205)
+++ trunk/Source/WebCore/ChangeLog	2015-08-10 06:50:56 UTC (rev 188206)
@@ -1,3 +1,18 @@
+2015-08-09  Hunseop Jeong  <hs85.je...@samsung.com>
+
+        [EFL] Use the non-overlay scrollbar
+        https://bugs.webkit.org/show_bug.cgi?id=147725
+
+        Reviewed by Gyuyoung Kim.
+
+        No new tests because there is no behavior change.
+
+        * platform/efl/ScrollbarThemeEfl.cpp:
+        (WebCore::ScrollbarThemeEfl::usesOverlayScrollbars):
+        Changed the condition of the 'usesOverlayScrollbars' to use the
+        setting value.
+        * platform/efl/ScrollbarThemeEfl.h:
+
 2015-08-09  Chris Dumez  <cdu...@apple.com>
 
         Page cache doesn't work for pages actively using Geolocation

Modified: trunk/Source/WebCore/platform/efl/ScrollbarThemeEfl.cpp (188205 => 188206)


--- trunk/Source/WebCore/platform/efl/ScrollbarThemeEfl.cpp	2015-08-10 06:41:20 UTC (rev 188205)
+++ trunk/Source/WebCore/platform/efl/ScrollbarThemeEfl.cpp	2015-08-10 06:50:56 UTC (rev 188206)
@@ -33,6 +33,7 @@
 #include "NotImplemented.h"
 #include "Page.h"
 #include "RenderThemeEfl.h"
+#include "Settings.h"
 
 #include <wtf/NeverDestroyed.h>
 
@@ -63,6 +64,11 @@
     return defaultThickness;
 }
 
+bool ScrollbarThemeEfl::usesOverlayScrollbars() const
+{
+    return Settings::usesOverlayScrollbars();
+}
+
 bool ScrollbarThemeEfl::hasThumb(Scrollbar& scrollbar)
 {
     return thumbLength(scrollbar) > 0;

Modified: trunk/Source/WebCore/platform/efl/ScrollbarThemeEfl.h (188205 => 188206)


--- trunk/Source/WebCore/platform/efl/ScrollbarThemeEfl.h	2015-08-10 06:41:20 UTC (rev 188205)
+++ trunk/Source/WebCore/platform/efl/ScrollbarThemeEfl.h	2015-08-10 06:50:56 UTC (rev 188206)
@@ -43,7 +43,7 @@
     void setScrollbarThickness(int thickness) { m_scrollbarThickness = thickness; }
 
 protected:
-    virtual bool usesOverlayScrollbars() const override { return true; }
+    virtual bool usesOverlayScrollbars() const override;
     virtual bool hasButtons(Scrollbar&) override { return false; }
     virtual bool hasThumb(Scrollbar&) override;
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to