Revert of [Reland 1] Pepper takes ownership of a mailbox before passing it to the texture layer.

gpu/command_buffer/tests/gl_texture_mailbox_unittest.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 <GLES2/gl2.h>
 #include <GLES2/gl2ext.h>
 #include <GLES2/gl2extchromium.h>
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 50 matching lines @@
     GLManager::Options options;
     options.share_mailbox_manager = &gl1_;
     gl2_.Initialize(options);
   }
 
   void TearDown() override {
     gl1_.Destroy();
     gl2_.Destroy();
   }
 
-  // The second GL context takes and consumes a mailbox from the first GL
-  // context. Assumes that |gl1_| is current.
-  Mailbox TakeAndConsumeMailbox() {
-    glResizeCHROMIUM(10, 10, 1, true);
-    glClearColor(0, 1, 1, 1);
-    glClear(GL_COLOR_BUFFER_BIT);
-    ::gles2::GetGLContext()->SwapBuffers();
-
-    Mailbox mailbox;
-    glGenMailboxCHROMIUM(mailbox.name);
-    gl1_.decoder()->TakeFrontBuffer(mailbox);
-
-    gl2_.MakeCurrent();
-    GLuint tex;
-    glGenTextures(1, &tex);
-    glBindTexture(GL_TEXTURE_2D, tex);
-    glConsumeTextureCHROMIUM(GL_TEXTURE_2D, mailbox.name);
-    glDeleteTextures(1, &tex);
-    glFlush();
-    gl1_.MakeCurrent();
-    return mailbox;
-  }
-
   GLManager gl1_;
   GLManager gl2_;
 };
 
 TEST_F(GLTextureMailboxTest, ProduceAndConsumeTexture) {
   gl1_.MakeCurrent();
 
   GLbyte mailbox1[GL_MAILBOX_SIZE_CHROMIUM];
   glGenMailboxCHROMIUM(mailbox1);
 
@@ skipping 239 matching lines @@
 
   // Mailbox should be gone now.
   glGenTextures(1, &tex2);
   glBindTexture(GL_TEXTURE_2D, tex2);
   glConsumeTextureCHROMIUM(GL_TEXTURE_2D, mailbox);
   EXPECT_EQ(static_cast<GLenum>(GL_INVALID_OPERATION), glGetError());
   glDeleteTextures(1, &tex2);
   EXPECT_EQ(static_cast<GLenum>(GL_NO_ERROR), glGetError());
 }
 
