Emilio Cobos Álvarez

Results 777 comments of Emilio Cobos Álvarez

This needs a rebase but I think the failure is legit. Is it crashing on the valist in that test? If you can reproduce locally, it'd be great to see...

Ok, to reproduce that on x86_64 you can: ``` cargo run -- --wrap-static-fns bindgen-tests/tests/headers/wrap-static-fns.h -- -target aarch64-unknown-linux-gnu ``` It seems bindgen can't reason about the builtin va_list there: https://github.com/Blizzard/clang/blob/6c67f57680c38ccd146786841eeac753ad45bf0f/lib/AST/ASTContext.cpp#L6099 As...

Please do file an issue about the wrap-static-fns not dealing with valists in aarch64

Ah, yeah, good point, I guess this is only about `inline-size`.

> While EdgeHTML's behavior makes the most sense in the end, it's also not spec-compatible right now because height should return its used value, not its computed value. The reason...

I think this is sensible, though it needs a rebase and it'd be good to document what the intended use case for having the mangling name around is.

Gecko at least uses the [current](https://html.spec.whatwg.org/multipage/webappapis.html#concept-current-everything) global's document at the point of construction of the observer. I think that's also what WebKit does given the `CurrentDocument` [here](https://searchfox.org/wubkat/rev/00b3d6a56f5149245b9e1501389ae169846ec240/Source/WebCore/page/ResizeObserver.idl#33). Not quite sure...

Something that would perhaps be a bit more consistent with existing APIs would be something like `DocumentOrShadowRoot.topmostTopLayerElement` (or something, naming TBD), where the API would appropriately re-target much like `fullscreenElement`...

Does `GeometryUtils.getBoxQuads` give you what you want? Seems like it would do exactly what you want?