Add a sky media service to make it possible to play a sound.

Add a sky media service to make it possible to play a sound.

I didn't exactly follow the android API, but pretty close.

Also started a java/common implementation for things like
DataPipeUtils. I only needed the ability to copy from a data
pipe to a file, so that's the only function we have so far.

Fixes https://github.com/domokit/mojo/issues/270

R=abarth@chromium.org, abarth@google.com

Committed: https://chromium.googlesource.com/external/mojo/+/1635aee07f7b54e42a47e67917c945bc6a055fb7

[eseidel, 2015-07-01 02:02:18]

I forgot to add the new mojo/java directory I added, oops.

[abarth-chromium, 2015-07-01 15:57:21]

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

[abarth-chromium, 2015-07-01 15:57:22]

https://codereview.chromium.org/1211913005/diff/20001/sky/sdk/example/widgets...
File sky/sdk/example/widgets/piano.dart (right):

https://codereview.chromium.org/1211913005/diff/20001/sky/sdk/example/widgets...
sky/sdk/example/widgets/piano.dart:39: final List<Map<int, Color> > pallets = [
s/> >/>>/

This ain't C++!

https://codereview.chromium.org/1211913005/diff/20001/sky/sdk/example/widgets...
sky/sdk/example/widgets/piano.dart:59: shell.requestService("mojo:yo_mama",
mediaService);
yo_mama!

[abarth-chromium, 2015-07-01 17:52:14]

lgtm

https://codereview.chromium.org/1211913005/diff/80001/mojo/java/org/chromium/...
File mojo/java/org/chromium/mojo/common/DataPipeUtils.java (right):

https://codereview.chromium.org/1211913005/diff/80001/mojo/java/org/chromium/...
mojo/java/org/chromium/mojo/common/DataPipeUtils.java:7: // TODO(eseidel): Can
we not depend on Android?
I think this is ok for this library.  We don't want mojo/public/java to depend
on Android, however.

https://codereview.chromium.org/1211913005/diff/80001/mojo/java/org/chromium/...
mojo/java/org/chromium/mojo/common/DataPipeUtils.java:29: class CopyToFileJob
implements Runnable {
Move outside the DataPipeUtils (but keep in this file)

https://codereview.chromium.org/1211913005/diff/80001/mojo/java/org/chromium/...
mojo/java/org/chromium/mojo/common/DataPipeUtils.java:90: public void
copyToFile(Core core, DataPipe.ConsumerHandle source, File dest,
public void copyToFile -> public static void copyToFile

https://codereview.chromium.org/1211913005/diff/80001/sky/sdk/example/widgets...
File sky/sdk/example/widgets/piano.dart (right):

https://codereview.chromium.org/1211913005/diff/80001/sky/sdk/example/widgets...
sky/sdk/example/widgets/piano.dart:12: import 'package:sky/mojo/shell.dart' as
shell;
Sort pls

https://codereview.chromium.org/1211913005/diff/80001/sky/sdk/example/widgets...
sky/sdk/example/widgets/piano.dart:19: Color color;
final?

https://codereview.chromium.org/1211913005/diff/80001/sky/sdk/example/widgets...
sky/sdk/example/widgets/piano.dart:46: List<Key> keys = [];
final

https://codereview.chromium.org/1211913005/diff/80001/sky/sdk/example/widgets...
sky/sdk/example/widgets/piano.dart:57: shell.requestService("mojo:yo_mama",
mediaService);
mojo:yo_mama -> mojo:sky_viewer

https://codereview.chromium.org/1211913005/diff/80001/sky/services/media/medi...
File sky/services/media/media.mojom (right):

https://codereview.chromium.org/1211913005/diff/80001/sky/services/media/medi...
sky/services/media/media.mojom:16: createPlayer(MediaPlayer& player);
prepare -> Prepare
start -> Start
pause -> Pause
seekTo -> SeekTo
createPlayer -> CreatePlayer

mojom is StudlyCaps.  The code generator will change the style for each target
language.

https://codereview.chromium.org/1211913005/diff/80001/sky/services/media/src/...
File sky/services/media/src/org/domokit/media/MediaPlayerImpl.java (right):

https://codereview.chromium.org/1211913005/diff/80001/sky/services/media/src/...
sky/services/media/src/org/domokit/media/MediaPlayerImpl.java:81: mTempFile =
File.createTempFile("hack", "hack", outputDir);
"hack", "hack"  ->  Can we use better names?

https://codereview.chromium.org/1211913005/diff/80001/sky/services/media/src/...
sky/services/media/src/org/domokit/media/MediaPlayerImpl.java:90:
utils.copyToFile(mCore, consumerHandle, mTempFile, mExecutor, new Runnable() {
We should be able to make it a static method.

[eseidel, 2015-07-01 19:29:45]

Committed patchset #6 (id:100001) manually as
1635aee07f7b54e42a47e67917c945bc6a055fb7 (presubmit successful).