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

Issue 2570483003: Cleanup the kernel service implementation and avoid printing on the console when no tracing flags a… (Closed)

Created:
4 years ago by kasperl
Modified:
4 years ago
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Cleanup the kernel service implementation and avoid printing on the console when no tracing flags are provided. R=kmillikin@google.com, hausner@google.com BUG= Committed: https://github.com/dart-lang/sdk/commit/53dd819a4c9817530dd8f04e422f50fb8b3bd28e

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+39 lines, -91 lines) Patch
M runtime/bin/main.cc View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/tools/kernel-service.dart View 2 chunks +39 lines, -90 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
kasperl
4 years ago (2016-12-12 09:02:25 UTC) #1
Kevin Millikin (Google)
LGTM.
4 years ago (2016-12-12 10:03:30 UTC) #2
hausner
Nice cleanup. Thanks Kasper.
4 years ago (2016-12-12 18:23:23 UTC) #3
kasperl
4 years ago (2016-12-13 08:08:18 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
53dd819a4c9817530dd8f04e422f50fb8b3bd28e (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698