cc: Make OutputSurface::Reshape abstract

content/browser/compositor/offscreen_browser_compositor_output_surface.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.
 
 #include "content/browser/compositor/offscreen_browser_compositor_output_surface.h"
 
 #include <utility>
 
 #include "base/logging.h"
 #include "build/build_config.h"
@@ skipping 39 matching lines @@
 void OffscreenBrowserCompositorOutputSurface::EnsureBackbuffer() {
   bool update_source_texture = !reflector_texture_ || reflector_changed_;
   reflector_changed_ = false;
   if (!reflector_texture_) {
     reflector_texture_.reset(new ReflectorTexture(context_provider()));
 
     GLES2Interface* gl = context_provider_->ContextGL();
 
     const int max_texture_size =
         context_provider_->ContextCapabilities().max_texture_size;
-    int texture_width = std::min(max_texture_size, surface_size_.width());
-    int texture_height = std::min(max_texture_size, surface_size_.height());
+    int texture_width = std::min(max_texture_size, reshape_size_.width());
+    int texture_height = std::min(max_texture_size, reshape_size_.height());
 
     gl->BindTexture(GL_TEXTURE_2D, reflector_texture_->texture_id());
     gl->TexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR);
     gl->TexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_LINEAR);
     gl->TexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_EDGE);
     gl->TexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_CLAMP_TO_EDGE);
     gl->TexImage2D(GL_TEXTURE_2D, 0, GLInternalFormat(kFboTextureFormat),
                    texture_width, texture_height, 0,
                    GLDataFormat(kFboTextureFormat),
                    GLDataType(kFboTextureFormat), nullptr);
@@ skipping 28 matching lines @@
     gl->DeleteFramebuffers(1, &fbo_);
     fbo_ = 0;
   }
 }
 
 void OffscreenBrowserCompositorOutputSurface::Reshape(
     const gfx::Size& size,
     float scale_factor,
     const gfx::ColorSpace& color_space,
     bool alpha) {
-  if (size == surface_size_)
-    return;
-
-  surface_size_ = size;
-  device_scale_factor_ = scale_factor;
+  reshape_size_ = size;
   DiscardBackbuffer();
   EnsureBackbuffer();
 }
 
 void OffscreenBrowserCompositorOutputSurface::BindFramebuffer() {
   bool need_to_bind = !!reflector_texture_.get();
 
   EnsureBackbuffer();
   DCHECK(reflector_texture_.get());
   DCHECK(fbo_);
 
   if (need_to_bind) {
     GLES2Interface* gl = context_provider_->ContextGL();
     gl->BindFramebuffer(GL_FRAMEBUFFER, fbo_);
   }
 }
 
 void OffscreenBrowserCompositorOutputSurface::SwapBuffers(
     cc::OutputSurfaceFrame frame) {
   gfx::Size surface_size = frame.size;
-  DCHECK(surface_size == surface_size_);
+  DCHECK(surface_size == reshape_size_);
   gfx::Rect swap_rect = frame.sub_buffer_rect;
 
   if (reflector_) {
     if (swap_rect == gfx::Rect(surface_size))
       reflector_->OnSourceSwapBuffers(surface_size);
     else
       reflector_->OnSourcePostSubBuffer(swap_rect, surface_size);
   }
 
   // TODO(oshima): sync with the reflector's SwapBuffersComplete
@@ skipping 36 matching lines @@
     reflector_changed_ = true;
     EnsureBackbuffer();
   }
 }
 
 void OffscreenBrowserCompositorOutputSurface::OnSwapBuffersComplete() {
   client_->DidSwapBuffersComplete();
 }
 
 }  // namespace content