Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(23)

Side by Side Diff: Source/core/rendering/RenderBox.h

Issue 559733005: Move painting code from RenderBoxModelObject into BoxPainter. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * Copyright (C) 2003, 2006, 2007 Apple Inc. All rights reserved. 4 * Copyright (C) 2003, 2006, 2007 Apple Inc. All rights reserved.
5 * 5 *
6 * This library is free software; you can redistribute it and/or 6 * This library is free software; you can redistribute it and/or
7 * modify it under the terms of the GNU Library General Public 7 * modify it under the terms of the GNU Library General Public
8 * License as published by the Free Software Foundation; either 8 * License as published by the Free Software Foundation; either
9 * version 2 of the License, or (at your option) any later version. 9 * version 2 of the License, or (at your option) any later version.
10 * 10 *
(...skipping 614 matching lines...) Expand 10 before | Expand all | Expand 10 after
625 625
626 protected: 626 protected:
627 virtual void willBeDestroyed() OVERRIDE; 627 virtual void willBeDestroyed() OVERRIDE;
628 628
629 629
630 virtual void styleWillChange(StyleDifference, const RenderStyle& newStyle) O VERRIDE; 630 virtual void styleWillChange(StyleDifference, const RenderStyle& newStyle) O VERRIDE;
631 virtual void styleDidChange(StyleDifference, const RenderStyle* oldStyle) OV ERRIDE; 631 virtual void styleDidChange(StyleDifference, const RenderStyle* oldStyle) OV ERRIDE;
632 virtual void updateFromStyle() OVERRIDE; 632 virtual void updateFromStyle() OVERRIDE;
633 633
634 // Returns false if it could not cheaply compute the extent (e.g. fixed back ground), in which case the returned rect may be incorrect. 634 // Returns false if it could not cheaply compute the extent (e.g. fixed back ground), in which case the returned rect may be incorrect.
635 bool getBackgroundPaintedExtent(LayoutRect&) const; 635 bool getBackgroundPaintedExtent(LayoutRect&);
chrishtr 2014/09/10 21:56:32 This is fallout from BoxPainter::m_renderBox not y
pdr. 2014/09/11 05:59:13 Could we add a fixme here so we don't forget to fi
chrishtr 2014/09/11 17:45:53 Done.
636 virtual bool foregroundIsKnownToBeOpaqueInRect(const LayoutRect& localRect, unsigned maxDepthToTest) const; 636 virtual bool foregroundIsKnownToBeOpaqueInRect(const LayoutRect& localRect, unsigned maxDepthToTest) const;
637 virtual bool computeBackgroundIsKnownToBeObscured() OVERRIDE; 637 virtual bool computeBackgroundIsKnownToBeObscured() OVERRIDE;
638 638
639 void computePositionedLogicalWidth(LogicalExtentComputedValues&) const; 639 void computePositionedLogicalWidth(LogicalExtentComputedValues&) const;
640 640
641 LayoutUnit computeIntrinsicLogicalWidthUsing(const Length& logicalWidthLengt h, LayoutUnit availableLogicalWidth, LayoutUnit borderAndPadding) const; 641 LayoutUnit computeIntrinsicLogicalWidthUsing(const Length& logicalWidthLengt h, LayoutUnit availableLogicalWidth, LayoutUnit borderAndPadding) const;
642 LayoutUnit computeIntrinsicLogicalContentHeightUsing(const Length& logicalHe ightLength, LayoutUnit intrinsicContentHeight, LayoutUnit borderAndPadding) cons t; 642 LayoutUnit computeIntrinsicLogicalContentHeightUsing(const Length& logicalHe ightLength, LayoutUnit intrinsicContentHeight, LayoutUnit borderAndPadding) cons t;
643 643
644 virtual bool shouldComputeSizeAsReplaced() const { return isReplaced() && !i sInlineBlockOrInlineTable(); } 644 virtual bool shouldComputeSizeAsReplaced() const { return isReplaced() && !i sInlineBlockOrInlineTable(); }
645 645
(...skipping 135 matching lines...) Expand 10 before | Expand all | Expand 10 after
781 if (UNLIKELY(inlineBoxWrapper() != 0)) 781 if (UNLIKELY(inlineBoxWrapper() != 0))
782 deleteLineBoxWrapper(); 782 deleteLineBoxWrapper();
783 } 783 }
784 784
785 ensureRareData().m_inlineBoxWrapper = boxWrapper; 785 ensureRareData().m_inlineBoxWrapper = boxWrapper;
786 } 786 }
787 787
788 } // namespace blink 788 } // namespace blink
789 789
790 #endif // RenderBox_h 790 #endif // RenderBox_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698