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

Unified Diff: Source/core/rendering/RenderBox.h

Issue 337473008: Refactor RenderBox::addVisualEffectOverflow() (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | Source/core/rendering/RenderBox.cpp » ('j') | Source/core/rendering/RenderBox.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/rendering/RenderBox.h
diff --git a/Source/core/rendering/RenderBox.h b/Source/core/rendering/RenderBox.h
index f0af62a93a0a9fd9c19aa6b252f9246fb33a48e9..1db90db31a2d585d4acbdc8a54e7b71912a3b72f 100644
--- a/Source/core/rendering/RenderBox.h
+++ b/Source/core/rendering/RenderBox.h
@@ -225,7 +225,9 @@ public:
// Clipped by the contents clip, if one exists.
void addContentsVisualOverflow(const LayoutRect&);
+ bool hasVisualEffectOverflow() const { return style()->boxShadow() || style()->hasBorderImageOutsets() || style()->hasOutline(); }
Julien - ping for review 2014/07/01 23:00:16 https://codereview.chromium.org/204843002 landed w
Xianzhu 2014/07/02 00:02:03 Done.
void addVisualEffectOverflow();
+ LayoutBoxExtent computeVisualEffectOverflowExtent() const;
void addOverflowFromChild(RenderBox* child) { addOverflowFromChild(child, child->locationOffset()); }
void addOverflowFromChild(RenderBox* child, const LayoutSize& delta);
void clearLayoutOverflow();
« no previous file with comments | « no previous file | Source/core/rendering/RenderBox.cpp » ('j') | Source/core/rendering/RenderBox.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698