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

style: Sync changes from mozilla-central. #25299

Merged
merged 28 commits into from Dec 17, 2019
Merged

Conversation

@emilio
Copy link
Member

emilio commented Dec 15, 2019

See individual commits for details.

jeffin143 and others added 8 commits Dec 1, 2019
…eConsts.h

Differential Revision: https://phabricator.services.mozilla.com/D55382
…nsts.h

Differential Revision: https://phabricator.services.mozilla.com/D55379
…zing border shorthand.

Differential Revision: https://phabricator.services.mozilla.com/D55588
…perty.

Differential Revision: https://phabricator.services.mozilla.com/D54722
…n.enabled.

Differential Revision: https://phabricator.services.mozilla.com/D55898
@highfive
Copy link

highfive commented Dec 15, 2019

warning Warning warning

  • These commits modify style, layout, and gfx code, but no tests are modified. Please consider adding a test!
@emilio
Copy link
Member Author

emilio commented Dec 15, 2019

r? for the layout changes @SimonSapin / @nox

I need to squash too.

There's no behavior change for now, but I want to land this to make implementing min() / max() / etc. easier.

@emilio
Copy link
Member Author

emilio commented Dec 15, 2019

bors-servo added a commit that referenced this pull request Dec 15, 2019
style: Sync changes from mozilla-central.

See individual commits for details.
@bors-servo
Copy link
Contributor

bors-servo commented Dec 15, 2019

Trying commit 0200923 with merge ffa9fee...

@atouchet
Copy link
Contributor

atouchet commented Dec 15, 2019

@emilio if you are updating selectors could you also bump the version for #24574?

@bors-servo
Copy link
Contributor

bors-servo commented Dec 15, 2019

💔 Test failed - status-taskcluster

@emilio
Copy link
Member Author

emilio commented Dec 15, 2019

@atouchet will do

@emilio
Copy link
Member Author

emilio commented Dec 15, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Dec 15, 2019

Trying commit 2d3e4e7 with merge b1d2836...

bors-servo added a commit that referenced this pull request Dec 15, 2019
style: Sync changes from mozilla-central.

See individual commits for details.
@bors-servo
Copy link
Contributor

bors-servo commented Dec 15, 2019

💔 Test failed - status-taskcluster

@CYBAI
Copy link
Collaborator

CYBAI commented Dec 16, 2019

  ▶ Unexpected subtest result in /css/cssom/border-shorthand-serialization.html:
  └ PASS [expected FAIL] Declaration with border longhands and border-image is not serialized to a border shorthand declaration.
bors-servo added a commit that referenced this pull request Dec 16, 2019
style: Sync changes from mozilla-central.

See individual commits for details.
@bors-servo
Copy link
Contributor

bors-servo commented Dec 16, 2019

💔 Test failed - status-taskcluster

@CYBAI
Copy link
Collaborator

CYBAI commented Dec 16, 2019

  ▶ TIMEOUT [expected PASS] /css/CSS2/floats-clear/floats-024.xht
  │ 
  │ 
  │ 
  │ 
  │ Shutting down the Constellation after generating an output file or exit flag specified
  └ 18 threads are still running after shutdown (bad).
  ▶ TIMEOUT [expected PASS] /css/CSS2/floats-clear/floats-040.xht
  │ 
  │ 
  │ 
  │ 
  │ Shutting down the Constellation after generating an output file or exit flag specified
  │ 18 threads are still running after shutdown (bad).
  │ 
  │ 
  │ 
  │ Shutting down the Constellation after generating an output file or exit flag specified
  └ 19 threads are still running after shutdown (bad).
  ▶ TIMEOUT [expected FAIL] /css/CSS2/floats-clear/floats-015.xht
  │ 
  │ 
  │ 
  │ 
  │ Shutting down the Constellation after generating an output file or exit flag specified
  └ 18 threads are still running after shutdown (bad).

Looks like #24762 (or #25258?)

@jdm
Copy link
Member

jdm commented Dec 16, 2019

@jdm jdm closed this Dec 16, 2019
@jdm jdm reopened this Dec 16, 2019
@jdm
Copy link
Member

jdm commented Dec 16, 2019

@bors-servo retry

@emilio
Copy link
Member Author

emilio commented Dec 16, 2019

@bors-servo r=emilio,nox

  • Huh, what's up with bors?
@bors-servo
Copy link
Contributor

bors-servo commented Dec 16, 2019

💡 This pull request was already approved, no need to approve it again.

  • There's another pull request that is currently being tested, blocking this pull request: #25306
@bors-servo
Copy link
Contributor

bors-servo commented Dec 16, 2019

📌 Commit 7513bc2 has been approved by emilio,nox

@jdm
Copy link
Member

jdm commented Dec 16, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Dec 17, 2019

Testing commit 7513bc2 with merge 858b9da...

bors-servo added a commit that referenced this pull request Dec 17, 2019
style: Sync changes from mozilla-central.

See individual commits for details.
@bors-servo
Copy link
Contributor

bors-servo commented Dec 17, 2019

💔 Test failed - status-taskcluster

@jdm
Copy link
Member

jdm commented Dec 17, 2019

@bors-servo retry p=2

@bors-servo
Copy link
Contributor

bors-servo commented Dec 17, 2019

Testing commit 7513bc2 with merge a8b8f46...

bors-servo added a commit that referenced this pull request Dec 17, 2019
style: Sync changes from mozilla-central.

See individual commits for details.
@bors-servo
Copy link
Contributor

bors-servo commented Dec 17, 2019

☀️ Test successful - status-taskcluster
Approved by: emilio,nox
Pushing a8b8f46 to master...

@bors-servo bors-servo merged commit 7513bc2 into servo:master Dec 17, 2019
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 Dec 17, 2019
3 of 3 tasks complete
@emilio emilio deleted the emilio:gecko-sync branch Feb 10, 2020
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

You can’t perform that action at this time.