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

Issue 295103010: Remove an obsolete comment. (Closed)

Created:
6 years, 7 months ago by Ilya Sherman
Modified:
6 years, 7 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Remove an obsolete comment. The code this is referring to has moved. It's now checked as part of testing whether metrics reporting is enabled, internally to the metrics code. BUG=none TEST=none R=asvitkine@chromium.org TBR=thestig@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=272525

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -1 line) Patch
M chrome/browser/chrome_browser_main.cc View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 7 (0 generated)
Ilya Sherman
6 years, 7 months ago (2014-05-23 08:58:40 UTC) #1
Alexei Svitkine (slow)
lgtm
6 years, 7 months ago (2014-05-23 09:00:40 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/isherman@chromium.org/295103010/1
6 years, 7 months ago (2014-05-23 09:02:29 UTC) #3
Ilya Sherman
TBR Lei for removing an out-of-date comment from chrome_browser_main.cc. [Hope you don't mind us TBRing ...
6 years, 7 months ago (2014-05-23 09:23:33 UTC) #4
Lei Zhang
lgtm
6 years, 7 months ago (2014-05-23 09:36:17 UTC) #5
commit-bot: I haz the power
FYI, CQ is re-trying this CL (attempt #1). Please consider checking whether the failures are ...
6 years, 7 months ago (2014-05-23 12:30:28 UTC) #6
commit-bot: I haz the power
6 years, 7 months ago (2014-05-23 14:46:10 UTC) #7
Message was sent while issue was closed.
Change committed as 272525

Powered by Google App Engine
This is Rietveld 408576698