Plumb resize notifications around sky/shell

sky/shell/sky_view.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 "sky/shell/sky_view.h"
 
 #include <android/input.h>
 #include <android/native_window_jni.h>
 
 #include "base/android/jni_android.h"
+#include "base/bind.h"
+#include "base/location.h"
 #include "jni/SkyView_jni.h"
 
 namespace sky {
 namespace shell {
 
 // static
 bool SkyView::Register(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
-SkyView::Delegate::~Delegate() {
-}
-
-SkyView::SkyView(Delegate* delegate) : delegate_(delegate), window_(NULL) {
+SkyView::SkyView(const Config& config) : config_(config), window_(nullptr) {
+  JNIEnv* env = base::android::AttachCurrentThread();
+  Java_SkyView_createForActivity(env, base::android::GetApplicationContext(),
+                                 reinterpret_cast<jlong>(this));

[eseidel, 2015/02/05 05:57:28]

Bleh.  Really?  Reinterpret-casting a pointer to a long?  That's OK in 64bit?

[abarth-chromium, 2015/02/05 06:16:23]

Yeah, that's how our JNI system works.  It's super lame but it does work on 64
bit.  We used to use a jint, which isn't ok on 64 bit.  When we started
supporting 64 bit, some poor person had to go through the entire system and
change jints to jlongs.  :(

JNI: Not even once.

[eseidel, 2015/02/05 07:06:44]

And we don't have any helpers to hide the crazy?  jniPtr(this)?

 }
 
 SkyView::~SkyView() {
   if (window_)
     ReleaseWindow();
 }
 
-void SkyView::Init() {
-  JNIEnv* env = base::android::AttachCurrentThread();
-  Java_SkyView_createForActivity(env, base::android::GetApplicationContext(),
-                                 reinterpret_cast<jlong>(this));
-}
-
 void SkyView::Destroy(JNIEnv* env, jobject obj) {
-  delegate_->OnDestroyed();
 }
 
 void SkyView::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);
   }
-  delegate_->OnAcceleratedWidgetAvailable(window_);
+  config_.gpu_task_runner->PostTask(
+      FROM_HERE, base::Bind(&GPUDelegate::OnAcceleratedWidgetAvailable,
+                            config_.gpu_delegate, window_));
 }
 
 void SkyView::SurfaceDestroyed(JNIEnv* env, jobject obj) {
   DCHECK(window_);
+  config_.gpu_task_runner->PostTask(
+      FROM_HERE, base::Bind(&GPUDelegate::OnDestroyed, config_.gpu_delegate));
   ReleaseWindow();
 }
 
 void SkyView::SurfaceSetSize(JNIEnv* env,
                              jobject obj,
                              jint width,
                              jint height,
                              jfloat density) {
+  config_.ui_task_runner->PostTask(
+      FROM_HERE,
+      base::Bind(&UIDelegate::OnViewportMetricsChanged, config_.ui_delegate,
+                 gfx::Size(width, height), density));
 }
 
 void SkyView::ReleaseWindow() {
   ANativeWindow_release(window_);
-  window_ = NULL;
+  window_ = nullptr;
 }
 
 }  // namespace shell
 }  // namespace sky