-TEST_F(GLTextureMailboxTest, TakeFrontBuffer) {
+TEST_F(GLTextureMailboxTest, ProduceFrontBuffer) {
   gl1_.MakeCurrent();
   Mailbox mailbox;
   glGenMailboxCHROMIUM(mailbox.name);
 
   gl2_.MakeCurrent();
-  glResizeCHROMIUM(10, 10, 1, true);
-  glClearColor(0, 1, 1, 1);
-  glClear(GL_COLOR_BUFFER_BIT);
-  ::gles2::GetGLContext()->SwapBuffers();
-  gl2_.decoder()->TakeFrontBuffer(mailbox);
+  gl2_.decoder()->ProduceFrontBuffer(mailbox);
 
   gl1_.MakeCurrent();
   GLuint tex1;
   glGenTextures(1, &tex1);
   glBindTexture(GL_TEXTURE_2D, tex1);
   glConsumeTextureCHROMIUM(GL_TEXTURE_2D, mailbox.name);
   EXPECT_EQ(static_cast<GLenum>(GL_NO_ERROR), glGetError());
-  EXPECT_EQ(0xFFFFFF00u, ReadTexel(tex1, 0, 0));
 
   gl2_.MakeCurrent();
+  glResizeCHROMIUM(10, 10, 1, true);
   glClearColor(1, 0, 0, 1);
   glClear(GL_COLOR_BUFFER_BIT);
   ::gles2::GetGLContext()->SwapBuffers();
 
   gl1_.MakeCurrent();
-  EXPECT_EQ(0xFFFFFF00u, ReadTexel(tex1, 0, 0));
-
-  glDeleteTextures(1, &tex1);
-
-  Mailbox mailbox2;
-  glGenMailboxCHROMIUM(mailbox2.name);
+  EXPECT_EQ(0xFF0000FFu, ReadTexel(tex1, 0, 0));
+  EXPECT_EQ(0xFF0000FFu, ReadTexel(tex1, 9, 9));
+  EXPECT_EQ(static_cast<GLenum>(GL_NO_ERROR), glGetError());
 
   gl2_.MakeCurrent();
-  gl2_.decoder()->ReturnFrontBuffer(mailbox, false);
-
-  // Flushing doesn't matter, only SwapBuffers().
   glClearColor(0, 1, 0, 1);
   glClear(GL_COLOR_BUFFER_BIT);
   glFlush();
 
-  gl2_.decoder()->TakeFrontBuffer(mailbox2);
+  gl1_.MakeCurrent();
+  EXPECT_EQ(0xFF0000FFu, ReadTexel(tex1, 0, 0));
+
+  gl2_.MakeCurrent();
+  ::gles2::GetGLContext()->SwapBuffers();
 
   gl1_.MakeCurrent();
-  glGenTextures(1, &tex1);
-  glBindTexture(GL_TEXTURE_2D, tex1);
-  glConsumeTextureCHROMIUM(GL_TEXTURE_2D, mailbox2.name);
-  EXPECT_EQ(static_cast<GLenum>(GL_NO_ERROR), glGetError());
-  EXPECT_EQ(0xFF0000FFu, ReadTexel(tex1, 0, 0));
+  EXPECT_EQ(0xFF00FF00u, ReadTexel(tex1, 0, 0));
 
   gl2_.MakeCurrent();
   gl2_.Destroy();
 
   gl1_.MakeCurrent();
-  EXPECT_EQ(0xFF0000FFu, ReadTexel(tex1, 0, 0));
+  EXPECT_EQ(0xFF00FF00u, ReadTexel(tex1, 0, 0));
   EXPECT_EQ(static_cast<GLenum>(GL_NO_ERROR), glGetError());
   glDeleteTextures(1, &tex1);
 }
 
-// The client, represented by |gl2_|, will request 5 frontbuffers, and then
-// start returning them.
-TEST_F(GLTextureMailboxTest, FrontBufferCache) {
-  gl1_.MakeCurrent();
-
-  std::vector<Mailbox> mailboxes;
-  for (int i = 0; i < 5; ++i) {
-    Mailbox mailbox = TakeAndConsumeMailbox();
-    mailboxes.push_back(mailbox);
-  }
-  EXPECT_EQ(5u, gl1_.decoder()->GetSavedBackTextureCountForTest());
-  EXPECT_EQ(5u, gl1_.decoder()->GetCreatedBackTextureCountForTest());
-
-  // If the textures aren't lost, they're reused.
-  for (int i = 0; i < 100; ++i) {
-    gl1_.decoder()->ReturnFrontBuffer(mailboxes[0], false);
-    mailboxes.erase(mailboxes.begin());
-
-    Mailbox mailbox = TakeAndConsumeMailbox();
-    mailboxes.push_back(mailbox);
-  }
-
-  EXPECT_EQ(5u, gl1_.decoder()->GetSavedBackTextureCountForTest());
-  EXPECT_EQ(5u, gl1_.decoder()->GetCreatedBackTextureCountForTest());
-
-  // If the textures are lost, they're not reused.
-  for (int i = 0; i < 100; ++i) {
-    gl1_.decoder()->ReturnFrontBuffer(mailboxes[0], true);
-    mailboxes.erase(mailboxes.begin());
-
-    Mailbox mailbox = TakeAndConsumeMailbox();
-    mailboxes.push_back(mailbox);
-  }
-
-  EXPECT_EQ(5u, gl1_.decoder()->GetSavedBackTextureCountForTest());
-  EXPECT_EQ(105u, gl1_.decoder()->GetCreatedBackTextureCountForTest());
-}
-
-// The client, represented by |gl2_|, will request and return 5 frontbuffers.
-// Then the size of the buffer will be changed. All cached frontbuffers should
-// be discarded.
-TEST_F(GLTextureMailboxTest, FrontBufferChangeSize) {
-  gl1_.MakeCurrent();
-
-  std::vector<Mailbox> mailboxes;
-  for (int i = 0; i < 5; ++i) {
-    Mailbox mailbox = TakeAndConsumeMailbox();
-    mailboxes.push_back(mailbox);
-  }
-  EXPECT_EQ(5u, gl1_.decoder()->GetSavedBackTextureCountForTest());
-
-  for (int i = 0; i < 5; ++i) {
-    gl1_.decoder()->ReturnFrontBuffer(mailboxes[i], false);
-  }
-  mailboxes.clear();
-  EXPECT_EQ(5u, gl1_.decoder()->GetSavedBackTextureCountForTest());
-
-  glResizeCHROMIUM(21, 31, 1, true);
-  ::gles2::GetGLContext()->SwapBuffers();
-  EXPECT_EQ(0u, gl1_.decoder()->GetSavedBackTextureCountForTest());
-}
-
 TEST_F(GLTextureMailboxTest, ProduceTextureDirectInvalidTarget) {
   gl1_.MakeCurrent();
 
   GLbyte mailbox1[GL_MAILBOX_SIZE_CHROMIUM];
   glGenMailboxCHROMIUM(mailbox1);
 
   GLuint tex1;
   glGenTextures(1, &tex1);
 
   glBindTexture(GL_TEXTURE_CUBE_MAP, tex1);
   uint32_t source_pixel = 0xFF0000FF;
   glTexImage2D(GL_TEXTURE_CUBE_MAP_POSITIVE_X,
                0,
                GL_RGBA,
                1, 1,
                0,
                GL_RGBA,
                GL_UNSIGNED_BYTE,
                &source_pixel);
 
   glProduceTextureDirectCHROMIUM(tex1, GL_TEXTURE_2D, mailbox1);
   EXPECT_EQ(static_cast<GLenum>(GL_INVALID_OPERATION), glGetError());
 }
 
 // http://crbug.com/281565
 #if !defined(OS_ANDROID)
-TEST_F(GLTextureMailboxTest, TakeFrontBufferMultipleContexts) {
+TEST_F(GLTextureMailboxTest, ProduceFrontBufferMultipleContexts) {
   gl1_.MakeCurrent();
   Mailbox mailbox[2];
   glGenMailboxCHROMIUM(mailbox[0].name);
   glGenMailboxCHROMIUM(mailbox[1].name);
   GLuint tex[2];
   glGenTextures(2, tex);
 
   GLManager::Options options;
   options.share_mailbox_manager = &gl1_;
   GLManager other_gl[2];
   for (size_t i = 0; i < 2; ++i) {
     other_gl[i].Initialize(options);
     other_gl[i].MakeCurrent();
-    glResizeCHROMIUM(10, 10, 1, true);
-    glClearColor(1 - i % 2, i % 2, 0, 1);
-    glClear(GL_COLOR_BUFFER_BIT);
-    ::gles2::GetGLContext()->SwapBuffers();
-    other_gl[i].decoder()->TakeFrontBuffer(mailbox[i]);
+    other_gl[i].decoder()->ProduceFrontBuffer(mailbox[i]);
     // Make sure both "other gl" are in the same share group.
     if (!options.share_group_manager)
       options.share_group_manager = other_gl+i;
   }
 
+
   gl1_.MakeCurrent();
   for (size_t i = 0; i < 2; ++i) {
     glBindTexture(GL_TEXTURE_2D, tex[i]);
     glConsumeTextureCHROMIUM(GL_TEXTURE_2D, mailbox[i].name);
     EXPECT_EQ(static_cast<GLenum>(GL_NO_ERROR), glGetError());
   }
 
+  for (size_t i = 0; i < 2; ++i) {
+    other_gl[i].MakeCurrent();
+    glResizeCHROMIUM(10, 10, 1, true);
+    glClearColor(1-i%2, i%2, 0, 1);
+    glClear(GL_COLOR_BUFFER_BIT);
+    ::gles2::GetGLContext()->SwapBuffers();
+  }
+
   gl1_.MakeCurrent();
   EXPECT_EQ(0xFF0000FFu, ReadTexel(tex[0], 0, 0));
   EXPECT_EQ(0xFF00FF00u, ReadTexel(tex[1], 9, 9));
 
   for (size_t i = 0; i < 2; ++i) {
     other_gl[i].MakeCurrent();
     other_gl[i].Destroy();
   }
 
   gl1_.MakeCurrent();
   glDeleteTextures(2, tex);
 }
 #endif
 
 }  // namespace gpu