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

Issue 24453002: Remove unused method PrefService::ReloadPersistentPrefs(). (Closed)

Created:
7 years, 3 months ago by gab
Modified:
7 years, 2 months ago
Reviewers:
battre
CC:
chromium-reviews, erikwright+watch_chromium.org
Visibility:
Public.

Description

Remove unused method PrefService::ReloadPersistentPrefs(). This was removed as part of removing the import process in issue 219419. BUG=219419 R=battre@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=225185

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -10 lines) Patch
M base/prefs/pref_service.h View 1 chunk +0 lines, -5 lines 0 comments Download
M base/prefs/pref_service.cc View 1 chunk +0 lines, -5 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
gab
Please take a look, feel free to CQ. Thanks! Gab
7 years, 3 months ago (2013-09-24 21:56:19 UTC) #1
battre
lgtm
7 years, 2 months ago (2013-09-25 09:51:41 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gab@chromium.org/24453002/1
7 years, 2 months ago (2013-09-25 09:51:44 UTC) #3
commit-bot: I haz the power
Retried try job too often on linux_rel for step(s) net_unittests, remoting_unittests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&number=171848
7 years, 2 months ago (2013-09-25 10:33:50 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gab@chromium.org/24453002/1
7 years, 2 months ago (2013-09-25 13:12:51 UTC) #5
commit-bot: I haz the power
Retried try job too often on linux_rel for step(s) unit_tests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&number=171896
7 years, 2 months ago (2013-09-25 14:09:52 UTC) #6
gab
7 years, 2 months ago (2013-09-25 14:15:12 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 manually as r225185 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698