Commit 550fc89a authored by Rune Lillesveen's avatar Rune Lillesveen Committed by Commit Bot

Visual change on background-clip changes.

The background-clip property affects background-color visually, so we
need to say a layer is visually different even without an image.

We had a wpt test for this which had a stray line which made the test
always pass.

Bug: 887350
Change-Id: Icc585f9e631485970b6509e015eff1e2007e4f0b
Reviewed-on: https://chromium-review.googlesource.com/1238448Reviewed-by: default avatarAnders Ruud <andruud@chromium.org>
Reviewed-by: default avatarFredrik Söderquist <fs@opera.com>
Commit-Queue: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#593523}
parent f73f0df8
...@@ -31,5 +31,4 @@ ...@@ -31,5 +31,4 @@
document.documentElement.classList.remove("reftest-wait"); document.documentElement.classList.remove("reftest-wait");
}); });
}); });
inner.style.backgroundClip = "border-box";
</script> </script>
...@@ -189,6 +189,8 @@ bool FillLayer::VisuallyEqual(const FillLayer& o) const { ...@@ -189,6 +189,8 @@ bool FillLayer::VisuallyEqual(const FillLayer& o) const {
if (image_ || o.image_) { if (image_ || o.image_) {
if (!LayerPropertiesEqual(o)) if (!LayerPropertiesEqual(o))
return false; return false;
} else if (clip_ != o.clip_) {
return false;
} }
if (next_ && o.next_) if (next_ && o.next_)
return next_->VisuallyEqual(*o.next_); return next_->VisuallyEqual(*o.next_);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment