Refactor context creation parameters into a struct.

android_webview/browser/scoped_app_gl_state_restore.cc

Index: android_webview/browser/scoped_app_gl_state_restore.cc
diff --git a/android_webview/browser/scoped_app_gl_state_restore.cc b/android_webview/browser/scoped_app_gl_state_restore.cc
index 42fbb7e9776d8d5509552766887fcb0b9fffe8b0..9b35faf0e7b664179fbba65d77a9d0a87666cc08 100644
--- a/android_webview/browser/scoped_app_gl_state_restore.cc
+++ b/android_webview/browser/scoped_app_gl_state_restore.cc
@@ -25,7 +25,7 @@ class AppContextSurface {
       : surface(new gl::GLSurfaceStub),
         context(gl::init::CreateGLContext(nullptr,
                                           surface.get(),
-                                          gl::PreferDiscreteGpu)) {}
+                                          gl::GLContextAttribs())) {}

[Torne, 2016/11/08 14:34:29]

If I'm reading correctly the default is PreferIntegratedGpu, but we were passing
PreferDiscreteGpu before. Is this an intentional change?

[Geoff Lang, 2016/11/08 14:44:01]

Yes, piman suggested it because Android doesn't have dual-gpu support.

   void MakeCurrent() { context->MakeCurrent(surface.get()); }
 
  private: