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

services/gles2/command_buffer_driver.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 "services/gles2/command_buffer_driver.h"
 
 #include "base/bind.h"
 #include "base/macros.h"
 #include "base/memory/shared_memory.h"
 #include "gpu/command_buffer/common/constants.h"
@@ skipping 35 matching lines @@
 };
 
 }  // anonymous namespace
 
 CommandBufferDriver::Client::~Client() {
 }
 
 CommandBufferDriver::CommandBufferDriver(
     gfx::GLShareGroup* share_group,
     gpu::gles2::MailboxManager* mailbox_manager,
-    gpu::SyncPointManager* sync_point_manager)
+    gpu::SyncPointManager* sync_point_manager,
+    gfx::SurfaceConfiguration requested_configuration)

[abarth-chromium, 2015/06/09 00:30:21]

gfx::SurfaceConfiguration -> const gfx::SurfaceConfiguration&

[iansf, 2015/06/09 01:52:04]

Done.

     : CommandBufferDriver(gfx::kNullAcceleratedWidget,
                           share_group,
                           mailbox_manager,
-                          sync_point_manager) {
+                          sync_point_manager,
+                          requested_configuration) {
 }
 
 CommandBufferDriver::CommandBufferDriver(
     gfx::AcceleratedWidget widget,
     gfx::GLShareGroup* share_group,
     gpu::gles2::MailboxManager* mailbox_manager,
-    gpu::SyncPointManager* sync_point_manager)
+    gpu::SyncPointManager* sync_point_manager,
+    gfx::SurfaceConfiguration requested_configuration)

[abarth-chromium, 2015/06/09 00:30:21]

gfx::SurfaceConfiguration -> const gfx::SurfaceConfiguration&

[iansf, 2015/06/09 01:52:04]

Done.

     : client_(nullptr),
       widget_(widget),
       share_group_(share_group),
       mailbox_manager_(mailbox_manager),
       sync_point_manager_(sync_point_manager),
+      requested_configuration_(requested_configuration),
       weak_factory_(this) {
 }
 
 CommandBufferDriver::~CommandBufferDriver() {
   if (decoder_) {
     bool have_context = decoder_->MakeCurrent();
     decoder_->Destroy(have_context);
   }
 }
 
 void CommandBufferDriver::Initialize(
     mojo::CommandBufferSyncClientPtr sync_client,
     mojo::CommandBufferLostContextObserverPtr loss_observer,
     mojo::ScopedSharedBufferHandle shared_state) {
   sync_client_ = sync_client.Pass();
   loss_observer_ = loss_observer.Pass();
   bool success = DoInitialize(shared_state.Pass());
   mojo::GpuCapabilitiesPtr capabilities =
       success ? mojo::GpuCapabilities::From(decoder_->GetCapabilities())
               : mojo::GpuCapabilities::New();
   sync_client_->DidInitialize(success, capabilities.Pass());
 }
 
 bool CommandBufferDriver::DoInitialize(
     mojo::ScopedSharedBufferHandle shared_state) {
   if (widget_ == gfx::kNullAcceleratedWidget)
-    surface_ = gfx::GLSurface::CreateOffscreenGLSurface(gfx::Size(1, 1));
+    surface_ = gfx::GLSurface::CreateOffscreenGLSurface(
+        gfx::Size(1, 1), requested_configuration_);
   else {
-    surface_ = gfx::GLSurface::CreateViewGLSurface(widget_);
+    surface_ =
+        gfx::GLSurface::CreateViewGLSurface(widget_, requested_configuration_);
     if (auto vsync_provider = surface_->GetVSyncProvider()) {
       vsync_provider->GetVSyncParameters(
           base::Bind(&CommandBufferDriver::OnUpdateVSyncParameters,
                      weak_factory_.GetWeakPtr()));
     }
   }
 
   if (!surface_.get())
     return false;
 
@@ skipping 129 matching lines @@
   client_->DidLoseContext();
 }
 
 void CommandBufferDriver::OnUpdateVSyncParameters(
     const base::TimeTicks timebase,
     const base::TimeDelta interval) {
   client_->UpdateVSyncParameters(timebase, interval);
 }
 
 }  // namespace gles2