diff options
| author | yusukes@chromium.org <yusukes@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-09 13:31:16 +0000 |
|---|---|---|
| committer | yusukes@chromium.org <yusukes@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-09 13:31:16 +0000 |
| commit | 432867bf871d35ab5da94a0d24fd5af9f6dc7d79 (patch) | |
| tree | 1a7d82fefd42e5782d1819fbfc46e6c6f1ac13db /chrome/browser/shell_integration.cc | |
| parent | ca4b5fa384e6ac932826d178f374d0d23e1296af (diff) | |
| download | chromium_src-432867bf871d35ab5da94a0d24fd5af9f6dc7d79.zip chromium_src-432867bf871d35ab5da94a0d24fd5af9f6dc7d79.tar.gz chromium_src-432867bf871d35ab5da94a0d24fd5af9f6dc7d79.tar.bz2 | |
Split the LanguageMenuButton class into 2 classes so that OOBE screen could easily reuse the drop-down menu (but not the "US" button).
The change is necessary for issue 6185.
- A new class InputMethodMenu, which provides the drop-down menu for switching input methods and keyboard layouts, is added.
- The LanguageMenuButton class is modified. Most of the code is moved to the new InputMethodMenu class. Now the class only provides the status area button ("US").
The InputMethodMenu class is almost identical to the old LanguageMenuButton, but
- The new one does not inherit StatusAreaButton since it's not a button
- UpdateIndicator() is renamed to UpdateUI() and changed to pure virtual since the new class does not have UI other than the drop-down menu.
- SetMinimumWidth() is added since the OOBE screen uses fix-width menu.
I think it might be better to rename language_menu_button.h to input_method_menu_button.h as well. If it's necessary, I'll file a bug and do it in a separate CL.
BUG=chromium-os:6185
TEST=manually checked, ran the unittests and browser tests as well.
Review URL: http://codereview.chromium.org/3562008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62082 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/shell_integration.cc')
0 files changed, 0 insertions, 0 deletions