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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutBox.cpp

Issue 2068723002: Paint local attachment backgrounds into composited scrolling contents layer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merge allLayersAreLocal and ComputedStyle::hasEntirelyLocalBackground Created 4 years, 4 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: third_party/WebKit/Source/core/layout/LayoutBox.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutBox.cpp b/third_party/WebKit/Source/core/layout/LayoutBox.cpp
index ed8fbdf554cf772335cb0ce3606ee482438e9846..cff61ddc8abcb125cd00722cf17b4b66b4b65126 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBox.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutBox.cpp
@@ -707,15 +707,53 @@ FloatQuad LayoutBox::absoluteContentQuad() const
return localToAbsoluteQuad(FloatRect(rect));
}
-LayoutRect LayoutBox::backgroundClipRect() const
-{
- // TODO(flackr): Check for the maximum background clip rect.
- switch (style()->backgroundClip()) {
+LayoutRect LayoutBox::backgroundRect(BackgroundRectType rectType) const
+{
+ EFillBox backgroundBox = TextFillBox;
+ // Find the largest background rect of the given opaqueness.
+ if (const FillLayer* current = &(style()->backgroundLayers())) {
+ do {
+ const FillLayer* cur = current;
+ current = current->next();
+ if (rectType == BackgroundKnownOpaqueRect) {
+ if (cur->blendMode() != WebBlendModeNormal || cur->composite() != CompositeSourceOver)
+ continue;
+
+ bool layerKnownOpaque = false;
+ // Check if the image or color has alpha.
+ if (const StyleImage* image = cur->image()) {
+ if (image->knownToBeOpaque(*this))
+ layerKnownOpaque = true;
+ }
+
+ // The background color is painted into the last layer.
+ if (!cur->next()) {
+ Color backgroundColor = resolveColor(CSSPropertyBackgroundColor);
+ if (!backgroundColor.hasAlpha())
+ layerKnownOpaque = true;
+ }
+
+ // If neither the image nor the color are opaque then skip this layer.
+ if (!layerKnownOpaque)
+ continue;
+ }
+ EFillBox currentClip = cur->clip();
+ // Restrict clip if attachment is local.
+ if (currentClip == BorderFillBox && cur->attachment() == LocalBackgroundAttachment)
+ currentClip = PaddingFillBox;
+
+ // If we're asking for the clip rect, a content-box clipped fill layer can be scrolled
+ // into the padding box of the overflow container.
+ if (rectType == BackgroundClipRect
+ && currentClip == ContentFillBox && cur->attachment() == LocalBackgroundAttachment) {
+ currentClip = PaddingFillBox;
+ }
+
+ backgroundBox = enclosingFillBox(backgroundBox, currentClip);
+ } while (current);
+ }
+ switch (backgroundBox) {
case BorderFillBox:
- // A 'border-box' clip on scrollable elements local attachment is treated as 'padding-box'.
- // https://www.w3.org/TR/css3-background/#the-background-attachment
- if (!style()->isOverflowVisible() && style()->backgroundLayers().attachment() == LocalBackgroundAttachment)
- return paddingBoxRect();
return borderBoxRect();
break;
case PaddingFillBox:
@@ -1396,10 +1434,6 @@ bool LayoutBox::backgroundIsKnownToBeOpaqueInRect(const LayoutRect& localRect) c
if (isDocumentElement() || backgroundStolenForBeingBody())
return false;
- Color backgroundColor = resolveColor(CSSPropertyBackgroundColor);
- if (backgroundColor.hasAlpha())
- return false;
-
// If the element has appearance, it might be painted by theme.
// We cannot be sure if theme paints the background opaque.
// In this case it is safe to not assume opaqueness.
@@ -1413,10 +1447,9 @@ bool LayoutBox::backgroundIsKnownToBeOpaqueInRect(const LayoutRect& localRect) c
return false;
if (hasClipPath())
return false;
- // FIXME: The background color clip is defined by the last layer.
- if (style()->backgroundLayers().next())
+ if (style()->hasBlendMode())
return false;
- return backgroundClipRect().contains(localRect);
+ return backgroundRect(BackgroundKnownOpaqueRect).contains(localRect);
}
static bool isCandidateForOpaquenessTest(const LayoutBox& childBox)
@@ -1784,13 +1817,12 @@ void LayoutBox::mapAncestorToLocal(const LayoutBoxModelObject* ancestor, Transfo
bool isFixedPos = style()->position() == FixedPosition;
- if (style()->canContainFixedPositionObjects() && !isFixedPos) {
- // If this box has a transform or contains paint, it acts as a fixed position container for fixed descendants,
- // and may itself also be fixed position. So propagate 'fixed' up only if this box is fixed position.
+ // If this box has a transform or contains paint, it acts as a fixed position container for fixed descendants,
+ // and may itself also be fixed position. So propagate 'fixed' up only if this box is fixed position.
+ if (style()->canContainFixedPositionObjects() && !isFixedPos)
mode &= ~IsFixed;
- } else if (isFixedPos) {
+ else if (isFixedPos)
mode |= IsFixed;
- }
LayoutBoxModelObject::mapAncestorToLocal(ancestor, transformState, mode);
}

Powered by Google App Engine
This is Rietveld 408576698