Add pref entry for virtual keyboard

Add pref entry for virtual keyboard.

This CL depends on http://codereview.chromium.org/7497028/.

BUG=None
TEST=manual, ran unit_tests

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=95578

[mazda, 2011-08-03 02:10:20]

LGTM

[bryeung, 2011-08-03 22:15:40]

http://codereview.chromium.org/7493077/diff/1/chrome/browser/chromeos/input_m...
File chrome/browser/chromeos/input_method/virtual_keyboard_selector.cc (right):

http://codereview.chromium.org/7493077/diff/1/chrome/browser/chromeos/input_m...
chrome/browser/chromeos/input_method/virtual_keyboard_selector.cc:139: void
VirtualKeyboardSelector::ClearUserPreference() {
should probably be ClearUserPreferences or ClearAllUserPreferences to make it
more clear that it is going to get rid of everything

http://codereview.chromium.org/7493077/diff/1/chrome/browser/chromeos/input_m...
File chrome/browser/chromeos/input_method/virtual_keyboard_selector_unittest.cc
(right):

http://codereview.chromium.org/7493077/diff/1/chrome/browser/chromeos/input_m...
chrome/browser/chromeos/input_method/virtual_keyboard_selector_unittest.cc:377:
selector.ClearUserPreference();
There should still be a test for RemoveUserPreference

http://codereview.chromium.org/7493077/diff/1/chrome/browser/chromeos/prefere...
File chrome/browser/chromeos/preferences.cc (right):

http://codereview.chromium.org/7493077/diff/1/chrome/browser/chromeos/prefere...
chrome/browser/chromeos/preferences.cc:578: VLOG(1) << "virtual keyboard
preference: use " << url
I'd rather not have this logging code if it isn't too inconvenient to remove it.

[Yusuke Sato, 2011-08-04 07:34:50]

http://codereview.chromium.org/7493077/diff/1/chrome/browser/chromeos/input_m...
File chrome/browser/chromeos/input_method/virtual_keyboard_selector.cc (right):

http://codereview.chromium.org/7493077/diff/1/chrome/browser/chromeos/input_m...
chrome/browser/chromeos/input_method/virtual_keyboard_selector.cc:139: void
VirtualKeyboardSelector::ClearUserPreference() {
On 2011/08/03 22:15:40, bryeung wrote:
> should probably be ClearUserPreferences or ClearAllUserPreferences to make it
> more clear that it is going to get rid of everything

Done.

http://codereview.chromium.org/7493077/diff/1/chrome/browser/chromeos/input_m...
File chrome/browser/chromeos/input_method/virtual_keyboard_selector_unittest.cc
(right):

http://codereview.chromium.org/7493077/diff/1/chrome/browser/chromeos/input_m...
chrome/browser/chromeos/input_method/virtual_keyboard_selector_unittest.cc:377:
selector.ClearUserPreference();
On 2011/08/03 22:15:40, bryeung wrote:
> There should still be a test for RemoveUserPreference

Done.

http://codereview.chromium.org/7493077/diff/1/chrome/browser/chromeos/prefere...
File chrome/browser/chromeos/preferences.cc (right):

http://codereview.chromium.org/7493077/diff/1/chrome/browser/chromeos/prefere...
chrome/browser/chromeos/preferences.cc:578: VLOG(1) << "virtual keyboard
preference: use " << url
On 2011/08/03 22:15:40, bryeung wrote:
> I'd rather not have this logging code if it isn't too inconvenient to remove
it.

Done.

[bryeung, 2011-08-04 17:28:01]

LGTM