Auto merge of #26185 - daemon1024:handle-null-proj, r=Manishearth

handle detached array in XRView

<!-- Please describe your changes on the following line: -->
Recompute XRView.projectionmatrix when the buffer is detached
---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #26153 (GitHub issue number if applicable)

<!-- Either: -->
- [ ] There are tests for these changes OR
- [ ] These changes do not require tests because ___

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
This commit is contained in:
bors-servo 2020-04-15 18:43:16 -04:00 committed by GitHub
commit 324e2d2b42
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -3,6 +3,7 @@
* file, You can obtain one at https://mozilla.org/MPL/2.0/. */
use crate::dom::bindings::codegen::Bindings::XRViewBinding::{XREye, XRViewMethods};
use crate::dom::bindings::reflector::DomObject;
use crate::dom::bindings::reflector::{reflect_dom_object, Reflector};
use crate::dom::bindings::root::{Dom, DomRoot};
use crate::dom::bindings::utils::create_typed_array;
@ -24,17 +25,24 @@ pub struct XRView {
proj: Heap<*mut JSObject>,
#[ignore_malloc_size_of = "mozjs"]
view: Heap<*mut JSObject>,
proj_array: Vec<f32>,
transform: Dom<XRRigidTransform>,
}
impl XRView {
fn new_inherited(session: &XRSession, transform: &XRRigidTransform, eye: XREye) -> XRView {
fn new_inherited(
session: &XRSession,
transform: &XRRigidTransform,
eye: XREye,
proj_array: Vec<f32>,
) -> XRView {
XRView {
reflector_: Reflector::new(),
session: Dom::from_ref(session),
eye,
proj: Heap::default(),
view: Heap::default(),
proj_array,
transform: Dom::from_ref(transform),
}
}
@ -55,13 +63,18 @@ impl XRView {
let transform = pose.pre_transform(&offset);
let transform = XRRigidTransform::new(global, cast_transform(transform));
// row_major since euclid uses row vectors
let proj = view.projection.to_row_major_array();
let ret = reflect_dom_object(
Box::new(XRView::new_inherited(session, &transform, eye)),
Box::new(XRView::new_inherited(
session,
&transform,
eye,
(&proj).to_vec(),
)),
global,
);
// row_major since euclid uses row vectors
let proj = view.projection.to_row_major_array();
let cx = global.get_cx();
create_typed_array(cx, &proj, &ret.proj);
ret
@ -80,6 +93,10 @@ impl XRViewMethods for XRView {
/// https://immersive-web.github.io/webxr/#dom-xrview-projectionmatrix
fn ProjectionMatrix(&self, _cx: JSContext) -> NonNull<JSObject> {
if self.proj.get().is_null() {
let cx = self.global().get_cx();
create_typed_array(cx, &self.proj_array, &self.proj);
}
NonNull::new(self.proj.get()).unwrap()
}