Title: [215780] trunk/Source/WebKit2
Revision
215780
Author
simon.fra...@apple.com
Date
2017-04-25 18:11:11 -0700 (Tue, 25 Apr 2017)

Log Message

REGRESSION (r213719): WebGL content on cubeslam.com is 1/4 expected size
https://bugs.webkit.org/show_bug.cgi?id=171300
rdar://problem/31051538

Reviewed by Dean Jackson.

Propagating the contentsScale to the UI process for canvas content layers causes
scaling issues, so don't do it for contents-provided layers.

No test because WebGL can't be tested in the iOS simulator.

* WebProcess/WebPage/mac/PlatformCALayerRemote.cpp:
(WebKit::PlatformCALayerRemote::PlatformCALayerRemote):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (215779 => 215780)


--- trunk/Source/WebKit2/ChangeLog	2017-04-26 00:57:59 UTC (rev 215779)
+++ trunk/Source/WebKit2/ChangeLog	2017-04-26 01:11:11 UTC (rev 215780)
@@ -1,3 +1,19 @@
+2017-04-25  Simon Fraser  <simon.fra...@apple.com>
+
+        REGRESSION (r213719): WebGL content on cubeslam.com is 1/4 expected size
+        https://bugs.webkit.org/show_bug.cgi?id=171300
+        rdar://problem/31051538
+
+        Reviewed by Dean Jackson.
+
+        Propagating the contentsScale to the UI process for canvas content layers causes
+        scaling issues, so don't do it for contents-provided layers.
+        
+        No test because WebGL can't be tested in the iOS simulator.
+
+        * WebProcess/WebPage/mac/PlatformCALayerRemote.cpp:
+        (WebKit::PlatformCALayerRemote::PlatformCALayerRemote):
+
 2017-04-25  Brady Eidson  <beid...@apple.com>
 
         Update WebProcess(Pool) to assume there's always a WebsiteDataStore

Modified: trunk/Source/WebKit2/WebProcess/WebPage/mac/PlatformCALayerRemote.cpp (215779 => 215780)


--- trunk/Source/WebKit2/WebProcess/WebPage/mac/PlatformCALayerRemote.cpp	2017-04-26 00:57:59 UTC (rev 215779)
+++ trunk/Source/WebKit2/WebProcess/WebPage/mac/PlatformCALayerRemote.cpp	2017-04-26 01:11:11 UTC (rev 215780)
@@ -76,7 +76,7 @@
     : PlatformCALayer(layerType, owner)
     , m_context(&context)
 {
-    if (owner) {
+    if (owner && layerType != LayerTypeContentsProvidedLayer) {
         m_properties.contentsScale = owner->platformCALayerDeviceScaleFactor();
         m_properties.notePropertiesChanged(RemoteLayerTreeTransaction::ContentsScaleChanged);
     }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to