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

Implement Console grouping APIs. #27088

Merged
merged 1 commit into from Jul 7, 2020
Merged

Implement Console grouping APIs. #27088

merged 1 commit into from Jul 7, 2020

Conversation

@jdm
Copy link
Member

jdm commented Jun 25, 2020

These are used by Hubs and other sites we want to run.


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes fix #9274
  • These changes do not require tests because we can't test stdout content or devtools messages.
@highfive
Copy link

highfive commented Jun 25, 2020

Heads up! This PR modifies the following files:

  • @asajeffrey: components/script/dom/globalscope.rs, components/script/dom/webidls/Console.webidl, components/script/dom/console.rs
  • @KiChjang: components/script/dom/globalscope.rs, components/script/dom/webidls/Console.webidl, components/script/dom/console.rs
@highfive
Copy link

highfive commented Jun 25, 2020

warning Warning warning

  • These commits modify script code, but no tests are modified. Please consider adding a test!
Copy link
Collaborator

CYBAI left a comment

Found wrong spec link 👀

components/script/dom/console.rs Outdated Show resolved Hide resolved
components/script/dom/console.rs Outdated Show resolved Hide resolved
@jdm
Copy link
Member Author

jdm commented Jun 30, 2020

r? @ferjm

@ferjm
ferjm approved these changes Jul 1, 2020
Copy link
Member

ferjm left a comment

r=me with the spec links fixed

* © Copyright 2014 Mozilla Foundation.
*/

// https://console.spec.whatwg.org/

This comment has been minimized.

@jdm jdm force-pushed the jdm:group branch from fbd72b2 to 09c7dae Jul 6, 2020
@jdm
Copy link
Member Author

jdm commented Jul 6, 2020

@bors-servo r=ferjm

@bors-servo
Copy link
Contributor

bors-servo commented Jul 6, 2020

📌 Commit 09c7dae has been approved by ferjm

@jdm
Copy link
Member Author

jdm commented Jul 6, 2020

@jdm jdm force-pushed the jdm:group branch from 09c7dae to 2d7effd Jul 6, 2020
@jdm
Copy link
Member Author

jdm commented Jul 6, 2020

@bors-servo r=ferjm

@bors-servo
Copy link
Contributor

bors-servo commented Jul 6, 2020

📌 Commit 2d7effd has been approved by ferjm

@bors-servo
Copy link
Contributor

bors-servo commented Jul 7, 2020

Testing commit 2d7effd with merge 713b26a...

bors-servo added a commit that referenced this pull request Jul 7, 2020
Implement Console grouping APIs.

These are used by Hubs and other sites we want to run.

---
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #9274
- [x] These changes do not require tests because we can't test stdout content or devtools messages.
@bors-servo
Copy link
Contributor

bors-servo commented Jul 7, 2020

💔 Test failed - status-taskcluster

@jdm
Copy link
Member Author

jdm commented Jul 7, 2020

@bors-servo
Copy link
Contributor

bors-servo commented Jul 7, 2020

Testing commit 2d7effd with merge a630944...

bors-servo added a commit that referenced this pull request Jul 7, 2020
Implement Console grouping APIs.

These are used by Hubs and other sites we want to run.

---
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #9274
- [x] These changes do not require tests because we can't test stdout content or devtools messages.
@highfive highfive removed the S-tests-failed label Jul 7, 2020
@bors-servo
Copy link
Contributor

bors-servo commented Jul 7, 2020

💔 Test failed - status-taskcluster

@jdm
Copy link
Member Author

jdm commented Jul 7, 2020

@bors-servo
Copy link
Contributor

bors-servo commented Jul 7, 2020

Testing commit 2d7effd with merge cbbbe16...

@bors-servo
Copy link
Contributor

bors-servo commented Jul 7, 2020

☀️ Test successful - status-taskcluster
Approved by: ferjm
Pushing cbbbe16 to master...

@bors-servo bors-servo merged commit cbbbe16 into servo:master Jul 7, 2020
2 checks passed
2 checks passed
Community-TC (pull_request) TaskGroup: success
Details
homu Test successful
Details
@bors-servo bors-servo mentioned this pull request Jul 7, 2020
4 of 4 tasks complete
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.

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