Update the native_viewport interface to allow specification of the surface configuration, currently…

apps/moterm/gl_helper_test_app.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.
 
 // This is a test application for gl_helper.*, which draws directly to a native
 // viewport (without using the view manager).
 
 #include <GLES2/gl2.h>
 #include <math.h>
 
@@ skipping 200 matching lines @@
   // |mojo::ApplicationDelegate|:
   void Initialize(mojo::ApplicationImpl* application_impl) override {
     DCHECK(!application_impl_);
     application_impl_ = application_impl;
 
     application_impl->ConnectToService("mojo:native_viewport_service",
                                        &native_viewport_);
     viewport_size_.width = 800;
     viewport_size_.height = 600;
     native_viewport_->Create(
-        viewport_size_.Clone(),
+        viewport_size_.Clone(), mojo::SurfaceConfiguration::New(),

[viettrungluu, 2015/06/09 22:38:35]

It's up to you, but you could also just pass nullptr here (and elsewhere), I
think.

[iansf, 2015/06/09 23:45:25]

That's true!  For example apps like this, do you think it's better to have a
potentially mysterious nullptr, or a more verbose explicit default?  I'm fine
either way.

         base::Bind(&GlHelperExampleApp::OnViewportMetricsReceived,
                    base::Unretained(this)));
     native_viewport_->Show();
 
     mojo::ContextProviderPtr context_provider;
     native_viewport_->GetContextProvider(GetProxy(&context_provider));
 
     mojo::DisplayFactoryPtr display_factory;
     application_impl_->ConnectToService("mojo:surfaces_service",
                                         &display_factory);
@@ skipping 38 matching lines @@
 };
 
 const GLuint GlHelperExampleApp::kPositionLocation;
 
 }  // namespace
 
 MojoResult MojoMain(MojoHandle application_request) {
   mojo::ApplicationRunnerChromium runner(new GlHelperExampleApp());
   return runner.Run(application_request);
 }