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

Issue 48123003: Create services subdir under chrome/browser and add initial owners (Closed)

Created:
7 years, 1 month ago by jianli
Modified:
7 years, 1 month ago
CC:
chromium-reviews, Nicolas Zea
Visibility:
Public.

Description

Create services subdir under chrome/browser and add initial owners The new services subdir contains all google services related codes, currently including sync, invalidation and signin. We will move those codes to this new unified directory in future patches. BUG=314886 TEST=none R=thakis@chromium.org, tim@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=233030

Patch Set 1 : Patch #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
A chrome/browser/services/OWNERS View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
jianli
thakis for approval.
7 years, 1 month ago (2013-11-05 00:01:35 UTC) #1
Nico
Can you file a tracking bug for moving things around, and reference that in all ...
7 years, 1 month ago (2013-11-05 00:04:38 UTC) #2
jianli
On 2013/11/05 00:04:38, Nico wrote: > Can you file a tracking bug for moving things ...
7 years, 1 month ago (2013-11-05 00:15:45 UTC) #3
Nico
lgtm
7 years, 1 month ago (2013-11-05 00:22:09 UTC) #4
tim (not reviewing)
LGTM, we'll want to add owners as the subdirs are moved over.
7 years, 1 month ago (2013-11-05 18:19:56 UTC) #5
jianli
7 years, 1 month ago (2013-11-05 18:22:11 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 manually as r233030 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698