Title: [160740] trunk/Source/WebKit2
Revision
160740
Author
[email protected]
Date
2013-12-17 17:05:50 -0800 (Tue, 17 Dec 2013)

Log Message

WebKit2 View Gestures: Fix the shadow layer restore-from-overpinched animation
https://bugs.webkit.org/show_bug.cgi?id=125884

Reviewed by Anders Carlsson.

* WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm:
(WebKit::TiledCoreAnimationDrawingArea::commitTransientZoom):
Set to-values on the right animations.

(WebKit::TiledCoreAnimationDrawingArea::adjustTransientZoom):
(WebKit::TiledCoreAnimationDrawingArea::applyTransientZoomToPage):
We don't need to explicitly set the shadow path, because PlatformCALayer::setBounds does it for us now.

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (160739 => 160740)


--- trunk/Source/WebKit2/ChangeLog	2013-12-18 00:57:44 UTC (rev 160739)
+++ trunk/Source/WebKit2/ChangeLog	2013-12-18 01:05:50 UTC (rev 160740)
@@ -1,3 +1,18 @@
+2013-12-17  Tim Horton  <[email protected]>
+
+        WebKit2 View Gestures: Fix the shadow layer restore-from-overpinched animation
+        https://bugs.webkit.org/show_bug.cgi?id=125884
+
+        Reviewed by Anders Carlsson.
+
+        * WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm:
+        (WebKit::TiledCoreAnimationDrawingArea::commitTransientZoom):
+        Set to-values on the right animations.
+
+        (WebKit::TiledCoreAnimationDrawingArea::adjustTransientZoom):
+        (WebKit::TiledCoreAnimationDrawingArea::applyTransientZoomToPage):
+        We don't need to explicitly set the shadow path, because PlatformCALayer::setBounds does it for us now.
+
 2013-12-17  Sam Weinig  <[email protected]>
 
         [WK2] Rename WebURLRequest to API::URLRequest

Modified: trunk/Source/WebKit2/WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm (160739 => 160740)


--- trunk/Source/WebKit2/WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm	2013-12-18 00:57:44 UTC (rev 160739)
+++ trunk/Source/WebKit2/WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm	2013-12-18 01:05:50 UTC (rev 160740)
@@ -733,7 +733,6 @@
 
     shadowLayer->setBounds(shadowBounds);
     shadowLayer->setPosition(origin + shadowBounds.center());
-    shadowLayer->platformLayer().shadowPath = adoptCF(CGPathCreateWithRect(shadowBounds, NULL)).get();
 
     m_transientZoomScale = scale;
 }
@@ -795,9 +794,9 @@
     RetainPtr<CABasicAnimation> shadowBoundsAnimation = transientZoomSnapAnimationForKeyPath("bounds");
     [shadowBoundsAnimation setToValue:[NSValue valueWithRect:shadowBounds]];
     RetainPtr<CABasicAnimation> shadowPositionAnimation = transientZoomSnapAnimationForKeyPath("position");
-    [shadowBoundsAnimation setToValue:[NSValue valueWithPoint:constrainedOrigin + shadowBounds.center()]];
+    [shadowPositionAnimation setToValue:[NSValue valueWithPoint:constrainedOrigin + shadowBounds.center()]];
     RetainPtr<CABasicAnimation> shadowPathAnimation = transientZoomSnapAnimationForKeyPath("shadowPath");
-    [shadowBoundsAnimation setToValue:(id)shadowPath.get()];
+    [shadowPathAnimation setToValue:(id)shadowPath.get()];
 
     [CATransaction begin];
     [CATransaction setCompletionBlock:^(void) {
@@ -830,7 +829,6 @@
     IntRect overflowRect = renderView->pixelSnappedLayoutOverflowRect();
     shadowLayer->setBounds(IntRect(IntPoint(), toIntSize(overflowRect.maxXMaxYCorner())));
     shadowLayer->setPosition(shadowLayer->bounds().center());
-    ScrollbarTheme::theme()->setUpContentShadowLayer(renderView->compositor().layerForContentShadow());
 
     FloatPoint unscrolledOrigin(origin);
     FloatRect visibleContentRect = m_webPage->mainFrameView()->visibleContentRect(ScrollableArea::IncludeScrollbars);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to