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

Merge dart:ui_internals into dart:ui #83

Closed
FlutterIssues opened this issue Nov 9, 2015 · 3 comments
Closed

Merge dart:ui_internals into dart:ui #83

FlutterIssues opened this issue Nov 9, 2015 · 3 comments
Assignees
Labels
c: API break Backwards-incompatible API changes engine flutter/engine repository. See also e: labels. team Infra upgrades, team productivity, code health, technical debt. See also team: labels.

Comments

@FlutterIssues
Copy link

Issue by Hixie
Thursday Jul 16, 2015 at 22:34 GMT
Originally opened as https://github.com/flutter/engine/issues/51


From @Hixie on June 18, 2015 19:55

We should move the "take*" functions from dart:internals to dart:sky.

While we're at it, we should probably remove contentAsText() and renderTreeAsText() (since those test obsolete features). This leaves only notifyTestComplete(), at which point we should rename dart:internals to dart:sky.testing or some such.

Copied from original issue: domokit/mojo#254

@FlutterIssues
Copy link
Author

Comment by Hixie
Thursday Jul 16, 2015 at 22:34 GMT


We might want to rename dart:sky_builtin to dart:sky.internals then too.

@FlutterIssues FlutterIssues added engine flutter/engine repository. See also e: labels. c: new feature Nothing broken; request for a new capability labels Nov 9, 2015
@Hixie Hixie changed the title Merge dart:sky.internals into dart:sky Merge dart:ui_internals into dart:ui Nov 11, 2015
@Hixie Hixie added team Infra upgrades, team productivity, code health, technical debt. See also team: labels. and removed c: new feature Nothing broken; request for a new capability labels Dec 16, 2015
@Hixie Hixie added this to the Blue Sky milestone Dec 16, 2015
@Hixie Hixie added the c: API break Backwards-incompatible API changes label Dec 16, 2015
@abarth abarth self-assigned this Feb 1, 2016
@abarth
Copy link
Contributor

abarth commented Feb 13, 2016

flutter/engine#2397

abarth added a commit to abarth/flutter that referenced this issue Feb 13, 2016
This library is being folded into dart:ui.

Fixes flutter#83
@abarth abarth closed this as completed Feb 13, 2016
abarth added a commit that referenced this issue Feb 13, 2016
This library is being folded into dart:ui.

Fixes #83
@github-actions
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
c: API break Backwards-incompatible API changes engine flutter/engine repository. See also e: labels. team Infra upgrades, team productivity, code health, technical debt. See also team: labels.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants