Add Contributing to Skia section of docs

Add Contributing to Skia section of docs

BUG=skia:

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

[hcm, 2015-01-20 16:48:03]

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

[hcm, 2015-01-20 16:49:56]

On 2015/01/20 16:48:03, hcm wrote:
> mailto:hcm@google.com changed reviewers:
> + mailto:jcgregorio@google.com

Side note: Pages are done but the nav order is getting weird...we should think
about a way to fix that so that we enter into User doc first, enter into Dev -->
code reviews/patches before reverts and special cases, etc.

[jcgregorio, 2015-01-20 17:59:00]

https://codereview.chromium.org/844433004/diff/100001/site/dev/contrib/cqkeyw...
File site/dev/contrib/cqkeywords.md (right):

https://codereview.chromium.org/844433004/diff/100001/site/dev/contrib/cqkeyw...
site/dev/contrib/cqkeywords.md:7: If you want to test your CL through the commit
queue but are not ready to commit the changes yet, you can add the following
line to the CL description:
Wrap all paragraphs to 80 or 100 cols.

https://codereview.chromium.org/844433004/diff/100001/site/dev/contrib/patch.md
File site/dev/contrib/patch.md (right):

https://codereview.chromium.org/844433004/diff/100001/site/dev/contrib/patch....
site/dev/contrib/patch.md:1: How to apply a patch from a codereview issue
Shorted title to "Applying patches"

[hcm, 2015-01-21 03:38:12]

On 2015/01/20 17:59:00, jcgregorio wrote:
>
https://codereview.chromium.org/844433004/diff/100001/site/dev/contrib/cqkeyw...
> File site/dev/contrib/cqkeywords.md (right):
> 
>
https://codereview.chromium.org/844433004/diff/100001/site/dev/contrib/cqkeyw...
> site/dev/contrib/cqkeywords.md:7: If you want to test your CL through the
commit
> queue but are not ready to commit the changes yet, you can add the following
> line to the CL description:
> Wrap all paragraphs to 80 or 100 cols.
> 
>
https://codereview.chromium.org/844433004/diff/100001/site/dev/contrib/patch.md
> File site/dev/contrib/patch.md (right):
> 
>
https://codereview.chromium.org/844433004/diff/100001/site/dev/contrib/patch....
> site/dev/contrib/patch.md:1: How to apply a patch from a codereview issue
> Shorted title to "Applying patches"

Updated, PTAL

[jcgregorio, 2015-01-21 13:15:23]

https://codereview.chromium.org/844433004/diff/120001/site/dev/contrib/flatte...
File site/dev/contrib/flatten.md (right):

https://codereview.chromium.org/844433004/diff/120001/site/dev/contrib/flatte...
site/dev/contrib/flatten.md:4: Many objects in Skia, such as SkShaders and other
effects on SkPaint, need to be flattened into a data stream for either transport
Sorry, should have been more specific, can you make sure paragraphs in all pages
are wrapped to either 80 or 100 cols.

[hcm, 2015-01-21 16:49:48]

On 2015/01/21 13:15:23, jcgregorio wrote:
>
https://codereview.chromium.org/844433004/diff/120001/site/dev/contrib/flatte...
> File site/dev/contrib/flatten.md (right):
> 
>
https://codereview.chromium.org/844433004/diff/120001/site/dev/contrib/flatte...
> site/dev/contrib/flatten.md:4: Many objects in Skia, such as SkShaders and
other
> effects on SkPaint, need to be flattened into a data stream for either
transport
> Sorry, should have been more specific, can you make sure paragraphs in all
pages
> are wrapped to either 80 or 100 cols.

Think I'm ready for another look, if nothing else because I can barely read
these files anymore and see the problems :)

[jcgregorio, 2015-01-21 17:55:39]

https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/flatte...
File site/dev/contrib/flatten.md (right):

https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/flatte...
site/dev/contrib/flatten.md:12: ~~~~
Add prettify before all the code sections in all the files, ala:

<!--?prettify?-->
~~~~

https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/style.md
File site/dev/contrib/style.md (right):

https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/style....
site/dev/contrib/style.md:46: ~~~~
<!--?prettify?-->

here and below

https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/submit.md
File site/dev/contrib/submit.md (right):

https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/submit...
site/dev/contrib/submit.md:38: ~~~~
<!--?prettify?-->

https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/submit...
site/dev/contrib/submit.md:61: ~~~~
<!--?prettify?-->

[hcm, 2015-01-21 19:23:53]

