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

Issue 1126243004: Summarize only per logging level (Closed)

Created:
5 years, 7 months ago by vsm
Modified:
5 years, 7 months ago
Reviewers:
Leaf
CC:
dev-compiler+reviews_dartlang.org, Siggi Cherem (dart-lang)
Base URL:
https://github.com/dart-lang/dev_compiler.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Summarize only per logging level This prunes out INFO messages by default - they end up being a substantial chunk of the payload in server mode. You can still serve them up by passing '-l ALL' or '-l INFO'. R=leafp@google.com Committed: https://github.com/dart-lang/dev_compiler/commit/3af4ea2a4278a6eed8b92def69ab176611292693

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -2 lines) Patch
M lib/devc.dart View 1 chunk +1 line, -1 line 0 comments Download
M lib/src/options.dart View 1 chunk +1 line, -1 line 0 comments Download
M lib/src/report.dart View 2 chunks +5 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
vsm
5 years, 7 months ago (2015-05-14 17:36:36 UTC) #2
Leaf
lgtm
5 years, 7 months ago (2015-05-14 17:41:47 UTC) #3
vsm
5 years, 7 months ago (2015-05-14 17:46:45 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
3af4ea2a4278a6eed8b92def69ab176611292693 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698