Diff
Modified: trunk/LayoutTests/ChangeLog (172290 => 172291)
--- trunk/LayoutTests/ChangeLog 2014-08-07 22:22:13 UTC (rev 172290)
+++ trunk/LayoutTests/ChangeLog 2014-08-07 22:25:11 UTC (rev 172291)
@@ -1,3 +1,25 @@
+2014-08-07 Beth Dakin <[email protected]>
+
+ Fixed backgrounds don't paint in blurred inset areas
+ https://bugs.webkit.org/show_bug.cgi?id=135712
+
+ Reviewed by Simon Fraser.
+
+ Unfortunately for this test to be valid in WK2, it needs to be run from the
+ platform/mac-wk2/tiled-drawing directory so that tiled drawing is enabled. So
+ there are two copies of the test so that we can get coverage for WK1 and WK2.
+ * platform/mac-wk1/TestExpectations:
+ * platform/mac-wk1/fast: Added.
+ * platform/mac-wk1/fast/backgrounds: Added.
+ * platform/mac-wk1/fast/backgrounds/resources: Added.
+ * platform/mac-wk1/fast/backgrounds/resources/green.png: Added.
+ * platform/mac-wk1/fast/backgrounds/top-content-inset-fixed-attachment-expected.html: Added.
+ * platform/mac-wk1/fast/backgrounds/top-content-inset-fixed-attachment.html: Added.
+ * platform/mac-wk2/tiled-drawing/resources/green.png: Added.
+ * platform/mac-wk2/tiled-drawing/top-content-inset-fixed-attachment-expected.html: Added.
+ * platform/mac-wk2/tiled-drawing/top-content-inset-fixed-attachment.html: Added.
+ * platform/mac/fast/backgrounds/resources: Added.
+
2014-08-07 Mark Lam <[email protected]>
http/tests/security/cross-frame-access-enumeration.html fails after FTLOPT merge.
Modified: trunk/LayoutTests/platform/mac-wk1/TestExpectations (172290 => 172291)
--- trunk/LayoutTests/platform/mac-wk1/TestExpectations 2014-08-07 22:22:13 UTC (rev 172290)
+++ trunk/LayoutTests/platform/mac-wk1/TestExpectations 2014-08-07 22:25:11 UTC (rev 172291)
@@ -51,6 +51,7 @@
[ MountainLion Mavericks ] platform/mac/fast/events/content-inset-hit-testing.html [ Skip ]
[ MountainLion Mavericks ] platform/mac/fast/events/content-inset-hit-testing-in-frame.html [ Skip ]
[ MountainLion Mavericks ] platform/mac/fast/scrolling/scroll-select-bottom-test.html [ Skip ]
+[ MountainLion Mavericks ] platform/mac-wk1/fast/backgrounds/top-content-inset-fixed-attachment.html [ Skip ]
### END OF (2) Failures without bug reports
########################################
Added: trunk/LayoutTests/platform/mac-wk1/fast/backgrounds/resources/green.png
(Binary files differ)
Added: svn:mime-type
Added: trunk/LayoutTests/platform/mac-wk1/fast/backgrounds/top-content-inset-fixed-attachment-expected.html (0 => 172291)
--- trunk/LayoutTests/platform/mac-wk1/fast/backgrounds/top-content-inset-fixed-attachment-expected.html (rev 0)
+++ trunk/LayoutTests/platform/mac-wk1/fast/backgrounds/top-content-inset-fixed-attachment-expected.html 2014-08-07 22:25:11 UTC (rev 172291)
@@ -0,0 +1,26 @@
+<html>
+<head>
+<style>
+
+body {
+ height:2000px;
+ margin:0;
+ overflow:hidden;
+}
+
+#fixed-background-box {
+ width:400px;
+ height:600px;
+ background-image:url(resources/green.png);
+ background-attachment:fixed;
+}
+
+</style>
+</head>
+
+<body>
+
+<div id="fixed-background-box"></div>
+
+</body>
+<html>
Added: trunk/LayoutTests/platform/mac-wk1/fast/backgrounds/top-content-inset-fixed-attachment.html (0 => 172291)
--- trunk/LayoutTests/platform/mac-wk1/fast/backgrounds/top-content-inset-fixed-attachment.html (rev 0)
+++ trunk/LayoutTests/platform/mac-wk1/fast/backgrounds/top-content-inset-fixed-attachment.html 2014-08-07 22:25:11 UTC (rev 172291)
@@ -0,0 +1,36 @@
+<html>
+<head>
+<style>
+
+body {
+ height:2000px;
+ margin:0;
+ overflow:hidden;
+}
+
+#fixed-background-box {
+ width:400px;
+ height:600px;
+ background-image:url(resources/green.png);
+ background-attachment:fixed;
+}
+
+</style>
+
+<script>
+function runTest() {
+ if (window.internals)
+ window.internals.setTopContentInset(100);
+ window.scrollTo(0, 100);
+}
+
+window.addEventListener('load', runTest, false);
+</script>
+</head>
+
+<body>
+
+<div id="fixed-background-box"></div>
+
+</body>
+<html>
Added: trunk/LayoutTests/platform/mac-wk2/tiled-drawing/resources/green.png
(Binary files differ)
Added: svn:mime-type
Added: trunk/LayoutTests/platform/mac-wk2/tiled-drawing/top-content-inset-fixed-attachment-expected.html (0 => 172291)
--- trunk/LayoutTests/platform/mac-wk2/tiled-drawing/top-content-inset-fixed-attachment-expected.html (rev 0)
+++ trunk/LayoutTests/platform/mac-wk2/tiled-drawing/top-content-inset-fixed-attachment-expected.html 2014-08-07 22:25:11 UTC (rev 172291)
@@ -0,0 +1,26 @@
+<html>
+<head>
+<style>
+
+body {
+ height:2000px;
+ margin:0;
+ overflow:hidden;
+}
+
+#fixed-background-box {
+ width:400px;
+ height:600px;
+ background-image:url(resources/green.png);
+ background-attachment:fixed;
+}
+
+</style>
+</head>
+
+<body>
+
+<div id="fixed-background-box"></div>
+
+</body>
+<html>
Added: trunk/LayoutTests/platform/mac-wk2/tiled-drawing/top-content-inset-fixed-attachment.html (0 => 172291)
--- trunk/LayoutTests/platform/mac-wk2/tiled-drawing/top-content-inset-fixed-attachment.html (rev 0)
+++ trunk/LayoutTests/platform/mac-wk2/tiled-drawing/top-content-inset-fixed-attachment.html 2014-08-07 22:25:11 UTC (rev 172291)
@@ -0,0 +1,36 @@
+<html>
+<head>
+<style>
+
+body {
+ height:2000px;
+ margin:0;
+ overflow:hidden;
+}
+
+#fixed-background-box {
+ width:400px;
+ height:600px;
+ background-image:url(resources/green.png);
+ background-attachment:fixed;
+}
+
+</style>
+
+<script>
+function runTest() {
+ if (window.internals)
+ window.internals.setTopContentInset(100);
+ window.scrollTo(0, 100);
+}
+
+window.addEventListener('load', runTest, false);
+</script>
+</head>
+
+<body>
+
+<div id="fixed-background-box"></div>
+
+</body>
+<html>
Modified: trunk/Source/WebCore/ChangeLog (172290 => 172291)
--- trunk/Source/WebCore/ChangeLog 2014-08-07 22:22:13 UTC (rev 172290)
+++ trunk/Source/WebCore/ChangeLog 2014-08-07 22:25:11 UTC (rev 172291)
@@ -1,3 +1,16 @@
+2014-08-07 Beth Dakin <[email protected]>
+
+ Fixed backgrounds don't paint in blurred inset areas
+ https://bugs.webkit.org/show_bug.cgi?id=135712
+
+ Reviewed by Simon Fraser.
+
+ Background geometry calculations for fixed background need to use the larger
+ visible rect.
+
+ * rendering/RenderBoxModelObject.cpp:
+ (WebCore::RenderBoxModelObject::calculateBackgroundImageGeometry):
+
2014-08-07 Jer Noble <[email protected]>
[EME][Mac] Error codes returned by AVSampleBufferDisplayLayer are negative; clip when passed to _javascript_ as doubles.
Modified: trunk/Source/WebCore/rendering/RenderBoxModelObject.cpp (172290 => 172291)
--- trunk/Source/WebCore/rendering/RenderBoxModelObject.cpp 2014-08-07 22:22:13 UTC (rev 172290)
+++ trunk/Source/WebCore/rendering/RenderBoxModelObject.cpp 2014-08-07 22:25:11 UTC (rev 172291)
@@ -1127,11 +1127,12 @@
if (frame().settings().fixedBackgroundsPaintRelativeToDocument())
viewportRect = view().unscaledDocumentRect();
else {
- viewportRect = view().viewRect();
+ viewportRect.setSize(view().frameView().unscaledVisibleContentSizeIncludingObscuredArea());
+ top += view().frameView().topContentInset(ScrollView::TopContentInsetType::WebCoreOrPlatformContentInset);
if (fixedBackgroundPaintsInLocalCoordinates())
viewportRect.setLocation(LayoutPoint());
else
- viewportRect.setLocation(toLayoutPoint(view().frameView().scrollOffsetForFixedPosition()));
+ viewportRect.setLocation(toLayoutPoint(view().frameView().documentScrollOffsetRelativeToViewOrigin()));
}
if (paintContainer)
