store map ptr in GrGeometryBuffer base class

src/gpu/gl/GrGLVertexBuffer.cpp

Index: src/gpu/gl/GrGLVertexBuffer.cpp
diff --git a/src/gpu/gl/GrGLVertexBuffer.cpp b/src/gpu/gl/GrGLVertexBuffer.cpp
index af6099353a5dc4bb53a9f107e3eefb1fbbabbf64..6b5b1765d218ee15cbaf38a87d78e44fe5b96724 100644
--- a/src/gpu/gl/GrGLVertexBuffer.cpp
+++ b/src/gpu/gl/GrGLVertexBuffer.cpp
@@ -21,13 +21,12 @@ void GrGLVertexBuffer::onRelease() {
     INHERITED::onRelease();
 }
 
-
 void GrGLVertexBuffer::onAbandon() {
     fImpl.abandon();
     INHERITED::onAbandon();
 }
 
-void* GrGLVertexBuffer::map() {
+void* GrGLVertexBuffer::onMap() {
     if (!this->wasDestroyed()) {
         return fImpl.map(this->getGpuGL());
     } else {
@@ -35,21 +34,13 @@ void* GrGLVertexBuffer::map() {
     }
 }
 
-void* GrGLVertexBuffer::mapPtr() const {
-    return fImpl.mapPtr();
-}
-
-void GrGLVertexBuffer::unmap() {
+void GrGLVertexBuffer::onUnmap() {
     if (!this->wasDestroyed()) {

[robertphillips, 2014/05/08 15:25:46]

Do we need this return?

[bsalomon, 2014/05/08 17:02:48]

nope, i had this function as a bool and then decided against it.

-        fImpl.unmap(this->getGpuGL());
+        return fImpl.unmap(this->getGpuGL());
     }
 }
 
-bool GrGLVertexBuffer::isMapped() const {
-    return fImpl.isMapped();
-}
-
-bool GrGLVertexBuffer::updateData(const void* src, size_t srcSizeInBytes) {
+bool GrGLVertexBuffer::onUpdateData(const void* src, size_t srcSizeInBytes) {
     if (!this->wasDestroyed()) {
         return fImpl.updateData(this->getGpuGL(), src, srcSizeInBytes);
     } else {