compositor: Fix texture flipping for SW mirroring with surfaceless

content/browser/compositor/reflector_impl.cc

 // Copyright 2014 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/reflector_impl.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "content/browser/compositor/browser_compositor_output_surface.h"
 #include "content/browser/compositor/owned_mailbox.h"
 #include "content/common/gpu/client/gl_helper.h"
 #include "ui/compositor/layer.h"
 
 namespace content {
 
 ReflectorImpl::ReflectorImpl(
     ui::Compositor* mirrored_compositor,
     ui::Layer* mirroring_layer,
     IDMap<BrowserCompositorOutputSurface>* output_surface_map,
     base::MessageLoopProxy* compositor_thread_loop,
     int surface_id)
     : impl_unsafe_(output_surface_map),
       main_unsafe_(mirrored_compositor, mirroring_layer),
       impl_message_loop_(compositor_thread_loop),
       main_message_loop_(base::MessageLoopProxy::current()),
-      surface_id_(surface_id) {
+      surface_id_(surface_id),
+      flip_texture_(false) {
   GLHelper* helper = ImageTransportFactory::GetInstance()->GetGLHelper();
   MainThreadData& main = GetMain();
   main.mailbox = new OwnedMailbox(helper);
   impl_message_loop_->PostTask(
       FROM_HERE,
       base::Bind(
           &ReflectorImpl::InitOnImplThread, this, main.mailbox->holder()));
 }
 
 ReflectorImpl::MainThreadData::MainThreadData(
@@ skipping 139 matching lines @@
 
 void ReflectorImpl::AttachToOutputSurfaceOnImplThread(
     const gpu::MailboxHolder& mailbox_holder,
     BrowserCompositorOutputSurface* output_surface) {
   ImplThreadData& impl = GetImpl();
   if (output_surface == impl.output_surface)
     return;
   if (impl.output_surface)
     DetachFromOutputSurface();
   impl.output_surface = output_surface;
+  flip_texture_ = !output_surface->capabilities().flipped_output_surface;

[danakj, 2015/01/22 01:04:20]

capabilities are not initialized until the context is bound (the line below
this)

https://code.google.com/p/chromium/codesearch#chromium/src/content/common/gpu...

[achaulk, 2015/01/23 16:45:52]

Done.

   output_surface->context_provider()->BindToCurrentThread();
   impl.gl_helper.reset(
       new GLHelper(output_surface->context_provider()->ContextGL(),
                    output_surface->context_provider()->ContextSupport()));
   impl.texture_id = impl.gl_helper->ConsumeMailboxToTexture(
       mailbox_holder.mailbox, mailbox_holder.sync_point);
   impl.gl_helper->ResizeTexture(impl.texture_id, output_surface->SurfaceSize());
   impl.gl_helper->Flush();
   output_surface->SetReflector(this);
   // The texture doesn't have the data, so invokes full redraw now.
   main_message_loop_->PostTask(
       FROM_HERE,
       base::Bind(&ReflectorImpl::FullRedrawContentOnMainThread,

[danakj, 2015/01/22 01:04:20]

Maybe pass the flipped bool along here, so you can store it in the
MainThreadData?

                  scoped_refptr<ReflectorImpl>(this)));
 }
 
 void ReflectorImpl::UpdateTextureSizeOnMainThread(gfx::Size size) {
   MainThreadData& main = GetMain();
   if (!main.mirroring_layer || !main.mailbox.get() ||
       main.mailbox->mailbox().IsZero())
     return;
   if (main.needs_set_mailbox) {
     main.mirroring_layer->SetTextureMailbox(
         cc::TextureMailbox(main.mailbox->holder()),
         cc::SingleReleaseCallback::Create(
             base::Bind(ReleaseMailbox, main.mailbox)),
         size);
     main.needs_set_mailbox = false;
   } else {
     main.mirroring_layer->SetTextureSize(size);
   }
   main.mirroring_layer->SetBounds(gfx::Rect(size));
+  main.mirroring_layer->SetTextureFlipped(flip_texture_);
 }
 
 void ReflectorImpl::FullRedrawOnMainThread(gfx::Size size) {
   MainThreadData& main = GetMain();
   if (!main.mirroring_layer)
     return;
   UpdateTextureSizeOnMainThread(size);
   main.mirroring_layer->SchedulePaint(main.mirroring_layer->bounds());
 }
 
-void ReflectorImpl::UpdateSubBufferOnMainThread(gfx::Size size,
-                                                gfx::Rect rect) {
+void ReflectorImpl::UpdateSubBufferOnMainThread(const gfx::Size& size,
+                                                const gfx::Rect& rect) {
   MainThreadData& main = GetMain();
   if (!main.mirroring_layer)
     return;
   UpdateTextureSizeOnMainThread(size);
+
+  int y = rect.y();
   // Flip the coordinates to compositor's one.
-  int y = size.height() - rect.y() - rect.height();
+  if (flip_texture_)
+    y = size.height() - rect.y() - rect.height();
   gfx::Rect new_rect(rect.x(), y, rect.width(), rect.height());
   main.mirroring_layer->SchedulePaint(new_rect);
 }
 
 void ReflectorImpl::FullRedrawContentOnMainThread() {
   MainThreadData& main = GetMain();
   main.mirrored_compositor->ScheduleFullRedraw();
 }
 
 }  // namespace content