Title: [176897] trunk/Tools
Revision
176897
Author
[email protected]
Date
2014-12-05 17:09:52 -0800 (Fri, 05 Dec 2014)

Log Message

[iOS] Query -[WAKWindow screenScale] instead of using WKGetScreenScaleFactor()
https://bugs.webkit.org/show_bug.cgi?id=139326

Reviewed by Anders Carlsson.

Ask the window for its screen scale factor instead of calling WKGetScreenScaleFactor()
to query the OS for it. This approach will ensure DumpRenderTree takes the snapshot
using the same scale factor as used in WebKit.

* DumpRenderTree/ios/PixelDumpSupportIOS.mm:
(createBitmapContextFromWebView):

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (176896 => 176897)


--- trunk/Tools/ChangeLog	2014-12-06 01:09:33 UTC (rev 176896)
+++ trunk/Tools/ChangeLog	2014-12-06 01:09:52 UTC (rev 176897)
@@ -1,3 +1,17 @@
+2014-12-05  Daniel Bates  <[email protected]>
+
+        [iOS] Query -[WAKWindow screenScale] instead of using WKGetScreenScaleFactor()
+        https://bugs.webkit.org/show_bug.cgi?id=139326
+
+        Reviewed by Anders Carlsson.
+
+        Ask the window for its screen scale factor instead of calling WKGetScreenScaleFactor()
+        to query the OS for it. This approach will ensure DumpRenderTree takes the snapshot
+        using the same scale factor as used in WebKit.
+
+        * DumpRenderTree/ios/PixelDumpSupportIOS.mm:
+        (createBitmapContextFromWebView):
+
 2014-12-05  Anders Carlsson  <[email protected]>
 
         REGRESSION (r176677): All tests crash on Windows

Modified: trunk/Tools/DumpRenderTree/ios/PixelDumpSupportIOS.mm (176896 => 176897)


--- trunk/Tools/DumpRenderTree/ios/PixelDumpSupportIOS.mm	2014-12-06 01:09:33 UTC (rev 176896)
+++ trunk/Tools/DumpRenderTree/ios/PixelDumpSupportIOS.mm	2014-12-06 01:09:52 UTC (rev 176897)
@@ -30,6 +30,7 @@
 #import "PixelDumpSupport.h"
 
 #import "DumpRenderTree.h"
+#import "DumpRenderTreeWindow.h"
 
 #define COMMON_DIGEST_FOR_OPENSSL
 #import <CommonCrypto/CommonDigest.h>
@@ -39,11 +40,11 @@
 #import <UIKit/UIView_Private.h>
 #import <UIKit/UIWebBrowserView.h>
 #import <WebKit/WebCoreThread.h>
-#import <WebKitSystemInterface.h>
 #import <wtf/RefCounted.h>
 #import <wtf/RefPtr.h>
 #import <wtf/RetainPtr.h>
 
+extern DumpRenderTreeWindow *gDrtWindow;
 extern UIWebBrowserView *gWebBrowserView;
 
 class BitmapContext : public RefCounted<BitmapContext> {
@@ -85,7 +86,7 @@
     [gWebBrowserView layoutIfNeeded]; // Re-enables tile painting, which was disabled when committing the frame load.
     [gWebBrowserView setNeedsDisplay];
 
-    UIGraphicsBeginImageContextWithOptions([[mainFrame webView] frame].size, YES /* opaque */, WKGetScreenScaleFactor());
+    UIGraphicsBeginImageContextWithOptions([[mainFrame webView] frame].size, YES /* opaque */, [gDrtWindow screenScale]);
     [[gWebBrowserView layer] renderInContext:UIGraphicsGetCurrentContext()];
     RefPtr<BitmapContext> context = BitmapContext::createFromUIImage(UIGraphicsGetImageFromCurrentImageContext());
     UIGraphicsEndImageContext();
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to