Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(528)

Side by Side Diff: components/exo/buffer_unittest.cc

Issue 2083853002: exo: Recreate Surface resources on context lost. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix nits Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « components/exo/buffer.cc ('k') | components/exo/surface.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <GLES2/gl2extchromium.h> 5 #include <GLES2/gl2extchromium.h>
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "cc/output/context_provider.h" 8 #include "cc/output/context_provider.h"
9 #include "cc/resources/single_release_callback.h" 9 #include "cc/resources/single_release_callback.h"
10 #include "components/exo/buffer.h" 10 #include "components/exo/buffer.h"
(...skipping 19 matching lines...) Expand all
30 TEST_F(BufferTest, ReleaseCallback) { 30 TEST_F(BufferTest, ReleaseCallback) {
31 gfx::Size buffer_size(256, 256); 31 gfx::Size buffer_size(256, 256);
32 std::unique_ptr<Buffer> buffer( 32 std::unique_ptr<Buffer> buffer(
33 new Buffer(exo_test_helper()->CreateGpuMemoryBuffer(buffer_size))); 33 new Buffer(exo_test_helper()->CreateGpuMemoryBuffer(buffer_size)));
34 34
35 // Set the release callback. 35 // Set the release callback.
36 int release_call_count = 0; 36 int release_call_count = 0;
37 buffer->set_release_callback( 37 buffer->set_release_callback(
38 base::Bind(&Release, base::Unretained(&release_call_count))); 38 base::Bind(&Release, base::Unretained(&release_call_count)));
39 39
40 buffer->OnAttach();
40 // Produce a texture mailbox for the contents of the buffer. 41 // Produce a texture mailbox for the contents of the buffer.
41 cc::TextureMailbox texture_mailbox; 42 cc::TextureMailbox texture_mailbox;
42 std::unique_ptr<cc::SingleReleaseCallback> buffer_release_callback = 43 std::unique_ptr<cc::SingleReleaseCallback> buffer_release_callback =
43 buffer->ProduceTextureMailbox(&texture_mailbox, false, true); 44 buffer->ProduceTextureMailbox(&texture_mailbox, false, true);
44 ASSERT_TRUE(buffer_release_callback); 45 ASSERT_TRUE(buffer_release_callback);
45 46
46 // Release buffer. 47 // Release buffer.
47 buffer_release_callback->Run(gpu::SyncToken(), false); 48 buffer_release_callback->Run(gpu::SyncToken(), false);
48 49
50 ASSERT_EQ(release_call_count, 0);
51
52 buffer->OnDetach();
53
49 // Release() should have been called exactly once. 54 // Release() should have been called exactly once.
50 ASSERT_EQ(release_call_count, 1); 55 ASSERT_EQ(release_call_count, 1);
51 } 56 }
52 57
53 TEST_F(BufferTest, IsLost) { 58 TEST_F(BufferTest, IsLost) {
54 gfx::Size buffer_size(256, 256); 59 gfx::Size buffer_size(256, 256);
55 std::unique_ptr<Buffer> buffer( 60 std::unique_ptr<Buffer> buffer(
56 new Buffer(exo_test_helper()->CreateGpuMemoryBuffer(buffer_size))); 61 new Buffer(exo_test_helper()->CreateGpuMemoryBuffer(buffer_size)));
57 62
63 buffer->OnAttach();
58 // Acquire a texture mailbox for the contents of the buffer. 64 // Acquire a texture mailbox for the contents of the buffer.
59 cc::TextureMailbox texture_mailbox; 65 cc::TextureMailbox texture_mailbox;
60 std::unique_ptr<cc::SingleReleaseCallback> buffer_release_callback = 66 std::unique_ptr<cc::SingleReleaseCallback> buffer_release_callback =
61 buffer->ProduceTextureMailbox(&texture_mailbox, false, true); 67 buffer->ProduceTextureMailbox(&texture_mailbox, false, true);
62 ASSERT_TRUE(buffer_release_callback); 68 ASSERT_TRUE(buffer_release_callback);
63 69
64 scoped_refptr<cc::ContextProvider> context_provider = 70 scoped_refptr<cc::ContextProvider> context_provider =
65 aura::Env::GetInstance() 71 aura::Env::GetInstance()
66 ->context_factory() 72 ->context_factory()
67 ->SharedMainThreadContextProvider(); 73 ->SharedMainThreadContextProvider();
68 if (context_provider) { 74 if (context_provider) {
69 gpu::gles2::GLES2Interface* gles2 = context_provider->ContextGL(); 75 gpu::gles2::GLES2Interface* gles2 = context_provider->ContextGL();
70 gles2->LoseContextCHROMIUM(GL_GUILTY_CONTEXT_RESET_ARB, 76 gles2->LoseContextCHROMIUM(GL_GUILTY_CONTEXT_RESET_ARB,
71 GL_INNOCENT_CONTEXT_RESET_ARB); 77 GL_INNOCENT_CONTEXT_RESET_ARB);
72 } 78 }
73 79
80 // Release buffer.
81 bool is_lost = true;
82 buffer_release_callback->Run(gpu::SyncToken(), is_lost);
83
74 // Producing a new texture mailbox for the contents of the buffer. 84 // Producing a new texture mailbox for the contents of the buffer.
75 std::unique_ptr<cc::SingleReleaseCallback> buffer_release_callback2 = 85 std::unique_ptr<cc::SingleReleaseCallback> buffer_release_callback2 =
76 buffer->ProduceTextureMailbox(&texture_mailbox, false, false); 86 buffer->ProduceTextureMailbox(&texture_mailbox, false, false);
77 ASSERT_TRUE(buffer_release_callback); 87 ASSERT_TRUE(buffer_release_callback2);
88 buffer->OnDetach();
78 89
79 // Release buffer.
80 bool is_lost = true;
81 buffer_release_callback->Run(gpu::SyncToken(), is_lost);
82 buffer_release_callback2->Run(gpu::SyncToken(), false); 90 buffer_release_callback2->Run(gpu::SyncToken(), false);
83 } 91 }
84 92
85 } // namespace 93 } // namespace
86 } // namespace exo 94 } // namespace exo
OLDNEW
« no previous file with comments | « components/exo/buffer.cc ('k') | components/exo/surface.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698