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

Issue 3015028: Add Simplified Chinese (Pinyin) input method DOMUI configuration. (Closed)

Created:
10 years, 5 months ago by kochi
Modified:
9 years, 7 months ago
CC:
chromium-reviews, dhg, arv (Not doing code reviews), davemoore+watch_chromium.org, ben+cc_chromium.org, satorux1, mazda
Base URL:
http://src.chromium.org/git/chromium.git
Visibility:
Public.

Description

Add Simplified Chinese (Pinyin) input method DOMUI configuration. Move utility functions into a header file (currently chewing and pinyin are the customers). TEST=manually check on the netbook by uncommenting TODO lines. BUG=chromium-os:4808 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=53753

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+182 lines, -28 lines) Patch
M chrome/browser/chromeos/dom_ui/language_chewing_options_handler.cc View 1 chunk +1 line, -27 lines 0 comments Download
A chrome/browser/chromeos/dom_ui/language_options_util.h View 1 chunk +40 lines, -0 lines 0 comments Download
M chrome/browser/chromeos/dom_ui/language_pinyin_options_handler.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/chromeos/dom_ui/language_pinyin_options_handler.cc View 2 chunks +33 lines, -0 lines 0 comments Download
M chrome/browser/resources/options/chromeos_language_pinyin_options.html View 1 chunk +108 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
kochi
Hi Erik, This time I filled in contents of Pinyin configuration DOMUI. Could you review?
10 years, 5 months ago (2010-07-26 11:37:29 UTC) #1
arv (Not doing code reviews)
lgtm
10 years, 5 months ago (2010-07-26 22:40:35 UTC) #2
kochi
10 years, 5 months ago (2010-07-27 05:50:30 UTC) #3
Thanks for your review.

Satoru-san submitted this CL at 0244413aea78f4282ce9cd159cf62a1738f35976 .

Powered by Google App Engine
This is Rietveld 408576698