mirror of
https://github.com/servo/servo.git
synced 2025-08-02 20:20:14 +01:00
Change old references of ecoal95
This commit is contained in:
parent
8e61a8a974
commit
5ed93a5c4a
6 changed files with 10 additions and 11 deletions
|
@ -81,7 +81,7 @@ impl WebGLPaintThread {
|
|||
CanvasMsg::Common(message) => {
|
||||
match message {
|
||||
CanvasCommonMsg::Close => break,
|
||||
// TODO(ecoal95): handle error nicely
|
||||
// TODO(emilio): handle error nicely
|
||||
CanvasCommonMsg::Recreate(size) => painter.recreate(size).unwrap(),
|
||||
}
|
||||
},
|
||||
|
|
|
@ -206,7 +206,7 @@ impl HTMLCanvasElement {
|
|||
CanvasData::Pixels(pixel_data)
|
||||
=> pixel_data.image_data.to_vec(),
|
||||
CanvasData::WebGL(_)
|
||||
// TODO(ecoal95): Not sure if WebGL canvas is required for 2d spec,
|
||||
// TODO(emilio): Not sure if WebGL canvas is required for 2d spec,
|
||||
// but I think it's not.
|
||||
=> return None,
|
||||
}
|
||||
|
|
|
@ -96,7 +96,7 @@ impl WebGLProgram {
|
|||
_ => return Err(WebGLError::InvalidOperation),
|
||||
};
|
||||
|
||||
// TODO(ecoal95): Differentiate between same shader already assigned and other previous
|
||||
// TODO(emilio): Differentiate between same shader already assigned and other previous
|
||||
// shader.
|
||||
if shader_slot.get().is_some() {
|
||||
return Err(WebGLError::InvalidOperation);
|
||||
|
|
|
@ -610,7 +610,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext {
|
|||
}
|
||||
}
|
||||
|
||||
// TODO(ecoal95): Probably in the future we should keep track of the
|
||||
// TODO(emilio): Probably in the future we should keep track of the
|
||||
// generated objects, either here or in the webgl thread
|
||||
// https://www.khronos.org/registry/webgl/specs/latest/1.0/#5.14.5
|
||||
fn CreateBuffer(&self) -> Option<Root<WebGLBuffer>> {
|
||||
|
@ -1116,8 +1116,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext {
|
|||
if texture.is_none() {
|
||||
return self.webgl_error(InvalidOperation);
|
||||
}
|
||||
// TODO(ecoal95): Validate more parameters
|
||||
|
||||
// TODO(emilio): Validate more parameters
|
||||
let source = match source {
|
||||
Some(s) => s,
|
||||
None => return,
|
||||
|
@ -1144,7 +1143,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext {
|
|||
};
|
||||
|
||||
let size = Size2D::new(img.width as i32, img.height as i32);
|
||||
// TODO(ecoal95): Validate that the format argument is coherent with the image.
|
||||
// TODO(emilio): Validate that the format argument is coherent with the image.
|
||||
// RGB8 should be easy to support too
|
||||
let mut data = match img.format {
|
||||
PixelFormat::RGBA8 => img.bytes.to_vec(),
|
||||
|
@ -1155,7 +1154,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext {
|
|||
|
||||
(data, size)
|
||||
},
|
||||
// TODO(ecoal95): Getting canvas data is implemented in CanvasRenderingContext2D, but
|
||||
// TODO(emilio): Getting canvas data is implemented in CanvasRenderingContext2D, but
|
||||
// we need to refactor it moving it to `HTMLCanvasElement` and supporting WebGLContext
|
||||
ImageDataOrHTMLImageElementOrHTMLCanvasElementOrHTMLVideoElement::HTMLCanvasElement(canvas) => {
|
||||
let canvas = canvas.r();
|
||||
|
@ -1170,7 +1169,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext {
|
|||
=> unimplemented!(),
|
||||
};
|
||||
|
||||
// TODO(ecoal95): Invert axis, convert colorspace, premultiply alpha if requested
|
||||
// TODO(emilio): Invert axis, convert colorspace, premultiply alpha if requested
|
||||
let msg = CanvasWebGLMsg::TexImage2D(target, level, internal_format as i32,
|
||||
size.width, size.height,
|
||||
format, data_type, pixels);
|
||||
|
|
|
@ -266,7 +266,7 @@ impl<Impl: SelectorImplExt> Stylist<Impl> {
|
|||
let map = match pseudo_element {
|
||||
Some(ref pseudo) => match self.pseudos_map.get(pseudo) {
|
||||
Some(map) => map,
|
||||
// TODO(ecoal95): get non eagerly-cascaded pseudo-element rules here.
|
||||
// TODO(emilio): get non eagerly-cascaded pseudo-element rules here.
|
||||
// Actually assume there are no rules applicable.
|
||||
None => return true,
|
||||
},
|
||||
|
|
|
@ -1,4 +1,4 @@
|
|||
@ecoal95
|
||||
@emilio
|
||||
@hallvors
|
||||
@kangxu
|
||||
@caitp
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue