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

Side by Side Diff: gpu/command_buffer/client/gpu_memory_buffer_tracker.cc

Issue 20017005: gpu: Refactor GpuMemoryBuffer framework for multi-process support. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Include proper internalformat support.[D Created 7 years, 4 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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 "gpu/command_buffer/client/gpu_memory_buffer_tracker.h" 5 #include "gpu/command_buffer/client/gpu_memory_buffer_tracker.h"
6 6
7 #include "base/memory/scoped_ptr.h" 7 #include "base/memory/scoped_ptr.h"
8 #include "gpu/command_buffer/client/cmd_buffer_helper.h"
8 #include "gpu/command_buffer/client/gles2_implementation.h" 9 #include "gpu/command_buffer/client/gles2_implementation.h"
9 #include "gpu/command_buffer/client/image_factory.h"
10 #include "ui/gfx/gpu_memory_buffer.h"
11 10
12 namespace gpu { 11 namespace gpu {
13 namespace gles2 { 12 namespace gles2 {
14 13
15 GpuMemoryBufferTracker::GpuMemoryBufferTracker(ImageFactory* factory) 14 GpuMemoryBufferTracker::GpuMemoryBufferTracker(CommandBufferHelper* helper)
16 : buffers_(), 15 : helper_(helper) {
17 factory_(factory) { 16 DCHECK(helper_);
18 } 17 }
19 18
20 GpuMemoryBufferTracker::~GpuMemoryBufferTracker() { 19 GpuMemoryBufferTracker::~GpuMemoryBufferTracker() {
21 while (!buffers_.empty()) { 20 while (!buffers_.empty()) {
22 RemoveBuffer(buffers_.begin()->first); 21 RemoveBuffer(buffers_.begin()->first);
23 } 22 }
24 } 23 }
25 24
26 GLuint GpuMemoryBufferTracker::CreateBuffer( 25 int32 GpuMemoryBufferTracker::CreateBuffer(
27 GLsizei width, GLsizei height, GLenum internalformat) { 26 size_t width, size_t height, int32 internalformat) {
28 GLuint image_id = 0; 27 int32 image_id = 0;
29 DCHECK(factory_); 28 gfx::GpuMemoryBuffer* buffer =
30 scoped_ptr<gfx::GpuMemoryBuffer> buffer = 29 helper_->command_buffer()->CreateGpuMemoryBuffer(
31 factory_->CreateGpuMemoryBuffer(width, height, internalformat, &image_id); 30 width, height, internalformat, &image_id);
32 31 if (!buffer)
33 if (buffer.get() == NULL)
34 return 0; 32 return 0;
35 33
36 std::pair<BufferMap::iterator, bool> result = 34 std::pair<BufferMap::iterator, bool> result =
37 buffers_.insert(std::make_pair(image_id, buffer.release())); 35 buffers_.insert(std::make_pair(image_id, buffer));
38 GPU_DCHECK(result.second); 36 GPU_DCHECK(result.second);
39 37
40 return image_id; 38 return image_id;
41 } 39 }
42 40
43 gfx::GpuMemoryBuffer* GpuMemoryBufferTracker::GetBuffer(GLuint image_id) { 41 gfx::GpuMemoryBuffer* GpuMemoryBufferTracker::GetBuffer(int32 image_id) {
44 BufferMap::iterator it = buffers_.find(image_id); 42 BufferMap::iterator it = buffers_.find(image_id);
45 return (it != buffers_.end()) ? it->second : NULL; 43 return (it != buffers_.end()) ? it->second : NULL;
46 } 44 }
47 45
48 void GpuMemoryBufferTracker::RemoveBuffer(GLuint image_id) { 46 void GpuMemoryBufferTracker::RemoveBuffer(int32 image_id) {
49 BufferMap::iterator buffer_it = buffers_.find(image_id); 47 BufferMap::iterator buffer_it = buffers_.find(image_id);
50 if (buffer_it != buffers_.end()) { 48 if (buffer_it != buffers_.end())
51 gfx::GpuMemoryBuffer* buffer = buffer_it->second;
52 buffers_.erase(buffer_it); 49 buffers_.erase(buffer_it);
53 delete buffer; 50 helper_->command_buffer()->DestroyGpuMemoryBuffer(image_id);
54 }
55 DCHECK(factory_);
56 factory_->DeleteGpuMemoryBuffer(image_id);
57 } 51 }
58 52
59 } // namespace gles2 53 } // namespace gles2
60 } // namespace gpu 54 } // namespace gpu
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698