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

Issue 6391003: Roll back an incorrect change to SyncPageHandler.ChromiumSyncCommandHandler. (Closed)

Created:
9 years, 11 months ago by Satish
Modified:
9 years, 7 months ago
Reviewers:
Nikita (slow)
CC:
chromium-reviews, darin-cc_chromium.org, cbentzel+watch_chromium.org, pam+watch_chromium.org
Visibility:
Public.

Description

Roll back an incorrect change to SyncPageHandler.ChromiumSyncCommandHandler. In CL http://codereview.chromium.org/6134003 I did a find/replace for all places in testserver.py that read the request body and incorrectly replaced this one as well. The intention was only to make the change to methods of TestPageHandler and not SyncPageHandler. BUG=none TEST=sync_integration_tests builds fine in the buildbots. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72486

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M net/tools/testserver/testserver.py View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
Satish
9 years, 11 months ago (2011-01-25 12:09:40 UTC) #1
Nikita (slow)
9 years, 11 months ago (2011-01-25 12:09:47 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698