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

Issue 99007: Let BaseWinUtilTest.FormatMessage pass in Japanese-localized WinXP.... (Closed)

Created:
11 years, 8 months ago by yutak
Modified:
9 years, 7 months ago
Reviewers:
sky, sky1
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Let BaseWinUtilTest.FormatMessage pass on localized Windows systems. Originally, BaseWinUtilTest.FormatMessage depends on current system locale, and fails on many localized systems. This patch makes the test pass on localized systems by temporarily changing the current thread's locale to US-English (en-US) during the test. BUG=11003

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+29 lines, -53 lines) Patch
M base/win_util_unittest.cc View 1 3 chunks +29 lines, -53 lines 0 comments Download

Messages

Total messages: 10 (0 generated)
yutak_google.com
A fix for BaseWinUtilTest.FormatMessage. See commit log for details. This is my first time to ...
11 years, 8 months ago (2009-04-24 23:05:30 UTC) #1
sky
Do we really want to expand this test for all languages we support? My vote ...
11 years, 8 months ago (2009-04-24 23:15:47 UTC) #2
yutak_google.com
Thank you for your review, and I'm sorry for my late response. Basically I agree. ...
11 years, 8 months ago (2009-04-27 18:01:34 UTC) #3
sky
Does SPI_SETDEFAULTINPUTLANG work for this?
11 years, 8 months ago (2009-04-27 18:21:12 UTC) #4
yutak_google.com
I think that is for changing keyboard layouts, not messages (the HKL type represents a ...
11 years, 8 months ago (2009-04-27 18:49:43 UTC) #5
yutak_google.com
SetLocaleInfo may be possible. I'll test it.
11 years, 8 months ago (2009-04-27 19:01:53 UTC) #6
yutak_google.com
Finally I've found SetThreadLocale API for this purpose and written a new patch using it. ...
11 years, 8 months ago (2009-04-27 21:50:44 UTC) #7
sky
LGTM
11 years, 8 months ago (2009-04-27 21:55:22 UTC) #8
yutak_google.com
Thank you for your review! I don't have the right to commit, so could you ...
11 years, 8 months ago (2009-04-27 22:06:19 UTC) #9
yutak_google.com
11 years, 8 months ago (2009-04-28 00:32:25 UTC) #10
Landed in r14685.

Powered by Google App Engine
This is Rietveld 408576698