Send size to NativeViewportClient::OnCreated instead of GLES2Client::DidCreateContext

mojo/services/native_viewport/native_viewport_service.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 "mojo/services/native_viewport/native_viewport_service.h"
 
 #include "base/message_loop/message_loop.h"
 #include "base/time/time.h"
 #include "mojo/services/gles2/gles2_impl.h"
 #include "mojo/services/native_viewport/geometry_conversions.h"
@@ skipping 61 matching lines @@
     DCHECK_EQ(event.time_stamp(), pending_event_timestamp_);
     waiting_for_event_ack_ = false;
   }
 
   void CreateGLES2ContextIfNeeded() {
     if (created_context_)
       return;
     if (widget_ == gfx::kNullAcceleratedWidget || !gles2_)
       return;
     gfx::Size size = native_viewport_->GetSize();
-    if (size.width() == 0 || size.height() == 0)
+    if (size.IsEmpty())
       return;
     gles2_->CreateContext(widget_, size);
     created_context_ = true;
   }
 
   virtual bool OnEvent(ui::Event* ui_event) MOJO_OVERRIDE {
     // Must not return early before updating capture.
     switch (ui_event->type()) {
     case ui::ET_MOUSE_PRESSED:
     case ui::ET_TOUCH_PRESSED:
@@ skipping 46 matching lines @@
   }
 
   virtual void OnAcceleratedWidgetAvailable(
       gfx::AcceleratedWidget widget) MOJO_OVERRIDE {
     widget_ = widget;
     CreateGLES2ContextIfNeeded();
   }
 
   virtual void OnBoundsChanged(const gfx::Rect& bounds) MOJO_OVERRIDE {
     CreateGLES2ContextIfNeeded();
+    AllocationScope allocation;
     client_->OnBoundsChanged(bounds);
   }
 
   virtual void OnDestroyed() MOJO_OVERRIDE {
     // TODO(beng):
     // Destroying |gles2_| on the shell thread here hits thread checker
     // asserts. All code must stop touching the AcceleratedWidget at this
     // point as it is dead after this call stack. jamesr said we probably
     // should make our own GLSurface and simply tell it to stop touching the
     // AcceleratedWidget via Destroy() but we have no good way of doing that
@@ skipping 47 matching lines @@
 extern "C" MOJO_NATIVE_VIEWPORT_EXPORT MojoResult MojoMain(
     const MojoHandle shell_handle) {
   base::MessageLoopForUI loop;
   mojo::services::NativeViewportService app(
       mojo::MakeScopedHandle(mojo::MessagePipeHandle(shell_handle)).Pass());
   base::MessageLoop::current()->Run();
   return MOJO_RESULT_OK;
 }
 
 #endif // !OS_ANDROID