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

Issue 552265: r37624 abandoned code that isn't in the project any more. Deleting the unused... (Closed)

Created:
10 years, 10 months ago by Avi (use Gerrit)
Modified:
9 years, 7 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

r37624 abandoned code that isn't in the project any more. Deleting the unused files. See http://codereview.chromium.org/556095 . BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=37759

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -136 lines) Patch
D net/base/cookie_policy.cc View 1 chunk +0 lines, -49 lines 0 comments Download
D net/base/cookie_policy_unittest.cc View 1 chunk +0 lines, -87 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Avi (use Gerrit)
Review to both of you since http://codereview.chromium.org/556095 was yours. Please don't leave abandoned source files ...
10 years, 10 months ago (2010-02-01 20:52:17 UTC) #1
Peter Kasting
I punt this review, as well as the question of merging it to 4.1, to ...
10 years, 10 months ago (2010-02-01 21:48:26 UTC) #2
Avi (use Gerrit)
On 2010/02/01 21:48:26, Peter Kasting wrote: > I punt this review, as well as the ...
10 years, 10 months ago (2010-02-01 21:51:13 UTC) #3
darin (slow to review)
10 years, 10 months ago (2010-02-01 22:20:43 UTC) #4
LGTM.  sorry for leaving these in the tree.

Powered by Google App Engine
This is Rietveld 408576698