Skip to content
This repository has been archived by the owner on Jul 10, 2023. It is now read-only.

Implement Android get_dimensions #133

Merged
merged 1 commit into from
Nov 21, 2017
Merged

Conversation

MortimerGoro
Copy link

@MortimerGoro MortimerGoro commented Nov 20, 2017

At some point Servo started using Glutin get_dimensions() method but it's not implemented on Android.

Please, publish the new version to crates.io once this is merged.


This change is Reviewable

@MortimerGoro
Copy link
Author

r? @jdm

@jdm
Copy link
Member

jdm commented Nov 20, 2017

@bors-servo: r+

@bors-servo
Copy link

📌 Commit 1180d27 has been approved by jdm

@bors-servo
Copy link

⌛ Testing commit 1180d27 with merge 8bb72af...

bors-servo pushed a commit that referenced this pull request Nov 20, 2017
Implement Android get_dimensions

At some point Servo started using Glutin `get_dimensions()` method but it's not implemented on Android.

Please, publish the new version to crates.io once this is merged.

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

☀️ Test successful - status-travis
Approved by: jdm
Pushing 8bb72af to servo...

@bors-servo bors-servo merged commit 1180d27 into servo:servo Nov 21, 2017
bors-servo pushed a commit to servo/servo that referenced this pull request Nov 23, 2017
Update glutin to 0.13.1

<!-- Please describe your changes on the following line: -->

Includes servo/glutin#133

---
<!-- 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
- [ ] These changes fix #__ (github issue number if applicable).

<!-- Either: -->
- [x] 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. -->

<!-- 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/19357)
<!-- Reviewable:end -->
bors-servo pushed a commit to servo/servo that referenced this pull request Nov 23, 2017
Update glutin to 0.13.1

<!-- Please describe your changes on the following line: -->

Includes servo/glutin#133

---
<!-- 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
- [ ] These changes fix #__ (github issue number if applicable).

<!-- Either: -->
- [x] 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. -->

<!-- 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/19357)
<!-- Reviewable:end -->
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Nov 24, 2017
…in_131); r=jdm

<!-- Please describe your changes on the following line: -->

Includes servo/glutin#133

---
<!-- 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
- [ ] These changes fix #__ (github issue number if applicable).

<!-- Either: -->
- [x] 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. -->

Source-Repo: https://github.com/servo/servo
Source-Revision: 177285ad4a2a7fc2c211d616e58f2a98844c8013

--HG--
extra : subtree_source : https%3A//hg.mozilla.org/projects/converted-servo-linear
extra : subtree_revision : ed4e16a621cde9d0f2e2650eafdec921fad4830f
kuoe0 pushed a commit to kuoe0/gecko-dev that referenced this pull request Nov 24, 2017
…in_131); r=jdm

<!-- Please describe your changes on the following line: -->

Includes servo/glutin#133

---
<!-- 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
- [ ] These changes fix #__ (github issue number if applicable).

<!-- Either: -->
- [x] 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. -->

Source-Repo: https://github.com/servo/servo
Source-Revision: 177285ad4a2a7fc2c211d616e58f2a98844c8013
aethanyc pushed a commit to aethanyc/gecko-dev that referenced this pull request Nov 26, 2017
…in_131); r=jdm

<!-- Please describe your changes on the following line: -->

Includes servo/glutin#133

---
<!-- 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
- [ ] These changes fix #__ (github issue number if applicable).

<!-- Either: -->
- [x] 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. -->

Source-Repo: https://github.com/servo/servo
Source-Revision: 177285ad4a2a7fc2c211d616e58f2a98844c8013
JerryShih pushed a commit to JerryShih/gecko-dev that referenced this pull request Nov 28, 2017
…in_131); r=jdm

<!-- Please describe your changes on the following line: -->

Includes servo/glutin#133

---
<!-- 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
- [ ] These changes fix #__ (github issue number if applicable).

<!-- Either: -->
- [x] 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. -->

Source-Repo: https://github.com/servo/servo
Source-Revision: 177285ad4a2a7fc2c211d616e58f2a98844c8013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants