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

Unified Diff: Source/core/rendering/RenderObject.cpp

Issue 176953008: Include the outline into the visual overflow (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 10 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
Index: Source/core/rendering/RenderObject.cpp
diff --git a/Source/core/rendering/RenderObject.cpp b/Source/core/rendering/RenderObject.cpp
index 595041c330ef6cc29bb42dc55922f705bb3a283a..1e01ca81c1c04a30cf8c8afd40169bca9ffcda9a 100644
--- a/Source/core/rendering/RenderObject.cpp
+++ b/Source/core/rendering/RenderObject.cpp
@@ -1912,12 +1912,6 @@ void RenderObject::setStyle(PassRefPtr<RenderStyle> style)
updateShapeImage(oldStyle ? oldStyle->shapeInside() : 0, m_style ? m_style->shapeInside() : 0);
updateShapeImage(oldStyle ? oldStyle->shapeOutside() : 0, m_style ? m_style->shapeOutside() : 0);
- // We need to ensure that view->maximalOutlineSize() is valid for any repaints that happen
- // during styleDidChange (it's used by clippedOverflowRectForRepaint()).
- // FIXME: Do this more cleanly. http://crbug.com/273904
- if (m_style->outlineWidth() > 0 && m_style->outlineSize() > view()->maximalOutlineSize())
- view()->setMaximalOutlineSize(m_style->outlineSize());
-
bool doesNotNeedLayout = !m_parent || isText();
styleDidChange(diff, oldStyle.get());
@@ -3047,13 +3041,6 @@ bool RenderObject::willRenderImage(ImageResource*)
return document().view()->isVisible();
}
-int RenderObject::maximalOutlineSize(PaintPhase p) const
-{
- if (p != PaintPhaseOutline && p != PaintPhaseSelfOutline && p != PaintPhaseChildOutlines)
- return 0;
- return view()->maximalOutlineSize();
-}
-
int RenderObject::caretMinOffset() const
{
return 0;

Powered by Google App Engine
This is Rietveld 408576698