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

Issue 15408004: sync: null-check ProfileManager in sync_start_util (Closed)

Created:
7 years, 7 months ago by tim (not reviewing)
Modified:
7 years, 7 months ago
Reviewers:
Raghu Simha, pavely
CC:
chromium-reviews, Raghu Simha, haitaol1, akalin
Visibility:
Public.

Description

sync: null-check ProfileManager in sync_start_util TBR=rsimha@chromium.org ^OWNERS stamp for basic change. BUG=80194 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=201109

Patch Set 1 #

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

Messages

Total messages: 4 (0 generated)
tim (not reviewing)
7 years, 7 months ago (2013-05-18 21:25:50 UTC) #1
pavely
On 2013/05/18 21:25:50, timsteele wrote: LGTM
7 years, 7 months ago (2013-05-20 18:16:05 UTC) #2
tim (not reviewing)
+Raghu for OWNERS approval.
7 years, 7 months ago (2013-05-20 18:23:19 UTC) #3
tim (not reviewing)
7 years, 7 months ago (2013-05-20 18:34:50 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 manually as r201109 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698