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

Unified Diff: chrome/browser/chromeos/login/wizard_accessibility_handler.cc

Issue 7327007: Moving notification types which are chrome specific to a new header file chrome_notification_type... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 5 months 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/login/wizard_accessibility_handler.cc
===================================================================
--- chrome/browser/chromeos/login/wizard_accessibility_handler.cc (revision 91880)
+++ chrome/browser/chromeos/login/wizard_accessibility_handler.cc (working copy)
@@ -100,7 +100,7 @@
namespace chromeos {
void WizardAccessibilityHandler::Observe(
- NotificationType type,
+ int type,
const NotificationSource& source,
const NotificationDetails& details) {
const AccessibilityControlInfo *control_info =
@@ -130,27 +130,27 @@
}
void WizardAccessibilityHandler::DescribeAccessibilityEvent(
- NotificationType event_type,
+ int event_type,
const AccessibilityControlInfo* control_info,
std::string* out_spoken_description,
EarconType* out_earcon) {
*out_spoken_description = std::string();
*out_earcon = NO_EARCON;
- switch (event_type.value) {
- case NotificationType::ACCESSIBILITY_CONTROL_FOCUSED:
+ switch (event_type) {
+ case chrome::ACCESSIBILITY_CONTROL_FOCUSED:
DescribeControl(control_info, false, out_spoken_description, out_earcon);
break;
- case NotificationType::ACCESSIBILITY_CONTROL_ACTION:
+ case chrome::ACCESSIBILITY_CONTROL_ACTION:
DescribeControl(control_info, true, out_spoken_description, out_earcon);
break;
- case NotificationType::ACCESSIBILITY_TEXT_CHANGED:
+ case chrome::ACCESSIBILITY_TEXT_CHANGED:
DescribeTextChanged(control_info, out_spoken_description, out_earcon);
break;
- case NotificationType::ACCESSIBILITY_MENU_OPENED:
+ case chrome::ACCESSIBILITY_MENU_OPENED:
*out_earcon = EARCON_OBJECT_OPENED;
break;
- case NotificationType::ACCESSIBILITY_MENU_CLOSED:
+ case chrome::ACCESSIBILITY_MENU_CLOSED:
*out_earcon = EARCON_OBJECT_CLOSED;
break;
default:

Powered by Google App Engine
This is Rietveld 408576698