Plumb resize notifications around sky/shell

sky/shell/shell.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/shell.h"
 
 #include "base/bind.h"
 #include "base/single_thread_task_runner.h"
 #include "base/threading/thread.h"
+#include "sky/shell/gpu/rasterizer.h"
 #include "sky/shell/sky_view.h"
+#include "sky/shell/ui/engine.h"
 
 namespace sky {
 namespace shell {
 
 Shell::Shell(scoped_refptr<base::SingleThreadTaskRunner> java_task_runner)
     : java_task_runner_(java_task_runner) {
 }
 
 Shell::~Shell() {
 }
 
 void Shell::Init() {
   gpu_thread_.reset(new base::Thread("gpu_thread"));
   gpu_thread_->Start();
   rasterizer_.reset(new Rasterizer());
 
   ui_thread_.reset(new base::Thread("ui_thread"));
   ui_thread_->Start();
   engine_.reset(new Engine());
 
   ui_thread_->message_loop()->PostTask(
       FROM_HERE, base::Bind(&Engine::Init, engine_->GetWeakPtr()));
 
-  view_.reset(new SkyView(this));
-  view_->Init();
-}
+  SkyView::Config config;
+  config.gpu_task_runner = gpu_thread_->message_loop()->task_runner();

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

Feels a little odd to not just ask the gpu_delegate for its runner.

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

Yeah, I just didn't want to talk to the gpu_delegate except on its own thread. 
It would be nice to have an atomic unit that was an interface and a task runner
that automatically forwarded your calls through the interface to the impl via
the task runner.  That's sort of what MojoMessagePipes accomplish, but with
extra layers of encoding/decoding to support multiprocess.

+  config.gpu_delegate = rasterizer_->GetWeakPtr();
+  config.ui_task_runner = ui_thread_->message_loop()->task_runner();
+  config.ui_delegate = engine_->GetWeakPtr();
 
-void Shell::OnAcceleratedWidgetAvailable(gfx::AcceleratedWidget widget) {
-  gpu_thread_->message_loop()->PostTask(
-      FROM_HERE,
-      base::Bind(&Rasterizer::Init, rasterizer_->GetWeakPtr(), widget));
-}
-
-void Shell::OnDestroyed() {
+  view_.reset(new SkyView(config));
 }
 
 }  // namespace shell
 }  // namespace sky