RESOLVED FIXED 135624
[iOS] Run ImageDiff in the sim bootstrap
https://bugs.webkit.org/show_bug.cgi?id=135624
Summary [iOS] Run ImageDiff in the sim bootstrap
David Farler
Reported 2014-08-05 15:14:14 PDT
Previously, I tried to use ImageDiff on OS X without trying to go through the sim bootstrap. However, ImageDiff relies on some WTF functionality and I don't want to introduce a new OS X build dependency when testing iOS. I've verified that running ImageDiff in the sim bootstrap is negligible, so we can create a custom ImageDiffer and use that for the iso-simulator platform. With this change, the only OS X tool that you'll need to build is the LayoutTestRelay: there are no WebKit dependencies there and it builds automatically when running layout tests.
Attachments
Patch (3.29 KB, patch)
2014-08-05 15:20 PDT, David Farler
ddkilzer: review+
Patch (3.29 KB, patch)
2014-08-05 15:29 PDT, David Farler
no flags
David Farler
Comment 1 2014-08-05 15:20:52 PDT
David Farler
Comment 2 2014-08-05 15:29:41 PDT
David Farler
Comment 3 2014-08-05 15:37:06 PDT
Sorry - uploaded a different patch to the same bug - reinstated original.
Simon Fraser (smfr)
Comment 4 2014-08-05 17:34:57 PDT
What does it mean to "run in the bootstrap"? Do you mean run in the simulated device?
David Farler
Comment 5 2014-08-05 17:52:10 PDT
Sort of, although not in a real SimDevice. It’s just wrapped in `xcrun -sdk iphonesimulator sim`. It has the bare minimum bootstrap to run a simulator binary but doesn’t have access to to other simulator services like the framebuffer, etc. It works in this case because it’s not a full UIKit application.
David Kilzer (:ddkilzer)
Comment 6 2014-08-05 18:29:33 PDT
(In reply to comment #4) > What does it mean to "run in the bootstrap"? Do you mean run in the simulated device? It means that ImageDiff runs in the simulator environment, but not as an app in SpringBoard. In other words, it uses simulator frameworks instead of Mac OS X frameworks to run on the command line.
David Kilzer (:ddkilzer)
Comment 7 2014-08-05 18:35:18 PDT
Comment on attachment 236050 [details] Patch r=me
David Farler
Comment 8 2014-08-05 18:40:15 PDT
Note You need to log in before you can comment on or make changes to this bug.