Flesh out GN/Android run docs.

Flesh out GN/Android run docs.

The process is simple enough that it's clearer to see all the steps laid out.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2343263002
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=2343263002

Committed: https://skia.googlesource.com/skia/+/e7904f3979e2e8b80b5d77165291e649ff1278dd

[mtklein_C, 2016-09-16 19:11:33]

Description was changed from

==========
Flesh out GN/Android run docs.

The process is simple enough that it's clearer to see all the steps laid out.

BUG=skia:
==========

to

==========
Flesh out GN/Android run docs.

The process is simple enough that it's clearer to see all the steps laid out.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2343263002
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=2343263002
==========

[mtklein_C, 2016-09-16 19:14:39]

mtklein@chromium.org changed reviewers:
+ csmartdalton@google.com

[csmartdalton, 2016-09-16 19:25:09]

https://codereview.chromium.org/2343263002/diff/40001/site/user/quick/gn.md
File site/user/quick/gn.md (right):

https://codereview.chromium.org/2343263002/diff/40001/site/user/quick/gn.md#n...
site/user/quick/gn.md:91: adb push resources /data/local/tmp
Resources are specific to the dm binary itself as opposed to gn. Can we leave it
out?

 adb push out/arm64/dm /data/local/tmp
 adb shell "cd /data/local/tmp; ./dm --src gm --config gpu"

Seeing those two steps laid out I think it should be obvious that you are
responsible to put files where the program can find them. And if anybody does
forget to push the resources because they are following those steps, dm will
complain and remind them to do so.

[mtklein_C, 2016-09-16 19:30:56]

https://codereview.chromium.org/2343263002/diff/40001/site/user/quick/gn.md
File site/user/quick/gn.md (right):

https://codereview.chromium.org/2343263002/diff/40001/site/user/quick/gn.md#n...
site/user/quick/gn.md:91: adb push resources /data/local/tmp
On 2016/09/16 at 19:25:09, csmartdalton wrote:
> Resources are specific to the dm binary itself as opposed to gn. Can we leave
it out?
> 
>  adb push out/arm64/dm /data/local/tmp
>  adb shell "cd /data/local/tmp; ./dm --src gm --config gpu"
> 
> Seeing those two steps laid out I think it should be obvious that you are
responsible to put files where the program can find them. And if anybody does
forget to push the resources because they are following those steps, dm will
complain and remind them to do so.

I see your point but every app we have to use as an example (dm, nanobench)
needs resources.  I'd rather not write instructions that fail.

[csmartdalton, 2016-09-16 19:34:01]

Fair enough, failing instructions are worse.

lgtm

[csmartdalton, 2016-09-16 19:34:02]

Fair enough, failing instructions are worse.

lgtm

[mtklein_C, 2016-09-16 19:39:18]

The CQ bit was checked by mtklein@chromium.org

[commit-bot: I haz the power, 2016-09-16 19:39:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-16 19:40:22]

Description was changed from

==========
Flesh out GN/Android run docs.

The process is simple enough that it's clearer to see all the steps laid out.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2343263002
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=2343263002
==========

to

==========
Flesh out GN/Android run docs.

The process is simple enough that it's clearer to see all the steps laid out.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2343263002
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=2343263002

Committed:
https://skia.googlesource.com/skia/+/e7904f3979e2e8b80b5d77165291e649ff1278dd
==========

[commit-bot: I haz the power, 2016-09-16 19:40:23]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/e7904f3979e2e8b80b5d77165291e649ff1278dd