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

Issue 396023003: Change SPACEPARITY in the Windows serial api implementation to ODDPARITY. (Closed)

Created:
6 years, 5 months ago by Sam McNally
Modified:
6 years, 5 months ago
CC:
chromium-reviews, chrome-apps-syd-reviews_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Change SPACEPARITY in the Windows serial api implementation to ODDPARITY. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=284000

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M device/serial/serial_io_handler_win.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
Sam McNally
6 years, 5 months ago (2014-07-17 08:03:40 UTC) #1
Ken Rockot(use gerrit already)
lgtm
6 years, 5 months ago (2014-07-17 14:46:37 UTC) #2
Sam McNally
The CQ bit was checked by sammc@chromium.org
6 years, 5 months ago (2014-07-17 21:26:38 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sammc@chromium.org/396023003/1
6 years, 5 months ago (2014-07-17 21:29:23 UTC) #4
commit-bot: I haz the power
Change committed as 284000
6 years, 5 months ago (2014-07-18 03:46:13 UTC) #5
gab
6 years, 5 months ago (2014-07-18 14:13:16 UTC) #6
Message was sent while issue was closed.
A revert of this CL has been created in
https://codereview.chromium.org/399353007/ by gab@chromium.org.

The reason for reverting is: Very speculative revert for current XP bots
failures (all 3 bots are failing with obscure errors and this is our last
suspect in the list...).

I'll reland if that's not it.

Thanks for your understanding!.

Powered by Google App Engine
This is Rietveld 408576698