Remove the ifdef for platform scrollbars vs. engine scrollbars until
authorhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 23 Aug 2006 21:36:58 +0000 (21:36 +0000)
committerhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 23 Aug 2006 21:36:58 +0000 (21:36 +0000)
        engine scrollbars actually exist.

        * platform/ScrollBar.h:
        (WebCore::ScrollBar::hasPlatformScrollBars):

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

WebCore/ChangeLog
WebCore/platform/ScrollBar.h

index 5cec33724fd3a7b1d647f8ecf87fd8b83aba155a..54b5bc14a5476acb93d4c251913826a4d4ab92df 100644 (file)
@@ -1,3 +1,11 @@
+2006-08-23  David Hyatt  <hyatt@apple.com>
+
+        Remove the ifdef for platform scrollbars vs. engine scrollbars until
+        engine scrollbars actually exist.
+
+        * platform/ScrollBar.h:
+        (WebCore::ScrollBar::hasPlatformScrollBars):
+
 2006-08-23  Justin Garcia  <justin.garcia@apple.com>
 
         Reviewed by harrison
index f95fb587dd6a459676bae8e341c12caa87aae483..01dbfa49d7e1bf83e9d5bf8d9e2bb96f8f2a801e 100644 (file)
@@ -82,12 +82,10 @@ public:
     virtual void paint(GraphicsContext*, const IntRect& damageRect) = 0;
 
     static bool hasPlatformScrollBars() {
-        // To use the platform's built-in scrollbars by default add to this ifdef.
-#if PLATFORM(MAC)
+        // To use the platform's built-in scrollbars by default, return true.  We may
+        // support styled engine scrollbars someday, and some platforms may wish to not
+        // implement a platform scrollbar at all by default.  That's what this method is for.
         return true;
-#else
-        return false;  // The engine will create scrollbars (so the RenderTheme will need to support scrollbar painting).
-#endif
     }
 
 protected: