Skip to content

Conversation

@ncarrillo
Copy link
Contributor

@ncarrillo ncarrillo commented Apr 21, 2018

Fixes #2672


This change is Reviewable

@glennw
Copy link
Member

glennw commented Apr 22, 2018

Thanks, looks good to me. I built Gecko and it compiles and runs with this PR, so 🚀

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit 4ab3936 has been approved by glennw

@bors-servo
Copy link
Contributor

⌛ Testing commit 4ab3936 with merge 53d21e7...

bors-servo pushed a commit that referenced this pull request Apr 22, 2018
Remove "Layer" space

Fixes #2672

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/2679)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - status-appveyor, status-taskcluster
Approved by: glennw
Pushing 53d21e7 to master...

@staktrace
Copy link
Contributor

staktrace commented Apr 23, 2018

For the record, this did break the Gecko build. (see https://bugzilla.mozilla.org/show_bug.cgi?id=1454623#c9)

@glennw
Copy link
Member

glennw commented Apr 23, 2018

(from discussion with kats on IRC)

The reason I was able to build locally in this case is that I didn't do a FFI bindings regeneration step, so it just happened to work since the memory layout between the two types is exactly the same. Sorry!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants