Bug 139314 - [iOS] DumpRenderTree produces many reference test mismatches
Summary: [iOS] DumpRenderTree produces many reference test mismatches
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: iPhone / iPad iOS 8.1
: P2 Normal
Assignee: Daniel Bates
URL:
Keywords: InRadar
Depends on: 137581
Blocks:
  Show dependency treegraph
 
Reported: 2014-12-05 14:02 PST by Daniel Bates
Modified: 2014-12-05 15:40 PST (History)
3 users (show)

See Also:


Attachments
Patch (8.64 KB, patch)
2014-12-05 14:07 PST, Daniel Bates
simon.fraser: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Bates 2014-12-05 14:02:35 PST
Many reference tests fail due to a scale factor mismatch in the snapshot of the test and the snapshot of its expected result. You can see such mismatches by using run-webkit-tests to run the following set of tests in the iOS Simulator with --fully-parallel, --child-processes=3, --no-retry-failures, --no-sample-on-timeout (*): LayoutTests/css2.1/20110323/at-import-001.htm, LayoutTests/css2.1/20110323/at-import-002.htm, LayoutTests/css2.1/20110323/at-import-003.htm, and LayoutTests/css2.1/20110323/at-import-004.htm.

(*) Pass these tests as command line arguments of a single run-webkit-tests run. That is, run something of the form: Tools/Scripts/run-webkit-tests --ios-sim --fully-parallel --child-processes=3 --no-retry-failures --no-sample-on-timeout ... LayoutTests/css2.1/20110323/at-import-001.htm LayoutTests/css2.1/20110323/at-import-002.htm LayoutTests/css2.1/20110323/at-import-003.htm LayoutTests/css2.1/20110323/at-import-004.htm.
Comment 1 Daniel Bates 2014-12-05 14:07:49 PST
Created attachment 242661 [details]
Patch
Comment 2 Simon Fraser (smfr) 2014-12-05 14:24:26 PST
Comment on attachment 242661 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=242661&action=review

> Tools/DumpRenderTree/ios/PixelDumpSupportIOS.mm:88
> +    UIGraphicsBeginImageContextWithOptions([[mainFrame webView] frame].size, TRUE, WKGetScreenScaleFactor());

TRUE -> YES

> Tools/DumpRenderTree/mac/Configurations/DumpRenderTreeApp.xcconfig:31
> +OTHER_LDFLAGS_iphoneos = $(inherited) -l$(WEBKIT_SYSTEM_INTERFACE_LIBRARY) -lWebCoreTestSupport -force_load $(BUILT_PRODUCTS_DIR)/libDumpRenderTree.a -framework QuartzCore -framework CFNetwork -framework CoreFoundation -framework CoreGraphics -framework Foundation -framework GraphicsServices -framework ImageIO -framework MobileCoreServices -lMobileGestalt -framework UIKit -framework WebCore -framework JavaScriptCore -framework WebKit $(OTHER_LDFLAGS_FONTS);

Why this change?
Comment 3 Daniel Bates 2014-12-05 15:24:59 PST
(In reply to comment #2)
> > Tools/DumpRenderTree/ios/PixelDumpSupportIOS.mm:88
> > +    UIGraphicsBeginImageContextWithOptions([[mainFrame webView] frame].size, TRUE, WKGetScreenScaleFactor());
> 
> TRUE -> YES
> 

Will fix.

> > Tools/DumpRenderTree/mac/Configurations/DumpRenderTreeApp.xcconfig:31
> > +OTHER_LDFLAGS_iphoneos = $(inherited) -l$(WEBKIT_SYSTEM_INTERFACE_LIBRARY) -lWebCoreTestSupport -force_load $(BUILT_PRODUCTS_DIR)/libDumpRenderTree.a -framework QuartzCore -framework CFNetwork -framework CoreFoundation -framework CoreGraphics -framework Foundation -framework GraphicsServices -framework ImageIO -framework MobileCoreServices -lMobileGestalt -framework UIKit -framework WebCore -framework JavaScriptCore -framework WebKit $(OTHER_LDFLAGS_FONTS);
> 
> Why this change?

Will revert this change. Will fix WebKitSystemInterface for iOS to avoid linking library MobileGestalt.
Comment 4 Daniel Bates 2014-12-05 15:33:25 PST
Committed r176880: <http://trac.webkit.org/changeset/176880>
Comment 5 Daniel Bates 2014-12-05 15:40:17 PST
<rdar://problem/18642889>