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

Issue 12600033: pkg/logging: removed deprecated 'on' API (Closed)

Created:
7 years, 9 months ago by kevmoo-old
Modified:
7 years, 9 months ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

pkg/logging: removed deprecated 'on' API Committed: https://code.google.com/p/dart/source/detail?r=20574

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -405 lines) Patch
M pkg/logging/lib/logging.dart View 5 chunks +0 lines, -61 lines 0 comments Download
M pkg/logging/pubspec.yaml View 1 chunk +2 lines, -8 lines 0 comments Download
D pkg/logging/test/logging_deprecated_test.dart View 1 chunk +0 lines, -336 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
kevmoo-old
PTAL
7 years, 9 months ago (2013-03-25 18:09:31 UTC) #1
gram
Looks okay to me but please wait for an lgtm from Siggi for this one.
7 years, 9 months ago (2013-03-25 18:40:43 UTC) #2
Siggi Cherem (dart-lang)
lgtm
7 years, 9 months ago (2013-03-27 19:07:41 UTC) #3
Siggi Cherem (dart-lang)
On 2013/03/27 19:07:41, Siggi Cherem (dart-lang) wrote: > lgtm ... and thanks for following up ...
7 years, 9 months ago (2013-03-27 19:08:05 UTC) #4
kevmoo-old
7 years, 9 months ago (2013-03-27 20:36:25 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 manually as r20574 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698