Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(678)

Side by Side Diff: sky/HACKING.md

Issue 850103003: Unprefix -webkit-filter and add a pixel test for it. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | sky/engine/core/animation/DeferredLegacyStyleInterpolationTest.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 Hacking on Sky 1 Hacking on Sky
2 ============== 2 ==============
3 3
4 Building 4 Building
5 -------- 5 --------
6 6
7 * Follow the setup & build instructions for [Mojo](https://github.com/domokit/mo jo) 7 * Follow the setup & build instructions for [Mojo](https://github.com/domokit/mo jo)
8 8
9 Running applications 9 Running applications
10 -------------------- 10 --------------------
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 47
48 Sky does not have proper pixel tests. Instead we have only reftests. 48 Sky does not have proper pixel tests. Instead we have only reftests.
49 If you want a pixel test, you need to dump the png from a reftest, 49 If you want a pixel test, you need to dump the png from a reftest,
50 upload it to googlestorage and then put and <img> pointing to the 50 upload it to googlestorage and then put and <img> pointing to the
51 uploaded file in the reference. 51 uploaded file in the reference.
52 52
53 1. Create your test (e.g. foo.sky). 53 1. Create your test (e.g. foo.sky).
54 2. Create an dummy reference file (foo-expected.sky). 54 2. Create an dummy reference file (foo-expected.sky).
55 3. Run the test (it will fail). 55 3. Run the test (it will fail).
56 4. Copy the -actual.png file to googlestorage (see below). 56 4. Copy the -actual.png file to googlestorage (see below).
57 5. Put an <img> pointing to your newly uploaded png in the reference file. 57 5. Put an <img> pointing to your newly uploaded png in the reference file at
58 http://storage.googleapis.com/mojo/sky-pngs/SHA1_HASH_HERE
58 59
59 Copying the file to googlestorage: 60 Copying the file to googlestorage:
60 $ sha1sum ../out/Debug/layout-test-results/framework/flights-app-pixels-actual.p ng 61 $ sha1sum ../out/Debug/layout-test-results/framework/flights-app-pixels-actual.p ng
61 db0508cdfe69e996a93464050dc383f6480f1283 ../out/Debug/layout-test-results/frame work/flights-app-pixels-actual.png 62 db0508cdfe69e996a93464050dc383f6480f1283 ../out/Debug/layout-test-results/frame work/flights-app-pixels-actual.png
62 $ gsutil.py cp ../out/Debug/layout-test-results/framework/flights-app-pixels-act ual.png gs://mojo/sky-pngs/db0508cdfe69e996a93464050dc383f6480f1283 63 $ gsutil.py cp ../out/Debug/layout-test-results/framework/flights-app-pixels-act ual.png gs://mojo/sky-pngs/db0508cdfe69e996a93464050dc383f6480f1283
63 64
64 Long-term, we should not have these tests at all and should just 65 Long-term, we should not have these tests at all and should just
65 dump paint commands. In the short-term, if we find we're doing this 66 dump paint commands. In the short-term, if we find we're doing this
66 a lot we should obviously automate this process, e.g. test_sky could 67 a lot we should obviously automate this process, e.g. test_sky could
67 do all of this work, including spitting out the correct reference file. 68 do all of this work, including spitting out the correct reference file.
OLDNEW
« no previous file with comments | « no previous file | sky/engine/core/animation/DeferredLegacyStyleInterpolationTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698