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

Issue 1395863002: Introduce the "Embedded" category. (Closed)

Created:
5 years, 2 months ago by sigurdm
Modified:
5 years, 2 months ago
CC:
reviews_dartlang.org
Base URL:
https://github.com/dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 : #

Total comments: 2

Patch Set 2 : Address review #

Total comments: 2

Patch Set 3 : Address review 2 #

Patch Set 4 : Fix analyzer test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+288 lines, -128 lines) Patch
M pkg/analyzer/lib/src/generated/sdk.dart View 3 chunks +18 lines, -3 lines 0 comments Download
M pkg/analyzer/test/generated/all_the_rest_test.dart View 1 2 3 5 chunks +6 lines, -6 lines 0 comments Download
M pkg/compiler/lib/src/apiimpl.dart View 6 chunks +71 lines, -54 lines 0 comments Download
M pkg/compiler/lib/src/compiler.dart View 1 chunk +10 lines, -2 lines 0 comments Download
M pkg/compiler/lib/src/dart2js.dart View 2 chunks +3 lines, -12 lines 0 comments Download
M pkg/compiler/samples/darttags/darttags.dart View 3 chunks +3 lines, -3 lines 0 comments Download
M pkg/compiler/samples/jsonify/jsonify.dart View 4 chunks +4 lines, -4 lines 0 comments Download
M sdk/lib/_internal/sdk_library_metadata/lib/libraries.dart View 1 2 3 5 chunks +84 lines, -40 lines 0 comments Download
M tests/compiler/dart2js/analyze_api_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M tests/compiler/dart2js/analyze_helper.dart View 3 chunks +3 lines, -1 line 0 comments Download
A tests/compiler/dart2js/categories_test.dart View 1 chunk +35 lines, -0 lines 0 comments Download
A tests/compiler/dart2js/embedded_category_api_boundary_test.dart View 1 chunk +48 lines, -0 lines 0 comments Download
M tests/compiler/dart2js/library_resolution_test.dart View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 17 (6 generated)
sigurdm
@johnniwinther: Do you know who would be the right one to review the analyzer changes?
5 years, 2 months ago (2015-10-08 12:17:10 UTC) #4
Johnni Winther
On 2015/10/08 12:17:10, sigurdm wrote: > @johnniwinther: Do you know who would be the right ...
5 years, 2 months ago (2015-10-08 13:14:48 UTC) #5
Johnni Winther
lgtm https://codereview.chromium.org/1395863002/diff/40001/sdk/lib/_internal/sdk_library_metadata/lib/libraries.dart File sdk/lib/_internal/sdk_library_metadata/lib/libraries.dart (right): https://codereview.chromium.org/1395863002/diff/40001/sdk/lib/_internal/sdk_library_metadata/lib/libraries.dart#newcode332 sdk/lib/_internal/sdk_library_metadata/lib/libraries.dart:332: if (_categories == "") return <Category>[]; const <Category>[]
5 years, 2 months ago (2015-10-08 13:14:56 UTC) #6
sigurdm
@paulberry: We are generalizing the concept of categories. Could you look at the changes to ...
5 years, 2 months ago (2015-10-08 13:29:01 UTC) #8
Brian Wilkerson
LGTM https://codereview.chromium.org/1395863002/diff/60001/sdk/lib/_internal/sdk_library_metadata/lib/libraries.dart File sdk/lib/_internal/sdk_library_metadata/lib/libraries.dart (right): https://codereview.chromium.org/1395863002/diff/60001/sdk/lib/_internal/sdk_library_metadata/lib/libraries.dart#newcode48 sdk/lib/_internal/sdk_library_metadata/lib/libraries.dart:48: categories: "Clientd", "Clientd" --> "Client"
5 years, 2 months ago (2015-10-08 17:03:17 UTC) #10
sigurdm
https://codereview.chromium.org/1395863002/diff/60001/sdk/lib/_internal/sdk_library_metadata/lib/libraries.dart File sdk/lib/_internal/sdk_library_metadata/lib/libraries.dart (right): https://codereview.chromium.org/1395863002/diff/60001/sdk/lib/_internal/sdk_library_metadata/lib/libraries.dart#newcode48 sdk/lib/_internal/sdk_library_metadata/lib/libraries.dart:48: categories: "Clientd", On 2015/10/08 17:03:17, Brian Wilkerson wrote: > ...
5 years, 2 months ago (2015-10-09 09:14:26 UTC) #11
sigurdm
Committed patchset #3 (id:80001) manually as 0aef0efb6a38a65fa676fa9e8daa85a972521745 (presubmit successful).
5 years, 2 months ago (2015-10-09 11:08:44 UTC) #12
sigurdm
PTAL
5 years, 2 months ago (2015-10-09 11:45:41 UTC) #14
Johnni Winther
lgtm
5 years, 2 months ago (2015-10-09 11:48:00 UTC) #15
Brian Wilkerson
LGTM
5 years, 2 months ago (2015-10-09 13:51:19 UTC) #16
sigurdm
5 years, 2 months ago (2015-10-12 10:16:00 UTC) #17
Message was sent while issue was closed.
Committed patchset #4 (id:120001) manually as
a763c71d92e2e1f3e36c702e2b1b3249f774ff9a (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698