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

Issue 2911903002: Use relative URIs for core library part-of statements. (Closed)

Created:
3 years, 6 months ago by Lasse Reichstein Nielsen
Modified:
3 years, 5 months ago
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Use relative URIs for core library part-of statements. Reapply "Change some platform libraries to use `part of URI`." Change to used relative URIs in the part-of declarations. R=brianwilkerson@google.com, floitsch@google.com Committed: https://github.com/dart-lang/sdk/commit/62bc61b367b14bfe3a265eaddcce0574e0059985

Patch Set 1 #

Total comments: 4

Patch Set 2 : Add newline before part-of declarations where missing. #

Patch Set 3 : Merge to head #

Patch Set 4 : Fix warnings in web-sockets tests, typo in fasta. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+374 lines, -152 lines) Patch
M pkg/front_end/lib/src/fasta/source/source_library_builder.dart View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/js_runtime/lib/annotations.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/js_runtime/lib/constant_map.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/js_runtime/lib/isolate_serialization.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/js_runtime/lib/js_array.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/js_runtime/lib/js_number.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/js_runtime/lib/js_rti.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/js_runtime/lib/js_string.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/js_runtime/lib/linked_hash_map.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/js_runtime/lib/native_helper.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/js_runtime/lib/regexp_helper.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/js_runtime/lib/string_helper.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/async/async_error.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/async/broadcast_stream_controller.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/async/deferred_load.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/async/future.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/async/future_impl.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/async/schedule_microtask.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/async/stream.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/async/stream_controller.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/async/stream_impl.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/async/stream_pipe.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/async/stream_transformers.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/async/timer.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/async/zone.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/collection/collections.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/collection/hash_map.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/collection/hash_set.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/collection/iterable.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/collection/iterator.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/collection/linked_hash_map.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/collection/linked_hash_set.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/collection/linked_list.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/collection/list.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/collection/maps.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/collection/queue.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/collection/set.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/collection/splay_tree.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/convert/ascii.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/convert/base64.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/convert/byte_conversion.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/convert/chunked_conversion.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/convert/codec.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/convert/converter.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/convert/encoding.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/convert/html_escape.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/convert/json.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/convert/latin1.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/convert/line_splitter.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/convert/string_conversion.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/convert/utf.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/annotations.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/bool.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/comparable.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/date_time.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/double.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/duration.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/errors.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/exceptions.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/expando.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/function.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/identical.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/int.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/invocation.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/iterable.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/iterator.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/list.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/map.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/null.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/num.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/object.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/pattern.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/print.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/regexp.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/set.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/sink.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/stacktrace.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/stopwatch.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/string.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/string_buffer.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/string_sink.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/symbol.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/type.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/core/uri.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/developer/extension.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/developer/profiler.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/developer/service.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/developer/timeline.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/html/html_common/conversions.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/html/html_common/conversions_dart2js.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/html/html_common/conversions_dartium.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/html/html_common/css_class_set.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/html/html_common/device.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/html/html_common/filtered_element_list.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/html/html_common/lists.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/internal/iterable.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/internal/list.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/internal/print.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/internal/sort.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/internal/symbol.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/bytes_builder.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/common.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/crypto.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/data_transformer.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/directory.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/directory_impl.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/eventhandler.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/file.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/file_impl.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/file_system_entity.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/http.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/http_date.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/http_headers.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/http_impl.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/http_parser.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/http_session.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/io_resource_info.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/io_service.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/io_sink.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/link.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/platform.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/platform_impl.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/process.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/secure_server_socket.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/secure_socket.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/security_context.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/service_object.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/socket.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/stdio.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/string_transformer.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/sync_socket.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/websocket.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/io/websocket_impl.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/isolate/capability.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/math/jenkins_smi_hash.dart View 1 1 chunk +2 lines, -1 line 0 comments Download
M sdk/lib/math/point.dart View 1 1 chunk +2 lines, -1 line 0 comments Download
M sdk/lib/math/random.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/math/rectangle.dart View 1 1 chunk +2 lines, -1 line 0 comments Download
M sdk/lib/vmservice/asset.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/vmservice/client.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/vmservice/constants.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/vmservice/devfs.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/vmservice/message.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/vmservice/message_router.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/vmservice/running_isolate.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/vmservice/running_isolates.dart View 1 chunk +1 line, -1 line 0 comments Download
A tests/standalone/io/hash_utils.dart View 1 2 3 1 chunk +217 lines, -0 lines 0 comments Download
M tests/standalone/io/web_socket_error_test.dart View 1 2 3 3 chunks +4 lines, -3 lines 0 comments Download
M tests/standalone/io/web_socket_ping_test.dart View 1 2 3 2 chunks +4 lines, -3 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
Lasse Reichstein Nielsen
3 years, 6 months ago (2017-05-29 10:18:29 UTC) #2
Brian Wilkerson
lgtm https://codereview.chromium.org/2911903002/diff/1/sdk/lib/html/html_common/conversions_dart2js.dart File sdk/lib/html/html_common/conversions_dart2js.dart (right): https://codereview.chromium.org/2911903002/diff/1/sdk/lib/html/html_common/conversions_dart2js.dart#newcode1 sdk/lib/html/html_common/conversions_dart2js.dart:1: part of "html_common.dart"; It would be good to ...
3 years, 6 months ago (2017-05-29 14:55:29 UTC) #3
floitsch
LGTM. https://codereview.chromium.org/2911903002/diff/1/sdk/lib/math/jenkins_smi_hash.dart File sdk/lib/math/jenkins_smi_hash.dart (left): https://codereview.chromium.org/2911903002/diff/1/sdk/lib/math/jenkins_smi_hash.dart#oldcode4 sdk/lib/math/jenkins_smi_hash.dart:4: part of dart.math; New line here too. https://codereview.chromium.org/2911903002/diff/1/sdk/lib/math/point.dart ...
3 years, 6 months ago (2017-05-29 15:03:03 UTC) #4
Lasse Reichstein Nielsen
3 years, 5 months ago (2017-07-03 08:04:12 UTC) #6
Message was sent while issue was closed.
Committed patchset #4 (id:60001) manually as
62bc61b367b14bfe3a265eaddcce0574e0059985 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698