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

Adapt layout 2020 viewer to new BoxTree and FragmentTree structs #26565

Merged
merged 1 commit into from Jun 18, 2020

Conversation

@ferjm
Copy link
Member

ferjm commented May 19, 2020

  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors

#26414 introduced some changes to how we represent the box and fragment trees, so we need to adapt the layout viewer to support them.

@ferjm
Copy link
Member Author

ferjm commented May 19, 2020

@highfive highfive assigned SimonSapin and unassigned asajeffrey May 19, 2020
@jdm
Copy link
Member

jdm commented Jun 9, 2020

@SimonSapin review ping.

@jdm
Copy link
Member

jdm commented Jun 17, 2020

@SimonSapin Another review ping.

@SimonSapin
Copy link
Member

SimonSapin commented Jun 18, 2020

Sorry for the delay. I’ve followed the instructions in etc/layout_viewer/README and verified that I could get some output about the layout trace rather than a JS error.

@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented Jun 18, 2020

📌 Commit 8882713 has been approved by SimonSapin

@bors-servo
Copy link
Contributor

bors-servo commented Jun 18, 2020

Testing commit 8882713 with merge d4d7cae...

@bors-servo
Copy link
Contributor

bors-servo commented Jun 18, 2020

☀️ Test successful - status-taskcluster
Approved by: SimonSapin
Pushing d4d7cae to master...

@bors-servo bors-servo merged commit d4d7cae into servo:master Jun 18, 2020
2 checks passed
2 checks passed
Community-TC (pull_request) TaskGroup: success
Details
homu Test successful
Details
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

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