aboutsummaryrefslogtreecommitdiffstats
path: root/components/script/dom/webglrenderingcontext.rs
diff options
context:
space:
mode:
authorbors-servo <lbergstrom+bors@mozilla.com>2019-10-07 18:39:36 -0400
committerGitHub <noreply@github.com>2019-10-07 18:39:36 -0400
commit75548f40c622f8bc2b3ec6b212fd4b378fcdc22f (patch)
treeb4843908c3ef75018418760a317a7289489bd831 /components/script/dom/webglrenderingcontext.rs
parent95f65cdd3139651ac267494f47cc3ed1f2df9fae (diff)
parent6dd40962ea707dc0e02676fccd9156efa314141d (diff)
downloadservo-75548f40c622f8bc2b3ec6b212fd4b378fcdc22f.tar.gz
servo-75548f40c622f8bc2b3ec6b212fd4b378fcdc22f.zip
Auto merge of #24340 - jdm:image-cache-cors, r=Manishearth
Allow using CORS filtered image responses as WebGL textures More specifically, this makes the "is this image same origin?" check consider the CORS status of the original response, rather than relying on an overly-strict "is this image's response's URL same-origin with a particular global?" check. To do this, we make the image cache double keyed based on the requested URL as well as the requesting origin, and store the CORS status of the eventual response with the final image that eventually gets sent to the HTMLImageElement consumer. --- - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes fix #24330 and fix #24368 - [x] There are tests for these changes <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/24340) <!-- Reviewable:end -->
Diffstat (limited to 'components/script/dom/webglrenderingcontext.rs')
-rw-r--r--components/script/dom/webglrenderingcontext.rs17
1 files changed, 10 insertions, 7 deletions
diff --git a/components/script/dom/webglrenderingcontext.rs b/components/script/dom/webglrenderingcontext.rs
index f8350759dd8..d6d46eee0ef 100644
--- a/components/script/dom/webglrenderingcontext.rs
+++ b/components/script/dom/webglrenderingcontext.rs
@@ -19,6 +19,7 @@ use crate::dom::bindings::inheritance::Castable;
use crate::dom::bindings::reflector::{reflect_dom_object, DomObject, Reflector};
use crate::dom::bindings::root::{Dom, DomOnceCell, DomRoot, LayoutDom, MutNullableDom};
use crate::dom::bindings::str::DOMString;
+use crate::dom::element::cors_setting_for_element;
use crate::dom::event::{Event, EventBubbles, EventCancelable};
use crate::dom::htmlcanvaselement::utils as canvas_utils;
use crate::dom::htmlcanvaselement::HTMLCanvasElement;
@@ -576,13 +577,15 @@ impl WebGLRenderingContext {
};
let window = window_from_node(&*self.canvas);
-
- let img = match canvas_utils::request_image_from_cache(&window, img_url) {
- ImageResponse::Loaded(img, _) => img,
- ImageResponse::PlaceholderLoaded(_, _) |
- ImageResponse::None |
- ImageResponse::MetadataLoaded(_) => return Ok(None),
- };
+ let cors_setting = cors_setting_for_element(image.upcast());
+
+ let img =
+ match canvas_utils::request_image_from_cache(&window, img_url, cors_setting) {
+ ImageResponse::Loaded(img, _) => img,
+ ImageResponse::PlaceholderLoaded(_, _) |
+ ImageResponse::None |
+ ImageResponse::MetadataLoaded(_) => return Ok(None),
+ };
let size = Size2D::new(img.width, img.height);