Initial native keyboard proof of concept.

examples/keyboard_client/keyboard_client.cc

+// 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.
+
+#include "base/logging.h"
+#include "mojo/public/c/system/main.h"
+#include "mojo/public/cpp/application/application_delegate.h"
+#include "mojo/public/cpp/application/application_impl.h"
+#include "mojo/public/cpp/application/application_runner.h"
+#include "mojo/public/cpp/utility/run_loop.h"
+#include "mojo/services/keyboard/public/interfaces/keyboard.mojom.h"
+
+namespace examples {
+
+class KeyboardDelegate : public mojo::ApplicationDelegate,
+                         public keyboard::KeyboardClient {
+ public:
+  KeyboardDelegate() : binding_(this) {}

[jamesr, 2015/05/08 23:45:23]

whenever you have a type with a vtable (i.e. inherits from a virtual class) it's
good practice to define a d'tor explicitly so it's clear that destruction is
virtual. for this it'd be something like

~KeyboardDelegate() override {}

since this class is tiny it isn't all that helpful but classes that start out
tiny have a habit of not staying small

[APW, 2015/05/09 00:45:58]

Done.

+
+  // mojo::ApplicationDelegate implementation.
+  void Initialize(mojo::ApplicationImpl* app) override {
+    app->ConnectToService("mojo:keyboard_native", &keyboard_);
+    keyboard_->ShowByRequest();
+    keyboard_->Hide();
+
+    mojo::InterfaceRequest<keyboard::KeyboardClient> request =
+        mojo::GetProxy(&keyboardClient_);

[jamesr, 2015/05/08 23:45:23]

i think you still want keyboardClient_ to be a local, although you should call
it keyboard_client or just client, i.e. something like

keyboard::KeyboardClientPtr client;
binding_.Bind(GetProxy(&client));
keyboard_->Show(client.Pass());

[APW, 2015/05/09 00:45:58]

Done.

+    binding_.Bind(request.Pass());
+    keyboard_->Show(keyboardClient_.Pass());
+  }
+
+  // keyboard::KeyboardClient implementation.
+  void CommitCompletion(keyboard::CompletionDataPtr completion) override {}
+
+  void CommitCorrection(keyboard::CorrectionDataPtr correction) override {}
+
+  void CommitText(const mojo::String& text,
+                  int32_t newCursorPosition) override {}

[jamesr, 2015/05/08 23:45:23]

by convention we use hacker_style_for_names in c++.
https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Naming

[APW, 2015/05/09 00:45:58]

Done.

+
+  void DeleteSurroundingText(int32_t beforeLength,
+                             int32_t afterLength) override {}
+
+  void SetComposingRegion(int32_t start, int32_t end) override {}
+
+  void SetComposingText(const mojo::String& text,
+                        int32_t newCursorPosition) override {}
+
+  void SetSelection(int32_t start, int32_t end) override {}
+
+ private:
+  mojo::Binding<keyboard::KeyboardClient> binding_;
+  keyboard::KeyboardServicePtr keyboard_;
+  keyboard::KeyboardClientPtr keyboardClient_;
+};

[jamesr, 2015/05/08 23:45:23]

add DISALLOW_COPY_AND_ASSIGN(KeyboardDelegate) to suppress the default
assignment operator and copy constructor. they won't work for this type, but the
macro will be clearer

[APW, 2015/05/09 00:45:58]

Done.

+
+}  // namespace examples
+
+MojoResult MojoMain(MojoHandle application_request) {
+  mojo::ApplicationRunner runner(new examples::KeyboardDelegate);
+  return runner.Run(application_request);
+}