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

Issue 9699039: roll rlz 72:79 (Closed)

Created:
8 years, 9 months ago by Nico
Modified:
8 years, 9 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

roll rlz 72:79 73: Delete unused function IsGoodRlz(). 74: Change the unit of several constants from WCHARs to bytes. 75: Start moving functions from win/lib/rlz_lib.h to lib/rlz_lib.h 76: Move GetPingParams() and ParseFinancialPingResponse() to the common code. 77: Move PingFinancialServer to common part. 78: Move FormFinancialPingRequest() to the common part. 79: Address comments from https://codereview.appspot.com/5823050/ BUG=chromium:46579 TEST=should be a no-op change on windows TBR=rogerta Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=126800

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
Nico
8 years, 9 months ago (2012-03-14 18:51:16 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thakis@chromium.org/9699039/1
8 years, 9 months ago (2012-03-14 18:55:39 UTC) #2
commit-bot: I haz the power
Try job failure for 9699039-1 (retry) on win_rel for steps "ui_tests, browser_tests". It's a second ...
8 years, 9 months ago (2012-03-14 21:38:47 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thakis@chromium.org/9699039/1
8 years, 9 months ago (2012-03-14 21:46:20 UTC) #4
commit-bot: I haz the power
8 years, 9 months ago (2012-03-15 00:21:15 UTC) #5
Change committed as 126800

Powered by Google App Engine
This is Rietveld 408576698