ResizableWindow: Add const to virtual members

BREAKING_CHANGES.md

@@ -2,6 +2,27 @@
 
 # Version 8.0.0
 
+## Change
+
+The virtual functions ResizableWindow::getBorderThickness() and
+ResizableWindow::getContentComponentBorder() are now const.
+
+**Possible Issues**
+
+Classes overriding these functions will fail to compile.
+
+**Workaround**
+
+Add 'const' to overriding functions.
+
+**Rationale**
+
+Omitting 'const' from these functions implies that they may change the state of
+the ResizableWindow, which would be unexpected behaviour for getter functions.
+It also means that the functions cannot be called from const member functions,
+which limits their usefulness.
+
+
 ## Change
 
 As part of the Unicode upgrades TextLayout codepaths have been unified across

extras/AudioPluginHost/Source/UI/PluginWindow.h

@@ -233,7 +233,7 @@ public:
     const AudioProcessorGraph::Node::Ptr node;
     const Type type;
 
-    BorderSize<int> getBorderThickness() override
+    BorderSize<int> getBorderThickness() const override
     {
        #if JUCE_IOS || JUCE_ANDROID
         const int border = 10;

modules/juce_gui_basics/windows/juce_DocumentWindow.cpp

@@ -247,12 +247,7 @@ void DocumentWindow::resized()
                             titleBarArea.getWidth(), menuBarHeight);
 }
 
-BorderSize<int> DocumentWindow::getBorderThickness()
-{
-    return ResizableWindow::getBorderThickness();
-}
-
-BorderSize<int> DocumentWindow::getContentComponentBorder()
+BorderSize<int> DocumentWindow::getContentComponentBorder() const
 {
     auto border = getBorderThickness();
 

modules/juce_gui_basics/windows/juce_DocumentWindow.h

@@ -279,9 +279,7 @@ public:
     /** @internal */
     void lookAndFeelChanged() override;
     /** @internal */
-    BorderSize<int> getBorderThickness() override;
-    /** @internal */
-    BorderSize<int> getContentComponentBorder() override;
+    BorderSize<int> getContentComponentBorder() const override;
     /** @internal */
     void mouseDoubleClick (const MouseEvent&) override;
     /** @internal */

modules/juce_gui_basics/windows/juce_ResizableWindow.cpp

@@ -160,7 +160,7 @@ void ResizableWindow::setContentComponentSize (int width, int height)
              height + border.getTopAndBottom());
 }
 
-BorderSize<int> ResizableWindow::getBorderThickness()
+BorderSize<int> ResizableWindow::getBorderThickness() const
 {
     if (isUsingNativeTitleBar() || isKioskMode())
         return {};
@@ -168,7 +168,7 @@ BorderSize<int> ResizableWindow::getBorderThickness()
     return BorderSize<int> ((resizableBorder != nullptr && ! isFullScreen()) ? 4 : 1);
 }
 
-BorderSize<int> ResizableWindow::getContentComponentBorder()
+BorderSize<int> ResizableWindow::getContentComponentBorder() const
 {
     return getBorderThickness();
 }

modules/juce_gui_basics/windows/juce_ResizableWindow.h

@@ -301,12 +301,12 @@ public:
     /** Returns the width of the frame to use around the window.
         @see getContentComponentBorder
     */
-    virtual BorderSize<int> getBorderThickness();
+    virtual BorderSize<int> getBorderThickness() const;
 
     /** Returns the insets to use when positioning the content component.
         @see getBorderThickness
     */
-    virtual BorderSize<int> getContentComponentBorder();
+    virtual BorderSize<int> getContentComponentBorder() const;
 
     //==============================================================================
     /** A set of colour IDs to use to change the colour of various aspects of the window.