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

Side by Side Diff: src/gpu/gl/GrGLBufferImpl.cpp

Issue 1490473003: Add transfer buffer support. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Some clean up Created 5 years 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
OLDNEW
1 /* 1 /*
2 * Copyright 2013 Google Inc. 2 * Copyright 2013 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #include "GrGLBufferImpl.h" 8 #include "GrGLBufferImpl.h"
9 #include "GrGLGpu.h" 9 #include "GrGLGpu.h"
10 10
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
58 fCPUData = nullptr; 58 fCPUData = nullptr;
59 VALIDATE(); 59 VALIDATE();
60 } 60 }
61 61
62 void* GrGLBufferImpl::map(GrGLGpu* gpu) { 62 void* GrGLBufferImpl::map(GrGLGpu* gpu) {
63 VALIDATE(); 63 VALIDATE();
64 SkASSERT(!this->isMapped()); 64 SkASSERT(!this->isMapped());
65 if (0 == fDesc.fID) { 65 if (0 == fDesc.fID) {
66 fMapPtr = fCPUData; 66 fMapPtr = fCPUData;
67 } else { 67 } else {
68 fMapPtr = gpu->mapBuffer(fDesc.fID, fBufferType, fDesc.fDynamic, fGLSize InBytes, 68 fMapPtr = gpu->mapBuffer(fDesc.fID, fBufferType, fDesc.fUsage, fGLSizeIn Bytes,
69 fDesc.fSizeInBytes); 69 fDesc.fSizeInBytes);
70 fGLSizeInBytes = fDesc.fSizeInBytes; 70 fGLSizeInBytes = fDesc.fSizeInBytes;
71 } 71 }
72 VALIDATE(); 72 VALIDATE();
73 return fMapPtr; 73 return fMapPtr;
74 } 74 }
75 75
76 void GrGLBufferImpl::unmap(GrGLGpu* gpu) { 76 void GrGLBufferImpl::unmap(GrGLGpu* gpu) {
77 VALIDATE(); 77 VALIDATE();
78 SkASSERT(this->isMapped()); 78 SkASSERT(this->isMapped());
(...skipping 11 matching lines...) Expand all
90 bool GrGLBufferImpl::updateData(GrGLGpu* gpu, const void* src, size_t srcSizeInB ytes) { 90 bool GrGLBufferImpl::updateData(GrGLGpu* gpu, const void* src, size_t srcSizeInB ytes) {
91 SkASSERT(!this->isMapped()); 91 SkASSERT(!this->isMapped());
92 VALIDATE(); 92 VALIDATE();
93 if (srcSizeInBytes > fDesc.fSizeInBytes) { 93 if (srcSizeInBytes > fDesc.fSizeInBytes) {
94 return false; 94 return false;
95 } 95 }
96 if (0 == fDesc.fID) { 96 if (0 == fDesc.fID) {
97 memcpy(fCPUData, src, srcSizeInBytes); 97 memcpy(fCPUData, src, srcSizeInBytes);
98 return true; 98 return true;
99 } 99 }
100 gpu->bufferData(fDesc.fID, fBufferType, fDesc.fDynamic, fDesc.fSizeInBytes, src, 100 gpu->bufferData(fDesc.fID, fBufferType, fDesc.fUsage, fDesc.fSizeInBytes, sr c,
101 srcSizeInBytes); 101 srcSizeInBytes);
102 #if GR_GL_USE_BUFFER_DATA_NULL_HINT 102 #if GR_GL_USE_BUFFER_DATA_NULL_HINT
103 fGLSizeInBytes = fDesc.fSizeInBytes; 103 fGLSizeInBytes = fDesc.fSizeInBytes;
104 #else 104 #else
105 fGLSizeInBytes = srcSizeInBytes; 105 fGLSizeInBytes = srcSizeInBytes;
106 #endif 106 #endif
107 VALIDATE(); 107 VALIDATE();
108 return true; 108 return true;
109 } 109 }
110 110
111 void GrGLBufferImpl::validate() const { 111 void GrGLBufferImpl::validate() const {
112 SkASSERT(GR_GL_ARRAY_BUFFER == fBufferType || GR_GL_ELEMENT_ARRAY_BUFFER == fBufferType); 112 SkASSERT(GR_GL_ARRAY_BUFFER == fBufferType || GR_GL_ELEMENT_ARRAY_BUFFER == fBufferType ||
113 GR_GL_PIXEL_PACK_BUFFER == fBufferType || GR_GL_PIXEL_UNPACK_BUFFER == fBufferType);
113 // The following assert isn't valid when the buffer has been abandoned: 114 // The following assert isn't valid when the buffer has been abandoned:
114 // SkASSERT((0 == fDesc.fID) == (fCPUData)); 115 // SkASSERT((0 == fDesc.fID) == (fCPUData));
115 SkASSERT(nullptr == fCPUData || 0 == fGLSizeInBytes); 116 SkASSERT(nullptr == fCPUData || 0 == fGLSizeInBytes);
116 SkASSERT(nullptr == fMapPtr || fCPUData || fGLSizeInBytes <= fDesc.fSizeInBy tes); 117 SkASSERT(nullptr == fMapPtr || fCPUData || fGLSizeInBytes <= fDesc.fSizeInBy tes);
117 SkASSERT(nullptr == fCPUData || nullptr == fMapPtr || fCPUData == fMapPtr); 118 SkASSERT(nullptr == fCPUData || nullptr == fMapPtr || fCPUData == fMapPtr);
118 } 119 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698