Commit a9438c2a authored by Elodie Banel's avatar Elodie Banel Committed by Commit Bot

Revert "[iOS] Accessibility adjustments"

This reverts commit 1e910c7f.

Reason for revert: Breaks ios_internal_chrome_external_url_egtests (see example failure here https://logs.chromium.org/v/?s=chrome%2Fbb%2Finternal.bling.main%2Fipad10-simulator%2F13834%2F%2B%2Frecipes%2Fsteps%2Fios_internal_chrome_external_url_egtests__iPad_Retina_iOS_10.2__on_Mac%2F0%2Flogs%2FExternalURLGWSTestCase__x2f_testGWSHistoryAutosuggest%2F0 )

Original change's description:
> [iOS] Accessibility adjustments
> 
> This CL adds a delay for the voice over announcement of the InProduct
> Help tips. This delay is added to prevent the announcement to be
> cancelled by the element being focused, when the tab is presented.
> It also adds an accessibility label to the empty space where the omnibox
> should be if the toolbar was visible.
> 
> Bug: 866936
> Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
> Change-Id: Ia75dd56772203e39b8d0bf1e94bef9cfacd87fdd
> Reviewed-on: https://chromium-review.googlesource.com/1158236
> Commit-Queue: edchin <edchin@chromium.org>
> Reviewed-by: edchin <edchin@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#579886}

TBR=edchin@chromium.org,gambard@chromium.org

Change-Id: I172a7e67b8ef5d4b4a2008f0f95ef99d7c0152e2
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 866936
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Reviewed-on: https://chromium-review.googlesource.com/1160281Reviewed-by: default avatarElodie Banel <lod@chromium.org>
Commit-Queue: Elodie Banel <lod@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580121}
parent 7aabaf07
...@@ -24,9 +24,6 @@ const NSTimeInterval kBubbleVisibilityDuration = 5.0; ...@@ -24,9 +24,6 @@ const NSTimeInterval kBubbleVisibilityDuration = 5.0;
// after the bubble first becomes visible. // after the bubble first becomes visible.
const NSTimeInterval kBubbleEngagementDuration = 30.0; const NSTimeInterval kBubbleEngagementDuration = 30.0;
// Delay before posting the VoiceOver notification.
const CGFloat kVoiceOverAnnouncementDelay = 1;
// The name for the histogram that tracks why a bubble was dismissed. // The name for the histogram that tracks why a bubble was dismissed.
const char kBubbleDismissalHistogramName[] = "IOS.IPHBubbleDismissalReason"; const char kBubbleDismissalHistogramName[] = "IOS.IPHBubbleDismissalReason";
...@@ -186,21 +183,8 @@ void LogBubbleDismissalReason(BubbleDismissalReason reason) { ...@@ -186,21 +183,8 @@ void LogBubbleDismissalReason(BubbleDismissalReason reason) {
repeats:NO]; repeats:NO];
if (self.voiceOverAnnouncement) { if (self.voiceOverAnnouncement) {
// The VoiceOverAnnouncement should be dispatched after a delay to account UIAccessibilityPostNotification(UIAccessibilityAnnouncementNotification,
// the fact that it can be presented right after a screen change (for self.voiceOverAnnouncement);
// example when the application or a new tab is opened). This screen change
// is changing the VoiceOver focus to focus a newly visible element. If this
// announcement is currently being read, it is cancelled. The added delay
// allows the announcement to be posted after the element is focused, so it
// is not cancelled.
dispatch_after(
dispatch_time(DISPATCH_TIME_NOW,
(int64_t)(kVoiceOverAnnouncementDelay * NSEC_PER_SEC)),
dispatch_get_main_queue(), ^{
UIAccessibilityPostNotification(
UIAccessibilityAnnouncementNotification,
self.voiceOverAnnouncement);
});
} }
} }
......
...@@ -362,8 +362,6 @@ const UIEdgeInsets kSearchBoxStretchInsets = {3, 3, 3, 3}; ...@@ -362,8 +362,6 @@ const UIEdgeInsets kSearchBoxStretchInsets = {3, 3, 3, 3};
- (void)addFakeTapView { - (void)addFakeTapView {
UIButton* fakeTapButton = [[UIButton alloc] init]; UIButton* fakeTapButton = [[UIButton alloc] init];
fakeTapButton.translatesAutoresizingMaskIntoConstraints = NO; fakeTapButton.translatesAutoresizingMaskIntoConstraints = NO;
fakeTapButton.accessibilityLabel =
l10n_util::GetNSString(IDS_ACCNAME_LOCATION);
[self.headerView addToolbarView:fakeTapButton]; [self.headerView addToolbarView:fakeTapButton];
[fakeTapButton addTarget:self [fakeTapButton addTarget:self
action:@selector(fakeOmniboxTapped:) action:@selector(fakeOmniboxTapped:)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment