Allow native_viewport to create new native windows on demand on Android.

services/native_viewport/platform_viewport_android.cc

 // Copyright 2013 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 "services/native_viewport/platform_viewport_android.h"
 
 #include <android/input.h>
 #include <android/native_window_jni.h>
 
 #include "base/android/jni_android.h"
 #include "jni/PlatformViewportAndroid_jni.h"
 #include "mojo/converters/geometry/geometry_type_converters.h"
 #include "mojo/converters/input_events/input_events_type_converters.h"
+#include "mojo/public/cpp/application/application_impl.h"
+#include "services/native_viewport/native_viewport_internal.mojom.h"
 #include "ui/events/event.h"
 #include "ui/events/keycodes/keyboard_code_conversion_android.h"
 #include "ui/gfx/point.h"
 
 namespace native_viewport {
 namespace {
 
 mojo::EventType MotionEventActionToEventType(jint action) {
   switch (action) {
     case AMOTION_EVENT_ACTION_DOWN:
@@ skipping 20 matching lines @@
 }  // namespace
 
 ////////////////////////////////////////////////////////////////////////////////
 // PlatformViewportAndroid, public:
 
 // static
 bool PlatformViewportAndroid::Register(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
-PlatformViewportAndroid::PlatformViewportAndroid(Delegate* delegate)
-    : delegate_(delegate),
+PlatformViewportAndroid::PlatformViewportAndroid(
+    mojo::ApplicationImpl* application,
+    Delegate* delegate)
+    : application_(application),
+      delegate_(delegate),
       window_(NULL),
       id_generator_(0),
-      weak_factory_(this) {
-}
+      weak_factory_(this) {}
 
 PlatformViewportAndroid::~PlatformViewportAndroid() {
   if (window_)
     ReleaseWindow();
+  JNIEnv* env = base::android::AttachCurrentThread();
   if (!java_platform_viewport_android_.is_empty()) {
-    JNIEnv* env = base::android::AttachCurrentThread();
     Java_PlatformViewportAndroid_detach(
         env, java_platform_viewport_android_.get(env).obj());
+  } else {
+    Java_PlatformViewportAndroid_withdrawRequest(env,
+                                                 reinterpret_cast<jlong>(this));

[qsr, 2015/08/07 14:09:39]

reinterpret_cast should be to intptr_t, here and everywhere on this file.

[etiennej, 2015/08/07 14:15:40]

Done.

   }
 }
 
 void PlatformViewportAndroid::Destroy(JNIEnv* env, jobject obj) {
   delegate_->OnDestroyed();
 }
 
+void PlatformViewportAndroid::SurfaceAttached(JNIEnv* env,
+                                              jobject obj,
+                                              jobject platform_viewport) {
+  java_platform_viewport_android_ =
+      JavaObjectWeakGlobalRef(env, platform_viewport);
+}
+
 void PlatformViewportAndroid::SurfaceCreated(JNIEnv* env,
                                              jobject obj,
                                              jobject jsurface) {
   base::android::ScopedJavaLocalRef<jobject> protector(env, jsurface);
   // Note: This ensures that any local references used by
   // ANativeWindow_fromSurface are released immediately. This is needed as a
   // workaround for https://code.google.com/p/android/issues/detail?id=68174
   {
     base::android::ScopedJavaLocalFrame scoped_local_reference_frame(env);
     window_ = ANativeWindow_fromSurface(env, jsurface);
@@ skipping 56 matching lines @@
   delegate_->OnEvent(event.Pass());
 
   return true;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // PlatformViewportAndroid, PlatformViewport implementation:
 
 void PlatformViewportAndroid::Init(const gfx::Rect& bounds) {
   JNIEnv* env = base::android::AttachCurrentThread();
-  java_platform_viewport_android_ = JavaObjectWeakGlobalRef(
-      env,
-      Java_PlatformViewportAndroid_create(env, reinterpret_cast<jlong>(this))
-          .obj());
+  Java_PlatformViewportAndroid_createRequest(env,
+                                             reinterpret_cast<jlong>(this));
+
+  NativeViewportShellServicePtr shell_service;
+  application_->ConnectToService("mojo:native_viewport_support",
+                                 &shell_service);
+  shell_service->CreateNewNativeWindow();
 }
 
 void PlatformViewportAndroid::Show() {
   // Nothing to do. View is created visible.
 }
 
 void PlatformViewportAndroid::Hide() {
   // Nothing to do. View is always visible.
 }
 
@@ skipping 17 matching lines @@
 
 void PlatformViewportAndroid::ReleaseWindow() {
   ANativeWindow_release(window_);
   window_ = NULL;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // PlatformViewport, public:
 
 // static
-scoped_ptr<PlatformViewport> PlatformViewport::Create(Delegate* delegate) {
+scoped_ptr<PlatformViewport> PlatformViewport::Create(
+    mojo::ApplicationImpl* application_,
+    Delegate* delegate) {
   return scoped_ptr<PlatformViewport>(
-      new PlatformViewportAndroid(delegate)).Pass();
+             new PlatformViewportAndroid(application_, delegate))
+      .Pass();
 }
 
 }  // namespace native_viewport