SampleApp documentation

Start SampleApp documentation

BUG=skia:359
Committed: https://skia.googlesource.com/skia/+/e4e01b8608cd7aeaf47c914ddfb2afa3239a0bc6

[herb_g, 2015-01-29 19:03:59]

herb@google.com changed reviewers:
+ hcm@google.com

[jcgregorio, 2015-01-29 19:10:16]

jcgregorio@google.com changed reviewers:
+ jcgregorio@google.com

[jcgregorio, 2015-01-29 19:10:17]

https://codereview.chromium.org/883203003/diff/60001/site/user/sample/samplea...
File site/user/sample/sampleapp.md (right):

https://codereview.chromium.org/883203003/diff/60001/site/user/sample/samplea...
site/user/sample/sampleapp.md:15: You can find how to build the SampleApp for
the various operating systems at the following links:
Make these relative links, also consider putting them in a list:

  * [Linux](/user/quick/linux)
  * [Android](/user/quick/android)

[hcm, 2015-01-29 20:14:18]

Thanks for doing this, one other small thing from me, otherwise looks good

https://codereview.chromium.org/883203003/diff/60001/site/user/sample/samplea...
File site/user/sample/sampleapp.md (right):

https://codereview.chromium.org/883203003/diff/60001/site/user/sample/samplea...
site/user/sample/sampleapp.md:1: About the SampleApp
Nit, but lets title it "About SampleApp"

[herb_g, 2015-01-30 02:24:15]

Address comments. PTAL

https://codereview.chromium.org/883203003/diff/60001/site/user/sample/samplea...
File site/user/sample/sampleapp.md (right):

https://codereview.chromium.org/883203003/diff/60001/site/user/sample/samplea...
site/user/sample/sampleapp.md:1: About the SampleApp
On 2015/01/29 20:14:18, hcm wrote:
> Nit, but lets title it "About SampleApp"

Done.

https://codereview.chromium.org/883203003/diff/60001/site/user/sample/samplea...
site/user/sample/sampleapp.md:15: You can find how to build the SampleApp for
the various operating systems at the following links:
On 2015/01/29 19:10:17, jcgregorio wrote:
> Make these relative links, also consider putting them in a list:
> 
>   * [Linux](/user/quick/linux)
>   * [Android](/user/quick/android)

Done.

[herb_g, 2015-01-30 02:25:04]

The CQ bit was checked by herb@google.com

[commit-bot: I haz the power, 2015-01-30 02:25:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/883203003/80001

[commit-bot: I haz the power, 2015-01-30 02:25:21]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2015-01-30 08:25 UTC

[hcm, 2015-01-30 04:29:19]

On 2015/01/30 02:25:21, I haz the power (commit-bot) wrote:
> Note for Reviewers:
> The CQ is waiting for an approval.
> If you believe that the CL is not ready yet, or if you would like to L-G-T-M
> with comments then please uncheck the CQ checkbox.
> 
> Waiting for LGTM from valid reviewer(s) till 2015-01-30 08:25 UTC

lgtm

[commit-bot: I haz the power, 2015-01-30 04:31:08]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/e4e01b8608cd7aeaf47c914ddfb2afa3239a0bc6

[tfarina, 2015-01-30 14:33:43]

https://codereview.chromium.org/883203003/diff/80001/site/user/sample/samplea...
File site/user/sample/sampleapp.md (right):

https://codereview.chromium.org/883203003/diff/80001/site/user/sample/samplea...
site/user/sample/sampleapp.md:3: The SampleApp displays a series of slides that
exhibit specific features of the Skia system. It is mainly controlled using the
keyboard: left (←) and (→) right arrows to move from slide to
slide. Some slides require use resources stored outside the program. These
resources are stored in the `<skia-path>/resources` directory.
nit: require use?

https://codereview.chromium.org/883203003/diff/80001/site/user/sample/samplea...
site/user/sample/sampleapp.md:13: * Display and manipulate your own pictures -
use the `--picture <skp-file-path>.skp` to use the `.skp` file as the slide
desplayed by SampleApp. A `.skp` file can be generated by...
generated by...? Explain how to generate it or just point to another doc that
explains it? Otherwise just remove this sentence?