Documentation - Remove references to `gclient sync`

Documentation - Remove references to `gclient sync`

skia.org/user/download
-   Move sections "Making changes" and "Uploading changes for review"
    to "How to submit a patch" page
-   Mention bin/sync-and-gyp

skia.org/dev/contrib/submit
-   Mention branch.autosetuprebase=always to fit with Skia's linear
    history style.

NOTRY=true
DOCS_PREVIEW= https://skia.org/user/download?cl=1411403010
DOCS_PREVIEW= https://skia.org/dev/contrib/submit?cl=1411403010

Committed: https://skia.googlesource.com/skia/+/3a210bfd403815bebbc7efabe7bbd373e5a3d8f8

[hal.canary, 2015-11-06 23:17:10]

Description was changed from

==========
Documentation - Remove referecnes to `gclient sync`
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=1411403010
==========

to

==========
Documentation - Remove referecnes to `gclient sync`
NOTRY=true
DOCS_PREVIEW= https://skia.org/user/download?cl=1411403010
==========

[hal.canary, 2015-11-06 23:18:38]

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

[hal.canary, 2015-11-06 23:18:39]

ptal

[hal.canary, 2015-11-09 16:29:26]

Description was changed from

==========
Documentation - Remove referecnes to `gclient sync`
NOTRY=true
DOCS_PREVIEW= https://skia.org/user/download?cl=1411403010
==========

to

==========
Documentation - Remove referecnes to `gclient sync`
NOTRY=true
DOCS_PREVIEW= https://skia.org/user/download?cl=1411403010
DOCS_PREVIEW= https://skia.org/dev/contrib/submit?cl=1411403010
==========

[hal.canary, 2015-11-09 16:42:15]

Description was changed from

==========
Documentation - Remove referecnes to `gclient sync`
NOTRY=true
DOCS_PREVIEW= https://skia.org/user/download?cl=1411403010
DOCS_PREVIEW= https://skia.org/dev/contrib/submit?cl=1411403010
==========

to

==========
Documentation - Remove referecnes to `gclient sync`

skia.org/user/download
-   Move sections "Making changes" and "Uploading changes for review"
    to "How to submit a patch" page
-   Mention bin/sync-and-gyp

skia.org/dev/contrib/submit
-   Mention branch.autosetuprebase=always to fit with Skia's linear
    history style.

NOTRY=true
DOCS_PREVIEW= https://skia.org/user/download?cl=1411403010
DOCS_PREVIEW= https://skia.org/dev/contrib/submit?cl=1411403010
==========

[hal.canary, 2015-11-09 16:42:36]

Description was changed from

==========
Documentation - Remove referecnes to `gclient sync`

skia.org/user/download
-   Move sections "Making changes" and "Uploading changes for review"
    to "How to submit a patch" page
-   Mention bin/sync-and-gyp

skia.org/dev/contrib/submit
-   Mention branch.autosetuprebase=always to fit with Skia's linear
    history style.

NOTRY=true
DOCS_PREVIEW= https://skia.org/user/download?cl=1411403010
DOCS_PREVIEW= https://skia.org/dev/contrib/submit?cl=1411403010
==========

to

==========
Documentation - Remove references to `gclient sync`

skia.org/user/download
-   Move sections "Making changes" and "Uploading changes for review"
    to "How to submit a patch" page
-   Mention bin/sync-and-gyp

skia.org/dev/contrib/submit
-   Mention branch.autosetuprebase=always to fit with Skia's linear
    history style.

NOTRY=true
DOCS_PREVIEW= https://skia.org/user/download?cl=1411403010
DOCS_PREVIEW= https://skia.org/dev/contrib/submit?cl=1411403010
==========

[hcm, 2015-11-09 18:23:56]

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

https://codereview.chromium.org/1411403010/diff/80001/site/dev/contrib/submit...
site/dev/contrib/submit.md:18: $ git config branch.autosetuprebase always
As discussed, either note this as optional and/or remove --rebase in pull
command

