diff options
| author | mukai@chromium.org <mukai@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-04-10 03:38:30 +0000 |
|---|---|---|
| committer | mukai@chromium.org <mukai@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-04-10 03:38:30 +0000 |
| commit | 8d228aa89d9f06ca00d2d712faad070af299576f (patch) | |
| tree | e89d1b10f67fd8248d84831c27468ba096a64745 /ui/message_center/views/notifier_settings_view.cc | |
| parent | 9b192f0b0be740a2f84c9a9e3e163a6a2173be1e (diff) | |
| download | chromium_src-8d228aa89d9f06ca00d2d712faad070af299576f.zip chromium_src-8d228aa89d9f06ca00d2d712faad070af299576f.tar.gz chromium_src-8d228aa89d9f06ca00d2d712faad070af299576f.tar.bz2 | |
Disallow focus on checkbox for notification settings.
This line was somehow removed during the code review but this is
necessary to fix crbug.com/196804. The checkbox is focusable by default
but we do not want to focus on the checkbox itself.
BUG=196804
Review URL: https://chromiumcodereview.appspot.com/13952003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193308 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ui/message_center/views/notifier_settings_view.cc')
| -rw-r--r-- | ui/message_center/views/notifier_settings_view.cc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/ui/message_center/views/notifier_settings_view.cc b/ui/message_center/views/notifier_settings_view.cc index ef8ce64..36f6f45 100644 --- a/ui/message_center/views/notifier_settings_view.cc +++ b/ui/message_center/views/notifier_settings_view.cc @@ -145,6 +145,7 @@ class NotifierSettingsView::NotifierButton : public views::CustomButton, views::BoxLayout::kHorizontal, 0, 0, kSpaceInButtonComponents)); checkbox_->SetChecked(notifier_->enabled); checkbox_->set_listener(this); + checkbox_->set_focusable(false); AddChildView(checkbox_); UpdateIconImage(notifier_->icon); AddChildView(new views::Label(notifier_->name)); |