On 2015/01/21 17:55:39, jcgregorio wrote:
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/flatte...
> File site/dev/contrib/flatten.md (right):
> 
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/flatte...
> site/dev/contrib/flatten.md:12: ~~~~
> Add prettify before all the code sections in all the files, ala:
> 
> <!--?prettify?-->
> ~~~~
> 
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/style.md
> File site/dev/contrib/style.md (right):
> 
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/style....
> site/dev/contrib/style.md:46: ~~~~
> <!--?prettify?-->
> 
> here and below
> 
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/submit.md
> File site/dev/contrib/submit.md (right):
> 
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/submit...
> site/dev/contrib/submit.md:38: ~~~~
> <!--?prettify?-->
> 
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/submit...
> site/dev/contrib/submit.md:61: ~~~~
> <!--?prettify?-->

Added the tags for all code snippets on style guide, submit, and flatten..
reviewed and other pages do not have code snippets

[jcgregorio, 2015-01-21 19:32:11]

On 2015/01/21 19:23:53, hcm wrote:
> On 2015/01/21 17:55:39, jcgregorio wrote:
> >
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/flatte...
> > File site/dev/contrib/flatten.md (right):
> > 
> >
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/flatte...
> > site/dev/contrib/flatten.md:12: ~~~~
> > Add prettify before all the code sections in all the files, ala:
> > 
> > <!--?prettify?-->
> > ~~~~
> > 
> >
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/style.md
> > File site/dev/contrib/style.md (right):
> > 
> >
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/style....
> > site/dev/contrib/style.md:46: ~~~~
> > <!--?prettify?-->
> > 
> > here and below
> > 
> >
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/submit.md
> > File site/dev/contrib/submit.md (right):
> > 
> >
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/submit...
> > site/dev/contrib/submit.md:38: ~~~~
> > <!--?prettify?-->
> > 
> >
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/submit...
> > site/dev/contrib/submit.md:61: ~~~~
> > <!--?prettify?-->
> 
> Added the tags for all code snippets on style guide, submit, and flatten..
> reviewed and other pages do not have code snippets

The third to the last code snippet doesn't look right on this page:

   http://skia.org/dev/contrib/flatten?cl=844433004

But that's the only issue I see, so

LGTM

after that's fixed.

[hcm, 2015-01-21 19:54:37]

On 2015/01/21 19:32:11, jcgregorio wrote:
> On 2015/01/21 19:23:53, hcm wrote:
> > On 2015/01/21 17:55:39, jcgregorio wrote:
> > >
> >
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/flatte...
> > > File site/dev/contrib/flatten.md (right):
> > > 
> > >
> >
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/flatte...
> > > site/dev/contrib/flatten.md:12: ~~~~
> > > Add prettify before all the code sections in all the files, ala:
> > > 
> > > <!--?prettify?-->
> > > ~~~~
> > > 
> > >
> >
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/style.md
> > > File site/dev/contrib/style.md (right):
> > > 
> > >
> >
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/style....
> > > site/dev/contrib/style.md:46: ~~~~
> > > <!--?prettify?-->
> > > 
> > > here and below
> > > 
> > >
> >
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/submit.md
> > > File site/dev/contrib/submit.md (right):
> > > 
> > >
> >
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/submit...
> > > site/dev/contrib/submit.md:38: ~~~~
> > > <!--?prettify?-->
> > > 
> > >
> >
>
https://codereview.chromium.org/844433004/diff/160001/site/dev/contrib/submit...
> > > site/dev/contrib/submit.md:61: ~~~~
> > > <!--?prettify?-->
> > 
> > Added the tags for all code snippets on style guide, submit, and flatten..
> > reviewed and other pages do not have code snippets
> 
> The third to the last code snippet doesn't look right on this page:
> 
>    http://skia.org/dev/contrib/flatten?cl=844433004
> 
> But that's the only issue I see, so
> 
> LGTM
> 
> after that's fixed.

Thanks for the catch, fixed.

[hcm, 2015-01-21 19:54:53]

The CQ bit was checked by hcm@google.com

[jcgregorio, 2015-01-21 19:54:55]

The CQ bit was checked by jcgregorio@google.com

[jcgregorio, 2015-01-21 19:54:55]

lgtm

[commit-bot: I haz the power, 2015-01-21 19:55:53]

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

[commit-bot: I haz the power, 2015-01-21 20:05:51]

Committed patchset #12 (id:210001) as
https://skia.googlesource.com/skia/+/fd1ad48d4d9073b90f58e60219637046a8446267