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

Issue 6685093: Use keyboard_layout instead of initial_locale (Closed)

Created:
9 years, 9 months ago by Nick Sanders
Modified:
9 years, 7 months ago
Reviewers:
Hung-Te, yongjaek, Jay Kim
CC:
chromium-os-reviews_chromium.org, sosa+cc_chromium.org, seano+cc_chromium.org, ericli, Tammo Spalink
Visibility:
Public.

Description

Use keyboard_layout instead of initial_locale * SelectKeyboard writes both kb layout and locale * hw components checks kb layout * still todo (another CL): reference kb file from kb layout * still todo (another CL): en-GB is stubbed out. Need actual png. BUG=chrome-os-partner:2591 TEST=will do Change-Id: If828499983fc2066197e4ecfd0713c144c78bbea Review URL: http://codereview.chromium.org/6679050 Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=6570817

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+92 lines, -7 lines) Patch
M client/site_tests/factory_Keyboard/factory_Keyboard.py View 1 chunk +4 lines, -1 line 0 comments Download
A client/site_tests/factory_Keyboard/src/en-GB.bindings View 1 chunk +79 lines, -0 lines 0 comments Download
A client/site_tests/factory_Keyboard/src/en-GB.png View Binary file 0 comments Download
M client/site_tests/factory_SelectKeyboard/factory_SelectKeyboard.py View 1 chunk +8 lines, -5 lines 0 comments Download
M client/site_tests/hardware_Components/hardware_Components.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
Nick Sanders
cherry pick to master
9 years, 9 months ago (2011-03-18 20:40:38 UTC) #1
Jay Kim
9 years, 9 months ago (2011-03-18 20:56:07 UTC) #2
LGTM. It is known that the png file has not been corrected yet, but all change
is integrated from the factory branch to the main sincerely.

Powered by Google App Engine
This is Rietveld 408576698