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

Issue 1323183002: Revert "Extract meta data about open sockets into separate class." (Closed)

Created:
5 years, 3 months ago by ricow1
Modified:
5 years, 3 months ago
Reviewers:
Søren Gjesse
CC:
reviews_dartlang.org, turnidge, rmacnak, Cutch, vm-dev_dartlang.org
Base URL:
https://github.com/dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Revert "Extract meta data about open sockets into separate class." This reverts commit 01eff8ce18be7eceb6ce0be1097005965edf2588. Seeing issues on windows and on dart2js R=sgjesse@google.com BUG= Committed: https://github.com/dart-lang/sdk/commit/665f397bb893413e914fa518f2baefe54b05317d

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+151 lines, -458 lines) Patch
M runtime/bin/socket_patch.dart View 20 chunks +151 lines, -67 lines 0 comments Download
D runtime/observatory/tests/service/tcp_socket_service_test.dart View 1 chunk +0 lines, -157 lines 0 comments Download
D runtime/observatory/tests/service/udp_socket_service_test.dart View 1 chunk +0 lines, -72 lines 0 comments Download
M sdk/lib/io/io.dart View 2 chunks +0 lines, -2 lines 0 comments Download
D sdk/lib/io/io_resource_info.dart View 1 chunk +0 lines, -159 lines 0 comments Download
M sdk/lib/io/iolib_sources.gypi View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
ricow1
tbr
5 years, 3 months ago (2015-09-02 14:18:43 UTC) #1
ricow1
Committed patchset #1 (id:1) manually as 665f397bb893413e914fa518f2baefe54b05317d (presubmit successful).
5 years, 3 months ago (2015-09-02 14:19:41 UTC) #2
Søren Gjesse
5 years, 3 months ago (2015-09-02 14:35:03 UTC) #3
Message was sent while issue was closed.
lgtm :-(

Powered by Google App Engine
This is Rietveld 408576698