Title: [113284] trunk
- Revision
- 113284
- Author
- [email protected]
- Date
- 2012-04-04 20:44:19 -0700 (Wed, 04 Apr 2012)
Log Message
Source/WebCore: https://bugs.webkit.org/show_bug.cgi?id=82994
Reviewed by James Robinson.
Fix an issue when removing elements with reflections from the document.
Test: compositing/reflections/remove-reflection.html
* platform/graphics/GraphicsLayer.cpp:
(WebCore::GraphicsLayer::willBeDestroyed):
LayoutTests: REGRESSION(112939): compositing/reflections/backface-hidden-reflection.html is crashing
https://bugs.webkit.org/show_bug.cgi?id=82994
Reviewed by James Robinson.
Test that removes reflected elements.
* compositing/reflections/remove-reflection-expected.txt: Added.
* compositing/reflections/remove-reflection.html: Added.
Modified Paths
Added Paths
Diff
Modified: trunk/LayoutTests/ChangeLog (113283 => 113284)
--- trunk/LayoutTests/ChangeLog 2012-04-05 03:43:21 UTC (rev 113283)
+++ trunk/LayoutTests/ChangeLog 2012-04-05 03:44:19 UTC (rev 113284)
@@ -1,3 +1,15 @@
+2012-04-04 Simon Fraser <[email protected]>
+
+ REGRESSION(112939): compositing/reflections/backface-hidden-reflection.html is crashing
+ https://bugs.webkit.org/show_bug.cgi?id=82994
+
+ Reviewed by James Robinson.
+
+ Test that removes reflected elements.
+
+ * compositing/reflections/remove-reflection-expected.txt: Added.
+ * compositing/reflections/remove-reflection.html: Added.
+
2012-04-04 Greg Billock <[email protected]>
Switch web intents API to be vendor-prefixed
Added: trunk/LayoutTests/compositing/reflections/remove-reflection-expected.txt (0 => 113284)
--- trunk/LayoutTests/compositing/reflections/remove-reflection-expected.txt (rev 0)
+++ trunk/LayoutTests/compositing/reflections/remove-reflection-expected.txt 2012-04-05 03:44:19 UTC (rev 113284)
@@ -0,0 +1 @@
+Test passes if it did not crash
Added: trunk/LayoutTests/compositing/reflections/remove-reflection.html (0 => 113284)
--- trunk/LayoutTests/compositing/reflections/remove-reflection.html (rev 0)
+++ trunk/LayoutTests/compositing/reflections/remove-reflection.html 2012-04-05 03:44:19 UTC (rev 113284)
@@ -0,0 +1,26 @@
+<style>
+* {
+ -webkit-box-reflect: below 2px;
+}
+
+.composited {
+ -webkit-transform-style: preserve-3d;
+}
+</style>
+
+<div>Test</div>
+<div class="composited">Test</div>
+<div>Test</div>
+<div>Test</div>
+
+<script>
+
+if (window.layoutTestController)
+ layoutTestController.dumpAsText();
+
+window._onload_ = function() {
+ document.execCommand("SelectAll", false, false)
+ window.getSelection().getRangeAt(0).deleteContents();
+ document.body.innerHTML = 'Test passes if it did not crash';
+};
+</script>
Modified: trunk/Source/WebCore/ChangeLog (113283 => 113284)
--- trunk/Source/WebCore/ChangeLog 2012-04-05 03:43:21 UTC (rev 113283)
+++ trunk/Source/WebCore/ChangeLog 2012-04-05 03:44:19 UTC (rev 113284)
@@ -1,3 +1,16 @@
+2012-04-04 Simon Fraser <[email protected]>
+
+ https://bugs.webkit.org/show_bug.cgi?id=82994
+
+ Reviewed by James Robinson.
+
+ Fix an issue when removing elements with reflections from the document.
+
+ Test: compositing/reflections/remove-reflection.html
+
+ * platform/graphics/GraphicsLayer.cpp:
+ (WebCore::GraphicsLayer::willBeDestroyed):
+
2012-04-04 Charles Wei <[email protected]>
Need using namespace std for system calls in MIMESniffing.cpp
Modified: trunk/Source/WebCore/platform/graphics/GraphicsLayer.cpp (113283 => 113284)
--- trunk/Source/WebCore/platform/graphics/GraphicsLayer.cpp 2012-04-05 03:43:21 UTC (rev 113283)
+++ trunk/Source/WebCore/platform/graphics/GraphicsLayer.cpp 2012-04-05 03:44:19 UTC (rev 113284)
@@ -105,6 +105,9 @@
m_client->verifyNotPainting();
#endif
+ if (m_replicaLayer)
+ m_replicaLayer->setReplicatedLayer(0);
+
if (m_replicatedLayer)
m_replicatedLayer->setReplicatedByLayer(0);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
