Constify and use reference for RenderStyle in LayoutTheme

Source/core/layout/LayoutThemeChromiumSkia.h

 /*
  * This file is part of the WebKit project.
  *
  * Copyright (C) 2006 Apple Computer, Inc.
  * Copyright (C) 2006 Michael Emmel mike.emmel@gmail.com
  * Copyright (C) 2007 Holger Hans Peter Freyther
  * Copyright (C) 2007 Alp Toker <alp@atoker.com>
  * Copyright (C) 2008, 2009 Google, Inc.
  * All rights reserved.
  *
@@ skipping 30 matching lines @@
 
     virtual String extraDefaultStyleSheet() override;
     virtual String extraQuirksStyleSheet() override;
 
     virtual Color platformTapHighlightColor() const override
     {
         return Color(defaultTapHighlightColor);
     }
 
     // A method asking if the theme's controls actually care about redrawing when hovered.
-    virtual bool supportsHover(const RenderStyle*) const override final;
+    virtual bool supportsHover(const RenderStyle&) const override final;
 
     // A method asking if the theme is able to draw the focus ring.
-    virtual bool supportsFocusRing(const RenderStyle*) const override;
+    virtual bool supportsFocusRing(const RenderStyle&) const override;
 
     // The platform selection color.
     virtual Color platformActiveSelectionBackgroundColor() const override;
     virtual Color platformInactiveSelectionBackgroundColor() const override;
     virtual Color platformActiveSelectionForegroundColor() const override;
     virtual Color platformInactiveSelectionForegroundColor() const override;
     virtual Color platformFocusRingColor() const override;
 
     // To change the blink interval, override caretBlinkIntervalInternal instead of this one so that we may share layout test code an intercepts.
     virtual double caretBlinkInterval() const override;
 
     // System fonts.
     virtual void systemFont(CSSValueID systemFontID, FontStyle&, FontWeight&, float& fontSize, AtomicString& fontFamily) const;
 
-    virtual int minimumMenuListSize(RenderStyle*) const override;
+    virtual int minimumMenuListSize(const RenderStyle&) const override;
 
-    virtual void setCheckboxSize(RenderStyle*) const override;
+    virtual void setCheckboxSize(RenderStyle&) const override;
 
-    virtual void setRadioSize(RenderStyle*) const override;
+    virtual void setRadioSize(RenderStyle&) const override;
 
-    virtual void adjustButtonStyle(RenderStyle*, Element*) const override;
+    virtual void adjustButtonStyle(RenderStyle&, Element*) const override;
 
     virtual bool paintTextArea(RenderObject*, const PaintInfo&, const IntRect&) override;
 
-    virtual void adjustSearchFieldStyle(RenderStyle*, Element*) const override;
+    virtual void adjustSearchFieldStyle(RenderStyle&, Element*) const override;
     virtual bool paintSearchField(RenderObject*, const PaintInfo&, const IntRect&) override;
 
-    virtual void adjustSearchFieldCancelButtonStyle(RenderStyle*, Element*) const override;
+    virtual void adjustSearchFieldCancelButtonStyle(RenderStyle&, Element*) const override;
     virtual bool paintSearchFieldCancelButton(RenderObject*, const PaintInfo&, const IntRect&) override;
 
-    virtual void adjustSearchFieldDecorationStyle(RenderStyle*, Element*) const override;
+    virtual void adjustSearchFieldDecorationStyle(RenderStyle&, Element*) const override;
 
-    virtual void adjustSearchFieldResultsDecorationStyle(RenderStyle*, Element*) const override;
+    virtual void adjustSearchFieldResultsDecorationStyle(RenderStyle&, Element*) const override;
     virtual bool paintSearchFieldResultsDecoration(RenderObject*, const PaintInfo&, const IntRect&) override;
 
     virtual bool paintMediaSliderTrack(RenderObject*, const PaintInfo&, const IntRect&) override;
     virtual bool paintMediaVolumeSliderTrack(RenderObject*, const PaintInfo&, const IntRect&) override;
-    virtual void adjustSliderThumbSize(RenderStyle*, Element*) const override;
+    virtual void adjustSliderThumbSize(RenderStyle&, Element*) const override;
     virtual bool paintMediaSliderThumb(RenderObject*, const PaintInfo&, const IntRect&) override;
     virtual bool paintMediaToggleClosedCaptionsButton(RenderObject*, const PaintInfo&, const IntRect&) override;
     virtual bool paintMediaCastButton(RenderObject*, const PaintInfo&, const IntRect&) override;
     virtual bool paintMediaVolumeSliderThumb(RenderObject*, const PaintInfo&, const IntRect&) override;
     virtual bool paintMediaPlayButton(RenderObject*, const PaintInfo&, const IntRect&) override;
     virtual bool paintMediaOverlayPlayButton(RenderObject*, const PaintInfo&, const IntRect&) override;
     virtual bool paintMediaMuteButton(RenderObject*, const PaintInfo&, const IntRect&) override;
     virtual String formatMediaControlsTime(float time) const override;
     virtual String formatMediaControlsCurrentTime(float currentTime, float duration) const override;
     virtual bool paintMediaFullscreenButton(RenderObject*, const PaintInfo&, const IntRect&) override;
 
     // MenuList refers to an unstyled menulist (meaning a menulist without
     // background-color or border set) and MenuListButton refers to a styled
     // menulist (a menulist with background-color or border set). They have
     // this distinction to support showing aqua style themes whenever they
     // possibly can, which is something we don't want to replicate.
     //
     // In short, we either go down the MenuList code path or the MenuListButton
     // codepath. We never go down both. And in both cases, they render the
     // entire menulist.
-    virtual void adjustMenuListStyle(RenderStyle*, Element*) const override;
-    virtual void adjustMenuListButtonStyle(RenderStyle*, Element*) const override;
+    virtual void adjustMenuListStyle(RenderStyle&, Element*) const override;
+    virtual void adjustMenuListButtonStyle(RenderStyle&, Element*) const override;
     virtual bool paintMenuListButton(RenderObject*, const PaintInfo&, const IntRect&) override;
 
     virtual double animationRepeatIntervalForProgressBar(RenderProgress*) const override;
     virtual double animationDurationForProgressBar(RenderProgress*) const override;
 
     // These methods define the padding for the MenuList's inner block.
-    virtual int popupInternalPaddingLeft(RenderStyle*) const override;
-    virtual int popupInternalPaddingRight(RenderStyle*) const override;
-    virtual int popupInternalPaddingTop(RenderStyle*) const override;
-    virtual int popupInternalPaddingBottom(RenderStyle*) const override;
+    virtual int popupInternalPaddingLeft(const RenderStyle&) const override;
+    virtual int popupInternalPaddingRight(const RenderStyle&) const override;
+    virtual int popupInternalPaddingTop(const RenderStyle&) const override;
+    virtual int popupInternalPaddingBottom(const RenderStyle&) const override;
 
     // Provide a way to pass the default font size from the Settings object
     // to the render theme. FIXME: http://b/1129186 A cleaner way would be
     // to remove the default font size from this object and have callers
     // that need the value to get it directly from the appropriate Settings
     // object.
     static void setDefaultFontSize(int);
 
 protected:
     virtual double caretBlinkIntervalInternal() const;
@@ skipping 10 matching lines @@
         ~DirectionFlippingScope();
 
     private:
         bool m_needsFlipping;
         const PaintInfo& m_paintInfo;
     };
 
 private:
     virtual bool shouldShowPlaceholderWhenFocused() const override;
 
-    int menuListInternalPadding(RenderStyle*, int paddingType) const;
+    int menuListInternalPadding(const RenderStyle&, int paddingType) const;
     bool paintMediaButtonInternal(GraphicsContext*, const IntRect&, Image*);
     IntRect convertToPaintingRect(RenderObject* inputRenderer, const RenderObject* partRenderer, LayoutRect partRect, const IntRect& localOffset) const;
 
     static const RGBA32 defaultTapHighlightColor = 0x2e000000; // 18% black.
 };
 
 } // namespace blink
 
 #endif // LayoutThemeChromiumSkia_h