Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(195)

Issue 1200953007: Move internals.dart out of Sky SDK (Closed)

Created:
5 years, 6 months ago by abarth-chromium
Modified:
5 years, 6 months ago
Reviewers:
eseidel, mpcomplete, Hixie
CC:
gregsimon, jackson_old, mojo-reviews_chromium.org, qsr+mojo_chromium.org, yzshen+watch_chromium.org
Base URL:
git@github.com:domokit/mojo.git@master
Target Ref:
refs/heads/master
Project:
mojo
Visibility:
Public.

Description

Move internals.dart out of Sky SDK This file doesn't belong in the SDK because it is an implementation detail of the Sky engine. Instead, this CL moves the code for dart:sky.internals into the snapshot. This CL is a step towards merging dart:sky.internals with dart:sky, which also resides in the snapshot. R=eseidel@chromium.org Committed: https://chromium.googlesource.com/external/mojo/+/3d58ea59585d63d013458e4602457754b3fd3670

Patch Set 1 #

Patch Set 2 : BUILD.gn #

Patch Set 3 : _____ #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -104 lines) Patch
M sky/engine/bindings/BUILD.gn View 3 chunks +11 lines, -5 lines 2 comments Download
M sky/engine/bindings/builtin.cc View 1 chunk +1 line, -1 line 0 comments Download
D sky/engine/bindings/builtin.dart View 1 chunk +0 lines, -57 lines 0 comments Download
A + sky/engine/bindings/builtin_natives.dart View 1 chunk +1 line, -1 line 0 comments Download
A + sky/engine/bindings/sky_internals.dart View 1 chunk +2 lines, -0 lines 0 comments Download
M sky/engine/bindings/snapshot.dart View 1 chunk +2 lines, -1 line 0 comments Download
M sky/sdk/BUILD.gn View 1 1 chunk +0 lines, -1 line 0 comments Download
D sky/sdk/lib/internals.dart View 1 chunk +0 lines, -14 lines 0 comments Download
M sky/shell/ui/internals.cc View 2 chunks +1 line, -6 lines 0 comments Download
M sky/tools/skyanalyzer View 1 2 1 chunk +5 lines, -3 lines 0 comments Download
M sky/viewer/internals.cc View 1 chunk +1 line, -15 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
abarth-chromium
5 years, 6 months ago (2015-06-23 05:14:14 UTC) #1
eseidel
lgtm Why is this its own dart: instead of just being part of one of ...
5 years, 6 months ago (2015-06-23 06:24:24 UTC) #3
abarth-chromium
https://codereview.chromium.org/1200953007/diff/40001/sky/engine/bindings/BUILD.gn File sky/engine/bindings/BUILD.gn (right): https://codereview.chromium.org/1200953007/diff/40001/sky/engine/bindings/BUILD.gn#newcode80 sky/engine/bindings/BUILD.gn:80: dart_sky_builtin_natives_path = On 2015/06/23 at 06:24:24, eseidel wrote: > ...
5 years, 6 months ago (2015-06-23 14:43:58 UTC) #4
abarth-chromium
5 years, 6 months ago (2015-06-23 14:44:34 UTC) #5
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
3d58ea59585d63d013458e4602457754b3fd3670 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698