New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 610795 link

Starred by 1 user
Status: Fixed
Owner:
█ aval...@chromium.org
Last visit > 30 days ago
Closed: Jun 2017
Cc:
wfh@chromium.org
creis@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 610819
Blocking:
issue 710982



Sign in to add a comment

<webivew> OOPIF: Enable WebViewAccessibilityTest.FocusAccessibility

Project Member Reported by aval...@chromium.org, May 10 2016 
The test works by tabbing from the embedder to the webview's <button> and watching for AX focus events.

With --use-cross-process-frames-for-guests, these events do not fire and the test times out.

Comment 1 by aval...@chromium.org, Dec 7 2016

Blockedon: 610819

Comment 2 by aval...@chromium.org, Jan 25 2017

Status: Started (was: Available)

Comment 3 by aval...@chromium.org, Feb 21 2017 

https://codereview.chromium.org/2674353003/ looks ready to land.

Comment 4 by aval...@chromium.org, Apr 12 2017

Blocking: 710982
Project Member

Comment 5 by bugdroid1@chromium.org, Jun 21 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d68cdd1afdff55d24558ef88441d9ac950d46dd4

commit d68cdd1afdff55d24558ef88441d9ac950d46dd4
Author: avallee <avallee@chromium.org>
Date: Wed Jun 21 22:07:27 2017

Enable WebViewAccessiblityTest for OOPIF webview.

+ Enabled WebViewAccessibilityTests for oopif.
+ Hooked up the inner AX tree to point back to the AX tree for the outer web
  contents. Otherwise BrowserAccessibility::Platform{ChildCount,GetChild}
  functions fail on the [child->GetParent() == this] check.

BUG= 610795 ,  685739 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2674353003
Cr-Commit-Position: refs/heads/master@{#481326}

[modify] https://crrev.com/d68cdd1afdff55d24558ef88441d9ac950d46dd4/chrome/browser/apps/guest_view/web_view_browsertest.cc
[modify] https://crrev.com/d68cdd1afdff55d24558ef88441d9ac950d46dd4/content/browser/frame_host/render_frame_host_delegate.cc
[modify] https://crrev.com/d68cdd1afdff55d24558ef88441d9ac950d46dd4/content/browser/frame_host/render_frame_host_delegate.h
[modify] https://crrev.com/d68cdd1afdff55d24558ef88441d9ac950d46dd4/content/browser/frame_host/render_frame_host_impl.cc
[modify] https://crrev.com/d68cdd1afdff55d24558ef88441d9ac950d46dd4/content/browser/web_contents/web_contents_impl.cc
[modify] https://crrev.com/d68cdd1afdff55d24558ef88441d9ac950d46dd4/content/browser/web_contents/web_contents_impl.h
[modify] https://crrev.com/d68cdd1afdff55d24558ef88441d9ac950d46dd4/content/renderer/render_frame_impl.cc

Comment 6 by aval...@chromium.org, Jun 22 2017

Status: Fixed (was: Started)
►

Sign in to add a comment