Added option --snapshot-after-run to enable an application snapshot to be taken after running the a…

Added option --snapshot-after-run to enable an application snapshot to be taken after running the application.

R=srdjan@google.com

Committed: https://github.com/dart-lang/sdk/commit/040637a0e0d514fb5d071c51b5d21892e1277bb3

[siva, 2015-10-16 21:12:14]

asiva@google.com changed reviewers:
+ srdjan@google.com

[srdjan, 2015-10-16 21:26:56]

lgtm

https://chromiumcodereview.appspot.com/1413753002/diff/1/runtime/bin/main.cc
File runtime/bin/main.cc (right):

https://chromiumcodereview.appspot.com/1413753002/diff/1/runtime/bin/main.cc#...
runtime/bin/main.cc:415: if (filename != NULL && strlen(filename) != 0) {
Add parens please

[siva, 2015-10-16 23:28:55]

Refactored code a bit to avoid code duplication while processing the two script
snapshot options.

PTAL.

[srdjan, 2015-10-16 23:37:24]

lgtm

https://chromiumcodereview.appspot.com/1413753002/diff/20001/runtime/bin/main.cc
File runtime/bin/main.cc (right):

https://chromiumcodereview.appspot.com/1413753002/diff/20001/runtime/bin/main...
runtime/bin/main.cc:386: bool *snapshot_option) {
star close to type not to name.

https://chromiumcodereview.appspot.com/1413753002/diff/20001/runtime/bin/main...
runtime/bin/main.cc:395: *snapshot_option = true;
Maybe set *snapshot_option = false as first instruction, so that it is always
set by this function.

[siva, 2015-10-16 23:49:52]

https://chromiumcodereview.appspot.com/1413753002/diff/20001/runtime/bin/main.cc
File runtime/bin/main.cc (right):

https://chromiumcodereview.appspot.com/1413753002/diff/20001/runtime/bin/main...
runtime/bin/main.cc:386: bool *snapshot_option) {
On 2015/10/16 23:37:24, srdjan wrote:
> star close to type not to name.

Done.

https://chromiumcodereview.appspot.com/1413753002/diff/20001/runtime/bin/main...
runtime/bin/main.cc:395: *snapshot_option = true;
On 2015/10/16 23:37:24, srdjan wrote:
> Maybe set *snapshot_option = false as first instruction, so that it is always
> set by this function.

Done.

[siva, 2015-10-17 00:10:35]

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