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

mojo/apps/js/main.js

Index: mojo/apps/js/main.js
diff --git a/mojo/apps/js/main.js b/mojo/apps/js/main.js
index 9ca9e9d4c7f3bd076aae146d56126920bb781364..348bd01f3a1b9821e4685fefe706480fd188d88d 100644
--- a/mojo/apps/js/main.js
+++ b/mojo/apps/js/main.js
@@ -295,7 +295,7 @@ define([
     this.remote_ = remote;
 
     var pipe = core.createMessagePipe();
-    new GLES2ClientImpl(pipe.handle0);
+    this.gles2_ = new GLES2ClientImpl(pipe.handle0);
 
     this.remote_.open();
     this.remote_.createGLES2Context(pipe.handle1);
@@ -303,7 +303,8 @@ define([
   NativeViewportClientImpl.prototype =
       Object.create(nativeViewport.NativeViewportClientStub.prototype);
 
-  NativeViewportClientImpl.prototype.onCreated = function() {
+  NativeViewportClientImpl.prototype.onCreated = function(width, height) {
+    this.gles2_.setDimensions(width, height);
     console.log('NativeViewportClientImpl.prototype.OnCreated');
   };
   NativeViewportClientImpl.prototype.didOpen = function() {
@@ -317,9 +318,7 @@ define([
     this.angle_ = 45;
   }
 
-  GLES2ClientImpl.prototype.didCreateContext = function(width, height) {
-    this.width_ = width;
-    this.height_ = height;
+  GLES2ClientImpl.prototype.didCreateContext = function() {
     this.program_ = loadProgram(this.gl_);
     this.positionLocation_ =
         this.gl_.getAttribLocation(this.program_, 'a_position');
@@ -333,7 +332,14 @@ define([
     this.timer_ = timer.createRepeating(16, this.handleTimer.bind(this));
   };
 
+  GLES2ClientImpl.prototype.setDimensions = function(width, height) {
+    this.width_ = width;
+    this.height_ = height;
+  }
+
   GLES2ClientImpl.prototype.drawCube = function() {
+    if (!this.width_ || !this.height_)
+      return;
     this.gl_.viewport(0, 0, this.width_, this.height_);
     this.gl_.clear(this.gl_.COLOR_BUFFER_BIT);
     this.gl_.useProgram(this.program_);