Add --compile flag.

Add --compile flag.

BUG=16716
.
R=alanknight@google.com

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

[Emily Fortuna, 2014-02-13 00:27:56]

https://codereview.chromium.org/162223002/diff/40001/pkg/docgen/bin/docgen.dart
File pkg/docgen/bin/docgen.dart (right):

https://codereview.chromium.org/162223002/diff/40001/pkg/docgen/bin/docgen.da...
pkg/docgen/bin/docgen.dart:132: '(if not already present) and start a simple
server', defaultsTo: false,
can we add "and compile with dart2js" here too, just to be clear that compile is
one step less than the serve option?

https://codereview.chromium.org/162223002/diff/40001/pkg/docgen/lib/docgen.dart
File pkg/docgen/lib/docgen.dart (right):

https://codereview.chromium.org/162223002/diff/40001/pkg/docgen/lib/docgen.da...
pkg/docgen/lib/docgen.dart:136: _Viewer._clone();
it's a mini function, but let's pull out lines 136-140 into a function so that
we can call it here and at 145.

[dgrove, 2014-02-13 00:58:40]

ptal

https://codereview.chromium.org/162223002/diff/40001/pkg/docgen/bin/docgen.dart
File pkg/docgen/bin/docgen.dart (right):

https://codereview.chromium.org/162223002/diff/40001/pkg/docgen/bin/docgen.da...
pkg/docgen/bin/docgen.dart:132: '(if not already present) and start a simple
server', defaultsTo: false,
On 2014/02/13 00:27:56, Emily Fortuna wrote:
> can we add "and compile with dart2js" here too, just to be clear that compile
is
> one step less than the serve option?

Done.

https://codereview.chromium.org/162223002/diff/40001/pkg/docgen/lib/docgen.dart
File pkg/docgen/lib/docgen.dart (right):

https://codereview.chromium.org/162223002/diff/40001/pkg/docgen/lib/docgen.da...
pkg/docgen/lib/docgen.dart:136: _Viewer._clone();
On 2014/02/13 00:27:56, Emily Fortuna wrote:
> it's a mini function, but let's pull out lines 136-140 into a function so that
> we can call it here and at 145.

Done.

[Alan Knight, 2014-02-13 01:07:00]

lgtm

[dgrove, 2014-02-13 01:07:21]

Committed patchset #5 manually as r32639 (presubmit successful).

[kevmoo, 2014-02-19 21:57:15]

Oops...

https://codereview.chromium.org/162223002/diff/180001/pkg/docgen/lib/docgen.dart
File pkg/docgen/lib/docgen.dart (left):

https://codereview.chromium.org/162223002/diff/180001/pkg/docgen/lib/docgen.d...
pkg/docgen/lib/docgen.dart:696: static void _moveDirectoryAndServe() {
Still being used...

https://code.google.com/p/dart/issues/detail?id=16959