Add minedigger example

Add minedigger example

A very basic minesweeper clone. Currently only tested in linux.

BUG=none
R=abarth@chromium.org, eseidel@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/475a02a2b46ffb03820f1569794021dd6acbc20a

[cpu_(ooo_6.6-7.5), 2015-06-23 01:00:22]

cpu@chromium.org changed reviewers:
+ abarth@chromium.org

[eseidel, 2015-06-23 01:00:59]

eseidel@chromium.org changed reviewers:
+ eseidel@chromium.org, ianh@google.com
- abarth@chromium.org

[eseidel, 2015-06-23 01:00:59]

Thanks!

Attempted to run it locally and hit an exception:

E/chromium(27811): Uncaught Error: Class 'Text' has no instance getter
'DefaultTextStyle'.

I can look in more detail tomorrow.

[cpu_(ooo_6.6-7.5), 2015-06-23 01:01:53]

cpu@chromium.org changed reviewers:
+ abarth@chromium.org
- eseidel@chromium.org, ianh@google.com

[cpu_(ooo_6.6-7.5), 2015-06-23 01:01:53]

Still not complete as per expectations of a seasoned minesweeper player. The
timer is not there nor the smiley face or cool looking counters.

[abarth-chromium, 2015-06-23 04:05:17]

LGTM

This is great.  Thanks so much!

https://codereview.chromium.org/1196213012/diff/1/sky/examples/mine_digger/mi...
File sky/examples/mine_digger/mine_digger.dart (right):

https://codereview.chromium.org/1196213012/diff/1/sky/examples/mine_digger/mi...
sky/examples/mine_digger/mine_digger.dart:7: import 'dart:core';
You don't need to import dart:core.  It's imported automatically for you.

https://codereview.chromium.org/1196213012/diff/1/sky/examples/mine_digger/mi...
sky/examples/mine_digger/mine_digger.dart:98: var flex_rows = new List<Flex>();
s/flex_rows/flexRows/

https://codereview.chromium.org/1196213012/diff/1/sky/examples/mine_digger/mi...
sky/examples/mine_digger/mine_digger.dart:123: }else {
s/}else {/} else {/

https://codereview.chromium.org/1196213012/diff/1/sky/examples/mine_digger/mi...
sky/examples/mine_digger/mine_digger.dart:142: print('game won!');
Mobile first!

https://codereview.chromium.org/1196213012/diff/1/sky/examples/mine_digger/mi...
sky/examples/mine_digger/mine_digger.dart:269: child : widget);
s/margin :/margin:/
s/child :/child:/

https://codereview.chromium.org/1196213012/diff/1/sky/examples/mine_digger/mi...
sky/examples/mine_digger/mine_digger.dart:323: child : text);
s/child :/child:/

[eseidel, 2015-06-23 16:25:45]

eseidel@chromium.org changed reviewers:
+ eseidel@chromium.org, ianh@google.com
- abarth@chromium.org

[eseidel, 2015-06-23 16:25:45]

Worked when I ran it just now. I think we should land this and iterate from
there.

[cpu_(ooo_6.6-7.5), 2015-06-23 17:36:54]

all changes addressed. thanks!

[eseidel, 2015-06-23 17:50:11]

eseidel@chromium.org changed reviewers:
+ abarth@chromium.org

[eseidel, 2015-06-23 17:50:12]

lgtm

Land away!

https://codereview.chromium.org/1196213012/diff/20001/sky/examples/mine_digge...
File sky/examples/mine_digger/mine_digger.dart (right):

https://codereview.chromium.org/1196213012/diff/20001/sky/examples/mine_digge...
sky/examples/mine_digger/mine_digger.dart:21: static final int rows = 9;
You should jsut check this in, but in case you need:

These could be const.  const means "evaluate this at compile time and share
instances between all const mentions of this value".  final means "evaluate on
first access, it will never change".

https://codereview.chromium.org/1196213012/diff/20001/sky/examples/mine_digge...
sky/examples/mine_digger/mine_digger.dart:152: key: 'flxv'));
I'm not sure keys are needed anymore.

https://codereview.chromium.org/1196213012/diff/20001/sky/examples/mine_digge...
sky/examples/mine_digger/mine_digger.dart:156: var board = generateBoard();
Types are your friend. :)

https://codereview.chromium.org/1196213012/diff/20001/sky/examples/mine_digge...
sky/examples/mine_digger/mine_digger.dart:165: decoration: new
BoxDecoration(backgroundColor: const Color(0xFFC0C0C0)),
theme/colors.dart has lots of helpful material design color constants if you
like.

[cpu_(ooo_6.6-7.5), 2015-06-23 18:19:10]

I converted all vars to actual types and use consts instead of finals.

https://codereview.chromium.org/1196213012/diff/20001/sky/examples/mine_digge...
File sky/examples/mine_digger/mine_digger.dart (right):

https://codereview.chromium.org/1196213012/diff/20001/sky/examples/mine_digge...
sky/examples/mine_digger/mine_digger.dart:152: key: 'flxv'));
On 2015/06/23 17:50:12, eseidel wrote:
> I'm not sure keys are needed anymore.

Acknowledged.

https://codereview.chromium.org/1196213012/diff/20001/sky/examples/mine_digge...
sky/examples/mine_digger/mine_digger.dart:165: decoration: new
BoxDecoration(backgroundColor: const Color(0xFFC0C0C0)),
On 2015/06/23 17:50:12, eseidel wrote:
> theme/colors.dart has lots of helpful material design color constants if you
> like.

Acknowledged.

[cpu_(ooo_6.6-7.5), 2015-06-23 18:20:09]

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