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

apps/moterm/gl_helper_test_app.cc

Index: apps/moterm/gl_helper_test_app.cc
diff --git a/apps/moterm/gl_helper_test_app.cc b/apps/moterm/gl_helper_test_app.cc
index f5cff3bb3b81bc3223ff9d4d3876b60a01abbe03..e2c88e3f6e82d47e75e4d42e18ce4b438de3f0e0 100644
--- a/apps/moterm/gl_helper_test_app.cc
+++ b/apps/moterm/gl_helper_test_app.cc
@@ -218,7 +218,7 @@ class GlHelperExampleApp : public mojo::ApplicationDelegate,
     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();