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

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

Issue 669553002: Remove <marquee> leftovers (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: rebase Created 6 years, 2 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 | « Source/core/rendering/RenderBlock.cpp ('k') | Source/core/rendering/RenderLayerScrollableArea.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/rendering/RenderBox.cpp
diff --git a/Source/core/rendering/RenderBox.cpp b/Source/core/rendering/RenderBox.cpp
index 7723e8b3de46741b10bbf1561a409f9c1ff84074..f3c1e57d1dbe502db0f34198f5866380987f9510 100644
--- a/Source/core/rendering/RenderBox.cpp
+++ b/Source/core/rendering/RenderBox.cpp
@@ -1995,25 +1995,12 @@ static bool isStretchingColumnFlexItem(const RenderObject* flexitem)
bool RenderBox::sizesLogicalWidthToFitContent(const Length& logicalWidth) const
{
- // Marquees in WinIE are like a mixture of blocks and inline-blocks. They size as though they're blocks,
- // but they allow text to sit on the same line as the marquee.
- if (isFloating() || (isInlineBlockOrInlineTable() && !isMarquee()))
+ if (isFloating() || isInlineBlockOrInlineTable())
return true;
if (logicalWidth.type() == Intrinsic)
return true;
- // Children of a horizontal marquee do not fill the container by default.
- // FIXME: Need to deal with MAUTO value properly. It could be vertical.
- // FIXME: Think about block-flow here. Need to find out how marquee direction relates to
- // block-flow (as well as how marquee overflow should relate to block flow).
- // https://bugs.webkit.org/show_bug.cgi?id=46472
- if (parent()->isMarquee()) {
- EMarqueeDirection dir = parent()->style()->marqueeDirection();
- if (dir == MAUTO || dir == MFORWARD || dir == MBACKWARD || dir == MLEFT || dir == MRIGHT)
- return true;
- }
-
// Flexible box items should shrink wrap, so we lay them out at their intrinsic widths.
// In the case of columns that have a stretch alignment, we go ahead and layout at the
// stretched size to avoid an extra layout when applying alignment.
@@ -3754,8 +3741,8 @@ PositionWithAffinity RenderBox::positionForPoint(const LayoutPoint& point)
bool RenderBox::shrinkToAvoidFloats() const
{
- // Floating objects don't shrink. Objects that don't avoid floats don't shrink. Marquees don't shrink.
- if ((isInline() && !isMarquee()) || !avoidsFloats() || isFloating())
+ // Floating objects don't shrink. Objects that don't avoid floats don't shrink.
+ if (isInline() || !avoidsFloats() || isFloating())
return false;
// Only auto width objects can possibly shrink to avoid floats.
« no previous file with comments | « Source/core/rendering/RenderBlock.cpp ('k') | Source/core/rendering/RenderLayerScrollableArea.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698