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

Issue 14478003: Transfer of WebData ownership, as discussed. (Closed)

Created:
7 years, 8 months ago by Jói
Modified:
7 years, 7 months ago
Reviewers:
dhollowa
CC:
chromium-reviews, Raman Kakilate, benquan, dhollowa+watch_chromium.org, ahutter, browser-components-watch_chromium.org, dbeam+watch-autofill_chromium.org, Dane Wallinga, dyu1, estade+watch_chromium.org, Albert Bodenhamer, Ilya Sherman
Visibility:
Public.

Description

Transfer of WebData ownership, as discussed. BUG=none R=dhollowa@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=198456

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -3 lines) Patch
M chrome/browser/webdata/OWNERS View 1 chunk +1 line, -1 line 0 comments Download
M components/webdata/OWNERS View 1 chunk +1 line, -1 line 0 comments Download
M components/webdata/encryptor/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
Jói
Hi David, Looks like I forgot to mail you this review after I prepared it ...
7 years, 7 months ago (2013-05-02 14:56:59 UTC) #1
Jói
gentle ping On Thu, May 2, 2013 at 3:56 PM, <joi@chromium.org> wrote: > Reviewers: dhollowa, ...
7 years, 7 months ago (2013-05-03 16:06:33 UTC) #2
Jói
gentle ping #2
7 years, 7 months ago (2013-05-06 14:49:07 UTC) #3
dhollowa
lgtm LGTM
7 years, 7 months ago (2013-05-06 14:53:11 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/joi@chromium.org/14478003/1
7 years, 7 months ago (2013-05-06 15:08:20 UTC) #5
Jói
7 years, 7 months ago (2013-05-06 15:23:56 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 manually as r198456 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698