https://codereview.chromium.org/1411403010/diff/80001/site/dev/contrib/submit...
site/dev/contrib/submit.md:153: Once your change has received an LGTM ("looks
good to me"), you can check the
okay to not spell out "looks good to me" again since acronym was defined above

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md
File site/user/download.md (right):

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:4: Install Depot Tools and Git
Depot Tools --> depot_tools (even though it doesn't look as nice, this is always
how it is spelled/capitalized in the Chromium docs as well)

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:21: Configure git
git --> Git to be consistent with above

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:29: Clone the Skia Repository
To be consistent with subheadings in related docs, perhaps just capitalize the
first word + proper nouns like Skia.  Repository --> repository

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:37: Get Skia's dependencies and generate Ninja Build files
lowercase b

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:49: Compile all Default Targets
lowercase d, t

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:56: Execute Skia Tests
lowercase t

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:59: [More about Skia correctnest testing
tools](../dev/testing/testing)
correctness*

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:65: Execute Skia Sample Application
lowercase s, a

[hcm, 2015-11-09 18:25:51]

On 2015/11/09 18:23:56, hcm wrote:
>
https://codereview.chromium.org/1411403010/diff/80001/site/dev/contrib/submit.md
> File site/dev/contrib/submit.md (right):
> 
>
https://codereview.chromium.org/1411403010/diff/80001/site/dev/contrib/submit...
> site/dev/contrib/submit.md:18: $ git config branch.autosetuprebase always
> As discussed, either note this as optional and/or remove --rebase in pull
> command
> 
>
https://codereview.chromium.org/1411403010/diff/80001/site/dev/contrib/submit...
> site/dev/contrib/submit.md:153: Once your change has received an LGTM ("looks
> good to me"), you can check the
> okay to not spell out "looks good to me" again since acronym was defined above
> 
> https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md
> File site/user/download.md (right):
> 
>
https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
> site/user/download.md:4: Install Depot Tools and Git
> Depot Tools --> depot_tools (even though it doesn't look as nice, this is
always
> how it is spelled/capitalized in the Chromium docs as well)
> 
>
https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
> site/user/download.md:21: Configure git
> git --> Git to be consistent with above
> 
>
https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
> site/user/download.md:29: Clone the Skia Repository
> To be consistent with subheadings in related docs, perhaps just capitalize the
> first word + proper nouns like Skia.  Repository --> repository
> 
>
https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
> site/user/download.md:37: Get Skia's dependencies and generate Ninja Build
files
> lowercase b
> 
>
https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
> site/user/download.md:49: Compile all Default Targets
> lowercase d, t
> 
>
https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
> site/user/download.md:56: Execute Skia Tests
> lowercase t
> 
>
https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
> site/user/download.md:59: [More about Skia correctnest testing
> tools](../dev/testing/testing)
> correctness*
> 
>
https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
> site/user/download.md:65: Execute Skia Sample Application
> lowercase s, a

nice, my comment about CL approval approved this one- but lgtm after these minor
adjustments!

[hal.canary, 2015-11-09 19:45:51]

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

https://codereview.chromium.org/1411403010/diff/80001/site/dev/contrib/submit...
site/dev/contrib/submit.md:18: $ git config branch.autosetuprebase always
On 2015/11/09 at 18:23:55, hcm wrote:
> As discussed, either note this as optional and/or remove --rebase in pull
command

done

https://codereview.chromium.org/1411403010/diff/80001/site/dev/contrib/submit...
site/dev/contrib/submit.md:153: Once your change has received an LGTM ("looks
good to me"), you can check the
On 2015/11/09 at 18:23:55, hcm wrote:
> okay to not spell out "looks good to me" again since acronym was defined above

done

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md
File site/user/download.md (right):

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:4: Install Depot Tools and Git
On 2015/11/09 at 18:23:56, hcm wrote:
> Depot Tools --> depot_tools (even though it doesn't look as nice, this is
always how it is spelled/capitalized in the Chromium docs as well)

done

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:21: Configure git
On 2015/11/09 at 18:23:55, hcm wrote:
> git --> Git to be consistent with above

done

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:29: Clone the Skia Repository
On 2015/11/09 at 18:23:56, hcm wrote:
> To be consistent with subheadings in related docs, perhaps just capitalize the
first word + proper nouns like Skia.  Repository --> repository

done

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:37: Get Skia's dependencies and generate Ninja Build files
On 2015/11/09 at 18:23:56, hcm wrote:
> lowercase b

done

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:49: Compile all Default Targets
On 2015/11/09 at 18:23:55, hcm wrote:
> lowercase d, t

done

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:56: Execute Skia Tests
On 2015/11/09 at 18:23:55, hcm wrote:
> lowercase t

done

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:59: [More about Skia correctnest testing
tools](../dev/testing/testing)
On 2015/11/09 at 18:23:55, hcm wrote:
> correctness*

done

https://codereview.chromium.org/1411403010/diff/80001/site/user/download.md#n...
site/user/download.md:65: Execute Skia Sample Application
On 2015/11/09 at 18:23:56, hcm wrote:
> lowercase s, a

done

[hal.canary, 2015-11-09 19:46:22]

The CQ bit was checked by halcanary@google.com

[hal.canary, 2015-11-09 19:46:23]

The patchset sent to the CQ was uploaded after l-g-t-m from hcm@google.com
Link to the patchset: https://codereview.chromium.org/1411403010/#ps100001
(title: "2015-11-09 (Monday) 14:43:06 EST")

[commit-bot: I haz the power, 2015-11-09 19:46:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1411403010/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1411403010/100001

[commit-bot: I haz the power, 2015-11-09 19:47:27]

Committed patchset #6 (id:100001) as
https://skia.googlesource.com/skia/+/3a210bfd403815bebbc7efabe7bbd373e5a3d8f8