Plumb resize notifications around sky/shell

sky/shell/sky_view.h

 // 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.
 
 #ifndef SKY_SHELL_SKY_VIEW_H_
 #define SKY_SHELL_SKY_VIEW_H_
 
 #include "base/android/jni_weak_ref.h"
 #include "base/android/scoped_java_ref.h"
 #include "base/macros.h"
-#include "ui/gfx/native_widget_types.h"
+#include "base/memory/weak_ptr.h"
+#include "base/single_thread_task_runner.h"
+#include "sky/shell/gpu_delegate.h"
+#include "sky/shell/ui_delegate.h"
 
 struct ANativeWindow;
 
 namespace sky {
 namespace shell {
 
 class SkyView {
  public:
+  struct Config {
+    base::WeakPtr<GPUDelegate> gpu_delegate;
+    scoped_refptr<base::SingleThreadTaskRunner> gpu_task_runner;

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

Is it OK to hold the runner after the delegate goes away?

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

Yes, base::SingleThreadTaskRunner is designed to be safe to hold forever with a
scoped_refptr.  If the thread that backs the task runner goes away, then the
tasks just don't run.  That's actually the original problem it was invented to
solve.

+
+    base::WeakPtr<UIDelegate> ui_delegate;
+    scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner;
+  };
+
   static bool Register(JNIEnv* env);
 
-  class Delegate {
-   public:
-    virtual void OnAcceleratedWidgetAvailable(
-        gfx::AcceleratedWidget widget) = 0;
-    virtual void OnDestroyed() = 0;
-
-   protected:
-    virtual ~Delegate();
-  };
-
-  explicit SkyView(Delegate* delegate);
+  explicit SkyView(const Config& config);
   ~SkyView();
 
-  void Init();
-
   // Called from Java
   void Destroy(JNIEnv* env, jobject obj);
   void SurfaceCreated(JNIEnv* env, jobject obj, jobject jsurface);
   void SurfaceDestroyed(JNIEnv* env, jobject obj);
   void SurfaceSetSize(JNIEnv* env,
                       jobject obj,
                       jint width,
                       jint height,
                       jfloat density);
 
  private:
   void ReleaseWindow();
 
-  Delegate* delegate_;
+  Config config_;
   ANativeWindow* window_;
 
   DISALLOW_COPY_AND_ASSIGN(SkyView);
 };
 
 }  // namespace shell
 }  // namespace sky
 
 #endif  // SKY_SHELL_SKY_VIEW_H_