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

Side by Side Diff: chrome/browser/ui/views/website_settings/permission_selector_view.cc

Issue 103493005: Rename View::set_focusable and View::set_accessibility_focusable (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Added a few missing files Created 7 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/website_settings/permission_selector_view.h" 5 #include "chrome/browser/ui/views/website_settings/permission_selector_view.h"
6 6
7 #include "base/strings/utf_string_conversions.h" 7 #include "base/strings/utf_string_conversions.h"
8 #include "chrome/browser/ui/website_settings/website_settings_ui.h" 8 #include "chrome/browser/ui/website_settings/website_settings_ui.h"
9 #include "grit/generated_resources.h" 9 #include "grit/generated_resources.h"
10 #include "ui/base/accessibility/accessible_view_state.h" 10 #include "ui/base/accessibility/accessible_view_state.h"
(...skipping 198 matching lines...) Expand 10 before | Expand all | Expand 10 after
209 menu_button_model_.reset(new PermissionMenuModel( 209 menu_button_model_.reset(new PermissionMenuModel(
210 this, url, type, default_setting, current_setting)); 210 this, url, type, default_setting, current_setting));
211 bool button_enabled = source == content_settings::SETTING_SOURCE_USER; 211 bool button_enabled = source == content_settings::SETTING_SOURCE_USER;
212 menu_button_ = new internal::PermissionMenuButton( 212 menu_button_ = new internal::PermissionMenuButton(
213 WebsiteSettingsUI::PermissionActionToUIString(current_setting, 213 WebsiteSettingsUI::PermissionActionToUIString(current_setting,
214 default_setting, 214 default_setting,
215 source), 215 source),
216 menu_button_model_.get(), 216 menu_button_model_.get(),
217 button_enabled); 217 button_enabled);
218 menu_button_->SetEnabled(button_enabled); 218 menu_button_->SetEnabled(button_enabled);
219 menu_button_->set_focusable(button_enabled); 219 menu_button_->SetFocusable(button_enabled);
220 menu_button_->SetAccessibleName( 220 menu_button_->SetAccessibleName(
221 WebsiteSettingsUI::PermissionTypeToUIString(type)); 221 WebsiteSettingsUI::PermissionTypeToUIString(type));
222 layout->AddView(menu_button_); 222 layout->AddView(menu_button_);
223 } 223 }
224 224
225 void PermissionSelectorView::AddObserver( 225 void PermissionSelectorView::AddObserver(
226 PermissionSelectorViewObserver* observer) { 226 PermissionSelectorViewObserver* observer) {
227 observer_list_.AddObserver(observer); 227 observer_list_.AddObserver(observer);
228 } 228 }
229 229
(...skipping 27 matching lines...) Expand all
257 content_settings::SETTING_SOURCE_USER)); 257 content_settings::SETTING_SOURCE_USER));
258 258
259 FOR_EACH_OBSERVER(PermissionSelectorViewObserver, 259 FOR_EACH_OBSERVER(PermissionSelectorViewObserver,
260 observer_list_, 260 observer_list_,
261 OnPermissionChanged(this)); 261 OnPermissionChanged(this));
262 } 262 }
263 263
264 bool PermissionSelectorView::IsCommandIdChecked(int command_id) { 264 bool PermissionSelectorView::IsCommandIdChecked(int command_id) {
265 return current_setting_ == CommandIdToContentSetting(command_id); 265 return current_setting_ == CommandIdToContentSetting(command_id);
266 } 266 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/toolbar/wrench_menu.cc ('k') | ui/app_list/views/search_result_actions_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698