Expose sky KeyboardService in android mojo_shell

sky/framework/shell.dart

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 import "dart:sky.internals" as internals;
+import "embedder.dart";
 import "package:mojo/public/dart/application.dart";
 import "package:mojo/public/interfaces/application/service_provider.mojom.dart";
-import 'package:mojo/public/dart/core.dart' as core;
+import "package:mojo/public/dart/core.dart" as core;
 
 ApplicationConnection _initConnection() {
   int rawHandle = internals.takeServicesProvidedByEmbedder();
   core.MojoHandle proxyHandle = new core.MojoHandle(rawHandle);
   ServiceProviderProxy serviceProvider = null;
-  if (proxyHandle.isValid)
-    serviceProvider = new ServiceProviderProxy.fromHandle(proxyHandle);
+  if (proxyHandle.isValid) serviceProvider =

[eseidel, 2015/04/15 23:09:04]

Really?  This is dart style?

+      new ServiceProviderProxy.fromHandle(proxyHandle);
   return new ApplicationConnection(null, serviceProvider);
 }
 
 final ApplicationConnection _connection = _initConnection();
 
-void requestService(Object proxy) {
-  _connection.requestService(proxy);
+void requestService(String url, Object proxy) {
+  if (embedder.shell == null) _connection.requestService(proxy);

[eseidel, 2015/04/15 23:09:04]

Why are one-line ifs a good thing? :p

[hansmuller1, 2015/04/15 23:13:34]

According to the Dart Style Guide, if statements should use curly braces and
span multiple lines except:

"short if statements with no else clause that fit on one line may omit the
braces.

if (arg == null) return defaultValue;"

+  else embedder.connectToService(url, proxy);
 }