diff options
| author | dmazzoni@chromium.org <dmazzoni@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-01 19:57:53 +0000 |
|---|---|---|
| committer | dmazzoni@chromium.org <dmazzoni@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-01 19:57:53 +0000 |
| commit | f27a0b59065835a9fe6f9a30681c36f14fe9edc3 (patch) | |
| tree | 0ddd5a6d702c7097493490cfbe47f6de6fa4d565 | |
| parent | bd554c644039736cf4eb3944f63b7a8e87c449d9 (diff) | |
| download | chromium_src-f27a0b59065835a9fe6f9a30681c36f14fe9edc3.zip chromium_src-f27a0b59065835a9fe6f9a30681c36f14fe9edc3.tar.gz chromium_src-f27a0b59065835a9fe6f9a30681c36f14fe9edc3.tar.bz2 | |
Disable the code that registers accessibility handlers for the wizard,
as a temporary fix for major Chrome OS problems. They were getting called
even though the user didn't explicitly enable accessibility, and continuing
to fire even after the wizard wasn't open anymore. Should re-enable as soon
as the real root cause has been fixed.
BUG=http://cros.com/7238
BUG=http://cros.com/7240
TEST=manual testing
Review URL: http://codereview.chromium.org/3569009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61217 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/browser/chromeos/login/wizard_accessibility_helper.cc | 37 |
1 files changed, 22 insertions, 15 deletions
diff --git a/chrome/browser/chromeos/login/wizard_accessibility_helper.cc b/chrome/browser/chromeos/login/wizard_accessibility_helper.cc index db0f097..91f3fb2 100644 --- a/chrome/browser/chromeos/login/wizard_accessibility_helper.cc +++ b/chrome/browser/chromeos/login/wizard_accessibility_helper.cc @@ -37,21 +37,28 @@ WizardAccessibilityHelper* WizardAccessibilityHelper::GetInstance() { WizardAccessibilityHelper::WizardAccessibilityHelper() { accessibility_handler_.reset(new WizardAccessibilityHandler()); profile_ = ProfileManager::GetDefaultProfile(); - registrar_.Add(accessibility_handler_.get(), - NotificationType::ACCESSIBILITY_CONTROL_FOCUSED, - NotificationService::AllSources()); - registrar_.Add(accessibility_handler_.get(), - NotificationType::ACCESSIBILITY_CONTROL_ACTION, - NotificationService::AllSources()); - registrar_.Add(accessibility_handler_.get(), - NotificationType::ACCESSIBILITY_TEXT_CHANGED, - NotificationService::AllSources()); - registrar_.Add(accessibility_handler_.get(), - NotificationType::ACCESSIBILITY_MENU_OPENED, - NotificationService::AllSources()); - registrar_.Add(accessibility_handler_.get(), - NotificationType::ACCESSIBILITY_MENU_CLOSED, - NotificationService::AllSources()); + + // http://crosbug.com/7238 + // TODO(dmazzoni), TODO(chaitanyag) + // Commented out to disable trapping handling and responding to + // accessibility notifications until the root cause of this bug has + // been determined. + // + //registrar_.Add(accessibility_handler_.get(), + // NotificationType::ACCESSIBILITY_CONTROL_FOCUSED, + // NotificationService::AllSources()); + //registrar_.Add(accessibility_handler_.get(), + // NotificationType::ACCESSIBILITY_CONTROL_ACTION, + // NotificationService::AllSources()); + //registrar_.Add(accessibility_handler_.get(), + // NotificationType::ACCESSIBILITY_TEXT_CHANGED, + // NotificationService::AllSources()); + //registrar_.Add(accessibility_handler_.get(), + // NotificationType::ACCESSIBILITY_MENU_OPENED, + // NotificationService::AllSources()); + //registrar_.Add(accessibility_handler_.get(), + // NotificationType::ACCESSIBILITY_MENU_CLOSED, + // NotificationService::AllSources()); } void WizardAccessibilityHelper::MaybeEnableAccessibility( |