Rename snapshot, other user-visible parts of docgen to dartdocgen.

Rename snapshot, other user-visible parts of docgen to dartdocgen.

BUG=
R=ricow@google.com

Committed: https://code.google.com/p/dart/source/detail?r=41529

Committed: https://code.google.com/p/dart/source/detail?r=41571

Committed: https://code.google.com/p/dart/source/detail?r=41573

[Alan Knight, 2014-11-04 00:47:34]

alanknight@google.com changed reviewers:
+ ricow@google.com

[ricow1, 2014-11-04 07:49:37]

how about just calling the other script from the deprecated shell scripts?
That way you only have to maintain one piece of code?

Other than that, lgtm

[Alan Knight, 2014-11-04 21:40:33]

Just paranoid about tweaking scripts, especially for Windows, but done.

[Alan Knight, 2014-11-05 20:23:15]

Committed patchset #3 (id:40001) manually as 41529 (presubmit successful).

[Alan Knight, 2014-11-05 21:12:04]

Reverted in 41531.

This fails in
http://chromegw.corp.google.com/i/client.dart/builders/dart-editor-linux-be/b...


with 

make -j 4 BUILDTYPE=ReleaseIA32 api_docs
make: *** No rule to make target `dartdocgen'.  Stop.

Added a comment for where I think this needs to be fixed, but I don't know how
to verify that without submitting, which I'd rather not do.



I think this is

https://codereview.chromium.org/696123005/diff/40001/editor/build/build.py
File editor/build/build.py (right):

https://codereview.chromium.org/696123005/diff/40001/editor/build/build.py#ne...
editor/build/build.py:1145: CallBuildScript('release', 'ia32', 'dartdocgen')
Should this be left as docgen? Or should the name in docgen.gyp be changed to
dartdocgen?

[sethladd, 2014-11-05 21:31:40]

IMO we should unify our names as much as we can, to make it easy for the next
engineer coming along to help maintain the system.

[ricow1, 2014-11-06 07:12:10]

https://codereview.chromium.org/696123005/diff/40001/editor/build/build.py
File editor/build/build.py (right):

https://codereview.chromium.org/696123005/diff/40001/editor/build/build.py#ne...
editor/build/build.py:1145: CallBuildScript('release', 'ia32', 'dartdocgen')
Yees, this is the gyp name of the target

[Alan Knight, 2014-11-06 17:15:20]

Committed patchset #3 (id:40001) manually as 41571 (presubmit successful).

[Alan Knight, 2014-11-06 17:41:33]

Committed patchset #3 (id:40001) manually as 41573 (presubmit successful).