Rename buffer_source_type to buffer_source (#31426)

* Rename buffer_source_type to buffer_source

Signed-off-by: Bentaimia Haddadi <haddadi.taym@gmail.com>

* Code format

Signed-off-by: Bentaimia Haddadi <haddadi.taym@gmail.com>

---------

Signed-off-by: Bentaimia Haddadi <haddadi.taym@gmail.com>
This commit is contained in:
Taym Haddadi 2024-02-26 15:21:46 +01:00 committed by GitHub
parent ef8a0b7f7b
commit a9a7e8a5cf
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
6 changed files with 22 additions and 23 deletions

View file

@ -67,14 +67,14 @@ unsafe impl<T> crate::dom::bindings::trace::JSTraceable for HeapBufferSource<T>
} }
} }
pub fn new_initialized_heap_buffer_source_types<T>( pub fn new_initialized_heap_buffer_source<T>(
init: HeapTypedArrayInit, init: HeapTypedArrayInit,
) -> Result<HeapBufferSource<T>, ()> ) -> Result<HeapBufferSource<T>, ()>
where where
T: TypedArrayElement + TypedArrayElementCreator, T: TypedArrayElement + TypedArrayElementCreator,
T::Element: Clone + Copy, T::Element: Clone + Copy,
{ {
let heap_buffer_source_types = match init { let heap_buffer_source = match init {
HeapTypedArrayInit::Buffer(buffer_source) => HeapBufferSource { HeapTypedArrayInit::Buffer(buffer_source) => HeapBufferSource {
buffer_source: buffer_source, buffer_source: buffer_source,
phantom: PhantomData::default(), phantom: PhantomData::default(),
@ -82,13 +82,13 @@ where
HeapTypedArrayInit::Info { len, cx } => { HeapTypedArrayInit::Info { len, cx } => {
rooted!(in (*cx) let mut array = ptr::null_mut::<JSObject>()); rooted!(in (*cx) let mut array = ptr::null_mut::<JSObject>());
let typed_array_result = let typed_array_result =
create_buffer_source_types_with_length::<T>(cx, len as usize, array.handle_mut()); create_buffer_source_with_length::<T>(cx, len as usize, array.handle_mut());
if typed_array_result.is_err() { if typed_array_result.is_err() {
return Err(()); return Err(());
} }
let heap_buffer_source_types = HeapBufferSource::<T>::default(); let heap_buffer_source = HeapBufferSource::<T>::default();
match &heap_buffer_source_types.buffer_source { match &heap_buffer_source.buffer_source {
BufferSource::Int8Array(buffer) | BufferSource::Int8Array(buffer) |
BufferSource::Int16Array(buffer) | BufferSource::Int16Array(buffer) |
BufferSource::Int32Array(buffer) | BufferSource::Int32Array(buffer) |
@ -106,10 +106,10 @@ where
buffer.set(*array); buffer.set(*array);
}, },
} }
heap_buffer_source_types heap_buffer_source
}, },
}; };
Ok(heap_buffer_source_types) Ok(heap_buffer_source)
} }
pub enum HeapTypedArrayInit { pub enum HeapTypedArrayInit {
@ -131,8 +131,7 @@ where
pub fn set_data(&self, cx: JSContext, data: &[T::Element]) -> Result<(), ()> { pub fn set_data(&self, cx: JSContext, data: &[T::Element]) -> Result<(), ()> {
rooted!(in (*cx) let mut array = ptr::null_mut::<JSObject>()); rooted!(in (*cx) let mut array = ptr::null_mut::<JSObject>());
let _: TypedArray<T, *mut JSObject> = let _: TypedArray<T, *mut JSObject> = create_buffer_source(cx, data, array.handle_mut())?;
create_buffer_source_types(cx, data, array.handle_mut())?;
match &self.buffer_source { match &self.buffer_source {
BufferSource::Int8Array(buffer) | BufferSource::Int8Array(buffer) |
@ -367,7 +366,7 @@ where
} }
/// <https://webidl.spec.whatwg.org/#arraybufferview-create> /// <https://webidl.spec.whatwg.org/#arraybufferview-create>
pub fn create_buffer_source_types<T>( pub fn create_buffer_source<T>(
cx: JSContext, cx: JSContext,
data: &[T::Element], data: &[T::Element],
dest: MutableHandleObject, dest: MutableHandleObject,
@ -384,7 +383,7 @@ where
} }
} }
fn create_buffer_source_types_with_length<T>( fn create_buffer_source_with_length<T>(
cx: JSContext, cx: JSContext,
len: usize, len: usize,
dest: MutableHandleObject, dest: MutableHandleObject,

View file

@ -15,7 +15,7 @@ use js::typedarray::{Float32Array, Float64Array};
use style::parser::ParserContext; use style::parser::ParserContext;
use url::Url; use url::Url;
use crate::dom::bindings::buffer_source::create_buffer_source_types; use crate::dom::bindings::buffer_source::create_buffer_source;
use crate::dom::bindings::cell::{DomRefCell, Ref}; use crate::dom::bindings::cell::{DomRefCell, Ref};
use crate::dom::bindings::codegen::Bindings::DOMMatrixBinding::{DOMMatrixInit, DOMMatrixMethods}; use crate::dom::bindings::codegen::Bindings::DOMMatrixBinding::{DOMMatrixInit, DOMMatrixMethods};
use crate::dom::bindings::codegen::Bindings::DOMMatrixReadOnlyBinding::DOMMatrixReadOnlyMethods; use crate::dom::bindings::codegen::Bindings::DOMMatrixReadOnlyBinding::DOMMatrixReadOnlyMethods;
@ -686,14 +686,14 @@ impl DOMMatrixReadOnlyMethods for DOMMatrixReadOnly {
.map(|&x| x as f32) .map(|&x| x as f32)
.collect(); .collect();
rooted!(in (*cx) let mut array = ptr::null_mut::<JSObject>()); rooted!(in (*cx) let mut array = ptr::null_mut::<JSObject>());
create_buffer_source_types(cx, &vec, array.handle_mut()) create_buffer_source(cx, &vec, array.handle_mut())
.expect("Converting matrix to float32 array should never fail") .expect("Converting matrix to float32 array should never fail")
} }
// https://drafts.fxtf.org/geometry-1/#dom-dommatrixreadonly-tofloat64array // https://drafts.fxtf.org/geometry-1/#dom-dommatrixreadonly-tofloat64array
fn ToFloat64Array(&self, cx: JSContext) -> Float64Array { fn ToFloat64Array(&self, cx: JSContext) -> Float64Array {
rooted!(in (*cx) let mut array = ptr::null_mut::<JSObject>()); rooted!(in (*cx) let mut array = ptr::null_mut::<JSObject>());
create_buffer_source_types(cx, &self.matrix.borrow().to_array(), array.handle_mut()) create_buffer_source(cx, &self.matrix.borrow().to_array(), array.handle_mut())
.expect("Converting matrix to float64 array should never fail") .expect("Converting matrix to float64 array should never fail")
} }
} }

View file

@ -9,7 +9,7 @@ use js::jsapi::JSObject;
use js::rust::HandleObject; use js::rust::HandleObject;
use js::typedarray::{ArrayBuffer, ArrayBufferU8}; use js::typedarray::{ArrayBuffer, ArrayBufferU8};
use crate::dom::bindings::buffer_source::create_buffer_source_types; use crate::dom::bindings::buffer_source::create_buffer_source;
use crate::dom::bindings::codegen::Bindings::BlobBinding::BlobMethods; use crate::dom::bindings::codegen::Bindings::BlobBinding::BlobMethods;
use crate::dom::bindings::codegen::Bindings::FileReaderSyncBinding::FileReaderSyncMethods; use crate::dom::bindings::codegen::Bindings::FileReaderSyncBinding::FileReaderSyncMethods;
use crate::dom::bindings::error::{Error, Fallible}; use crate::dom::bindings::error::{Error, Fallible};
@ -95,7 +95,7 @@ impl FileReaderSyncMethods for FileReaderSync {
// step 2 // step 2
rooted!(in(*cx) let mut array_buffer = ptr::null_mut::<JSObject>()); rooted!(in(*cx) let mut array_buffer = ptr::null_mut::<JSObject>());
create_buffer_source_types::<ArrayBufferU8>(cx, &blob_contents, array_buffer.handle_mut()) create_buffer_source::<ArrayBufferU8>(cx, &blob_contents, array_buffer.handle_mut())
.map_err(|_| Error::JSFailed) .map_err(|_| Error::JSFailed)
} }
} }

