fixup! fix(core): use BufferMapState::Active for any BufferUsages::MAP_* flags

Author: ErichDonGubler

wgpu-hal/src/gles/device.rs

@@ -526,7 +526,10 @@ impl crate::Device for super::Device {
                 .private_caps
                 .contains(PrivateCapabilities::BUFFER_ALLOCATION);
 
-        if emulate_map && desc.usage.intersects(wgt::BufferUses::MAP_WRITE) {
+        if emulate_map
+            && (desc.usage.intersects(wgt::BufferUses::MAP_WRITE)
+                && !desc.usage.intersects(wgt::BufferUses::MAP_READ))
+        {
             return Ok(super::Buffer {
                 raw: None,
                 target,
@@ -692,12 +695,21 @@ impl crate::Device for super::Device {
     }
     unsafe fn unmap_buffer(&self, buffer: &super::Buffer) {
         if let Some(raw) = buffer.raw {
-            if buffer.data.is_none() {
-                let gl = &self.shared.context.lock();
-                unsafe { gl.bind_buffer(buffer.target, Some(raw)) };
-                unsafe { gl.unmap_buffer(buffer.target) };
-                unsafe { gl.bind_buffer(buffer.target, None) };
-                *lock(&buffer.offset_of_current_mapping) = 0;
+            match &buffer.data {
+                None => {
+                    let gl = &self.shared.context.lock();
+                    unsafe { gl.bind_buffer(buffer.target, Some(raw)) };
+                    unsafe { gl.unmap_buffer(buffer.target) };
+                    unsafe { gl.bind_buffer(buffer.target, None) };
+                    *lock(&buffer.offset_of_current_mapping) = 0;
+                }
+                Some(data) => {
+                    let gl = &self.shared.context.lock();
+                    let data = lock(&**data);
+                    unsafe { gl.bind_buffer(buffer.target, Some(raw)) };
+                    unsafe { gl.buffer_sub_data_u8_slice(buffer.target, 0, &data) };
+                    unsafe { gl.bind_buffer(buffer.target, None) };
+                }
             }
         }
     }