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

Issue 6534013: [Sync] Set logging that may contain personal information to verbosity level 2. (Closed)

Created:
9 years, 10 months ago by Nicolas Zea
Modified:
9 years, 7 months ago
Reviewers:
akalin
CC:
chromium-reviews, ncarter (slow), idana, Raghu Simha, Ilya Sherman, James Hawkins, tim (not reviewing), dhollowa
Visibility:
Public.

Description

[Sync] Set logging that may contain personal information to verbosity level 2. BUG=68954 TEST=Sign into sync for first time with --vmodule="*/sync/*"=1 and see what gets printed Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=75336

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -15 lines) Patch
M chrome/browser/sync/engine/download_updates_command.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/sync/engine/syncer_util.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/sync/glue/autofill_model_associator.cc View 2 chunks +4 lines, -4 lines 0 comments Download
M chrome/browser/sync/glue/autofill_profile_model_associator.cc View 6 chunks +9 lines, -9 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Nicolas Zea
Take a look?
9 years, 10 months ago (2011-02-17 02:36:30 UTC) #1
akalin
9 years, 10 months ago (2011-02-17 02:41:40 UTC) #2
On 2011/02/17 02:36:30, nzea wrote:
> Take a look?

LGTm

Powered by Google App Engine
This is Rietveld 408576698