View file

@ -14,7 +14,7 @@ use js::rust::HandleObject;
use js::typedarray::{ClampedU8, CreateWith, Uint8ClampedArray}; use js::typedarray::{ClampedU8, CreateWith, Uint8ClampedArray};
use super::bindings::buffer_source::{ use super::bindings::buffer_source::{
new_initialized_heap_buffer_source_types, BufferSource, HeapBufferSource, HeapTypedArrayInit, new_initialized_heap_buffer_source, BufferSource, HeapBufferSource, HeapTypedArrayInit,
}; };
use crate::dom::bindings::codegen::Bindings::CanvasRenderingContext2DBinding::ImageDataMethods; use crate::dom::bindings::codegen::Bindings::CanvasRenderingContext2DBinding::ImageDataMethods;
use crate::dom::bindings::error::{Error, Fallible}; use crate::dom::bindings::error::{Error, Fallible};
@ -63,7 +63,7 @@ impl ImageData {
opt_height: Option<u32>, opt_height: Option<u32>,
jsobject: *mut JSObject, jsobject: *mut JSObject,
) -> Fallible<DomRoot<ImageData>> { ) -> Fallible<DomRoot<ImageData>> {
let heap_typed_array = match new_initialized_heap_buffer_source_types::<ClampedU8>( let heap_typed_array = match new_initialized_heap_buffer_source::<ClampedU8>(
HeapTypedArrayInit::Buffer(BufferSource::Uint8ClampedArray(Heap::boxed(jsobject))), HeapTypedArrayInit::Buffer(BufferSource::Uint8ClampedArray(Heap::boxed(jsobject))),
) { ) {
Ok(heap_typed_array) => heap_typed_array, Ok(heap_typed_array) => heap_typed_array,
@ -117,7 +117,7 @@ impl ImageData {
let len = width * height * 4; let len = width * height * 4;
let heap_typed_array = let heap_typed_array =
match new_initialized_heap_buffer_source_types::<ClampedU8>(HeapTypedArrayInit::Info { match new_initialized_heap_buffer_source::<ClampedU8>(HeapTypedArrayInit::Info {
len, len,
cx, cx,
}) { }) {

View file

@ -17,7 +17,7 @@ use script_traits::serializable::BlobImpl;
use script_traits::MsDuration; use script_traits::MsDuration;
use servo_config::prefs; use servo_config::prefs;
use crate::dom::bindings::buffer_source::create_buffer_source_types; use crate::dom::bindings::buffer_source::create_buffer_source;
use crate::dom::bindings::callback::ExceptionHandling; use crate::dom::bindings::callback::ExceptionHandling;
use crate::dom::bindings::codegen::Bindings::EventListenerBinding::EventListener; use crate::dom::bindings::codegen::Bindings::EventListenerBinding::EventListener;
use crate::dom::bindings::codegen::Bindings::FunctionBinding::Function; use crate::dom::bindings::codegen::Bindings::FunctionBinding::Function;
@ -214,7 +214,7 @@ impl TestBindingMethods for TestBinding {
let data: [u8; 16] = [0; 16]; let data: [u8; 16] = [0; 16];
rooted!(in (*cx) let mut array = ptr::null_mut::<JSObject>()); rooted!(in (*cx) let mut array = ptr::null_mut::<JSObject>());
create_buffer_source_types(cx, &data, array.handle_mut()) create_buffer_source(cx, &data, array.handle_mut())
.expect("Creating ClampedU8 array should never fail") .expect("Creating ClampedU8 array should never fail")
} }
fn AnyAttribute(&self, _: SafeJSContext) -> JSVal { fn AnyAttribute(&self, _: SafeJSContext) -> JSVal {

View file

@ -9,7 +9,7 @@ use js::jsapi::JSObject;
use js::rust::HandleObject; use js::rust::HandleObject;
use js::typedarray::Uint8Array; use js::typedarray::Uint8Array;
use crate::dom::bindings::buffer_source::create_buffer_source_types; use crate::dom::bindings::buffer_source::create_buffer_source;
use crate::dom::bindings::codegen::Bindings::TextEncoderBinding::TextEncoderMethods; use crate::dom::bindings::codegen::Bindings::TextEncoderBinding::TextEncoderMethods;
use crate::dom::bindings::error::Fallible; use crate::dom::bindings::error::Fallible;
use crate::dom::bindings::reflector::{reflect_dom_object_with_proto, Reflector}; use crate::dom::bindings::reflector::{reflect_dom_object_with_proto, Reflector};
@ -55,7 +55,7 @@ impl TextEncoderMethods for TextEncoder {
let encoded = input.0.as_bytes(); let encoded = input.0.as_bytes();
rooted!(in(*cx) let mut js_object = ptr::null_mut::<JSObject>()); rooted!(in(*cx) let mut js_object = ptr::null_mut::<JSObject>());
create_buffer_source_types(cx, &encoded, js_object.handle_mut()) create_buffer_source(cx, &encoded, js_object.handle_mut())
.expect("Converting input to uint8 array should never fail") .expect("Converting input to uint8 array should never fail")
} }
} }