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

Issue 23872008: Reapply "Make LinkedHashMap also have a factory constructor and be customizable"" (Closed)

Created:
7 years, 3 months ago by Lasse Reichstein Nielsen
Modified:
7 years, 3 months ago
Reviewers:
floitsch
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Visibility:
Public.

Description

Reapply "Make LinkedHashMap also have a factory constructor and be customizable"" The change was reverted due to failing Dartium tests that were not easily handelable on a Friday afternoon. Committed: https://code.google.com/p/dart/source/detail?r=27284

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+388 lines, -378 lines) Patch
M editor/util/plugins/com.google.dart.java2dart/resources/java_core.dart View 1 chunk +2 lines, -5 lines 0 comments Download
M pkg/analyzer_experimental/lib/src/generated/java_core.dart View 1 chunk +2 lines, -5 lines 0 comments Download
M pkg/mdv/lib/src/template_iterator.dart View 1 chunk +1 line, -3 lines 0 comments Download
M pkg/serialization/lib/src/reader_writer.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/serialization/lib/src/serialization_helpers.dart View 1 chunk +0 lines, -61 lines 0 comments Download
D pkg/serialization/test/polyfill_identity_map_test.dart View 1 chunk +0 lines, -48 lines 0 comments Download
M runtime/lib/collection_patch.dart View 13 chunks +133 lines, -169 lines 0 comments Download
M sdk/lib/_internal/lib/collection_patch.dart View 16 chunks +149 lines, -35 lines 0 comments Download
M sdk/lib/collection/hash_map.dart View 2 chunks +11 lines, -1 line 0 comments Download
M sdk/lib/collection/linked_hash_map.dart View 2 chunks +4 lines, -33 lines 0 comments Download
M sdk/lib/collection/splay_tree.dart View 8 chunks +31 lines, -12 lines 0 comments Download
M tests/corelib/map_test.dart View 3 chunks +51 lines, -2 lines 0 comments Download
M tests/language/issue10561_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M tests/language/issue9949_test.dart View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Lasse Reichstein Nielsen
TBR
7 years, 3 months ago (2013-09-09 10:49:05 UTC) #1
Lasse Reichstein Nielsen
Committed patchset #1 manually as r27284 (presubmit successful).
7 years, 3 months ago (2013-09-09 10:49:49 UTC) #2
floitsch
7 years, 3 months ago (2013-09-09 11:50:27 UTC) #3
Message was sent while issue was closed.
LGTM.

Powered by Google App Engine
This is Rietveld 408576698