Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update webidl of ServiceWorkerRegistration #22171

Merged
merged 1 commit into from Nov 16, 2018
Merged

Conversation

@CYBAI
Copy link
Collaborator

CYBAI commented Nov 11, 2018

I'll start to work on #19302 and I'd like to update these sw related webidl one by one.


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes is part of #19302
  • These changes do not require tests because it just updates the webidl for ServiceWorkerRegistration; related tests should be updated when we start to implement its functions.

This change is Reviewable

@highfive
Copy link

highfive commented Nov 11, 2018

Heads up! This PR modifies the following files:

  • @asajeffrey: components/script/dom/webidls/ServiceWorkerRegistration.webidl, components/script/dom/serviceworkerregistration.rs
  • @KiChjang: components/script/dom/webidls/ServiceWorkerRegistration.webidl, components/script/dom/serviceworkerregistration.rs
@highfive
Copy link

highfive commented Nov 11, 2018

warning Warning warning

  • These commits modify script code, but no tests are modified. Please consider adding a test!
@CYBAI CYBAI mentioned this pull request Nov 11, 2018
5 of 6 tasks complete
@CYBAI
Copy link
Collaborator Author

CYBAI commented Nov 11, 2018

@bors-servo try=wpt

@bors-servo
Copy link
Contributor

bors-servo commented Nov 11, 2018

Trying commit 72ffbb7 with merge a2b3dfa...

bors-servo added a commit that referenced this pull request Nov 11, 2018
Update webidl of ServiceWorkerRegistration

I'll start to work on #19302 and I'd like to update these sw related webidl one by one.

---
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes is part of #19302
- [ ] There are tests for these changes OR
- [ ] These changes do not require tests because _____

<!-- 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/22171)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 11, 2018

☀️ Test successful - linux-rel-css, linux-rel-wpt
State: approved= try=True

@jdm
Copy link
Member

jdm commented Nov 13, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Nov 13, 2018

📌 Commit 72ffbb7 has been approved by jdm

@highfive highfive assigned jdm and unassigned Manishearth Nov 13, 2018
bors-servo added a commit that referenced this pull request Nov 13, 2018
Update webidl of ServiceWorkerRegistration

I'll start to work on #19302 and I'd like to update these sw related webidl one by one.

---
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes is part of #19302
- [x] These changes do not require tests because it just updates the webidl for `ServiceWorkerRegistration`; related tests should be updated when we start to implement its functions.

<!-- 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/22171)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 13, 2018

Testing commit 72ffbb7 with merge 8ba70bf...

@bors-servo
Copy link
Contributor

bors-servo commented Nov 13, 2018

💔 Test failed - status-taskcluster

@CYBAI
Copy link
Collaborator Author

CYBAI commented Nov 13, 2018

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Nov 13, 2018

💣 Failed to start rebuilding: Unknown error

@bors-servo
Copy link
Contributor

bors-servo commented Nov 13, 2018

Testing commit 72ffbb7 with merge d796f4a...

bors-servo added a commit that referenced this pull request Nov 13, 2018
Update webidl of ServiceWorkerRegistration

I'll start to work on #19302 and I'd like to update these sw related webidl one by one.

---
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes is part of #19302
- [x] These changes do not require tests because it just updates the webidl for `ServiceWorkerRegistration`; related tests should be updated when we start to implement its functions.

<!-- 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/22171)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 13, 2018

💔 Test failed - mac-rel-wpt4

@jdm
Copy link
Member

jdm commented Nov 15, 2018

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Nov 15, 2018

Testing commit 72ffbb7 with merge e7874ba...

bors-servo added a commit that referenced this pull request Nov 15, 2018
Update webidl of ServiceWorkerRegistration

I'll start to work on #19302 and I'd like to update these sw related webidl one by one.

---
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes is part of #19302
- [x] These changes do not require tests because it just updates the webidl for `ServiceWorkerRegistration`; related tests should be updated when we start to implement its functions.

<!-- 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/22171)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 16, 2018

💔 Test failed - linux-dev

@jdm
Copy link
Member

jdm commented Nov 16, 2018

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Nov 16, 2018

💣 Failed to start rebuilding: Unknown error

@bors-servo
Copy link
Contributor

bors-servo commented Nov 16, 2018

Testing commit 72ffbb7 with merge eb141fa...

bors-servo added a commit that referenced this pull request Nov 16, 2018
Update webidl of ServiceWorkerRegistration

I'll start to work on #19302 and I'd like to update these sw related webidl one by one.

---
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes is part of #19302
- [x] These changes do not require tests because it just updates the webidl for `ServiceWorkerRegistration`; related tests should be updated when we start to implement its functions.

<!-- 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/22171)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 16, 2018

💔 Test failed - linux-rel-wpt

@CYBAI
Copy link
Collaborator Author

CYBAI commented Nov 16, 2018

gcc-5: warning: .: linker input file unused because linking not done
ar: .libs/libmesagallium.a: No space left on device
make[5]: *** [libmesagallium.la] Error 1
make[4]: *** [all-recursive] Error 1
make[3]: *** [all] Error 2
make[2]: *** [all-recursive] Error 1
make[1]: *** [all] Error 2
make: *** [all-recursive] Error 1
@jdm
Copy link
Member

jdm commented Nov 16, 2018

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Nov 16, 2018

Testing commit 72ffbb7 with merge 1421ad2...

bors-servo added a commit that referenced this pull request Nov 16, 2018
Update webidl of ServiceWorkerRegistration

I'll start to work on #19302 and I'd like to update these sw related webidl one by one.

---
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes is part of #19302
- [x] These changes do not require tests because it just updates the webidl for `ServiceWorkerRegistration`; related tests should be updated when we start to implement its functions.

<!-- 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/22171)
<!-- Reviewable:end -->
@jdm
Copy link
Member

jdm commented Nov 16, 2018

@bors-servo retry

bors-servo added a commit that referenced this pull request Nov 16, 2018
Update webidl of ServiceWorkerRegistration

I'll start to work on #19302 and I'd like to update these sw related webidl one by one.

---
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes is part of #19302
- [x] These changes do not require tests because it just updates the webidl for `ServiceWorkerRegistration`; related tests should be updated when we start to implement its functions.

<!-- 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/22171)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 16, 2018

Testing commit 72ffbb7 with merge fa77675...

@bors-servo bors-servo merged commit 72ffbb7 into servo:master Nov 16, 2018
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@CYBAI CYBAI deleted the CYBAI:sw-up-to-date branch Nov 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.