New issue
Advanced search Search tips

Issue 879147 link

Starred by 1 user
Status: Fixed
Owner:
mrobin...@igalia.com
Closed: Sep 18
Cc:
swarnasree.mukkala@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

ax_dump_tree and ax_dump_events do not work on Aura Linux

Project Member Reported by mrobin...@igalia.com, Aug 30 
UserAgent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:62.0) Gecko/20100101 Firefox/62.0

Steps to reproduce the problem:
1. Try to build and run ax_dump_tree when building for Aura Linux.

What is the expected behavior?

What went wrong?
This does not work. The implementation is missing.

Did this work before? N/A 

Chrome version: <Copy from: 'about:version'>  Channel: n/a
OS Version: 
Flash Version:

Comment 1 by mrobin...@igalia.com, Aug 30 

Apologies for the badly categorized bug.

Comment 2 by viswa.karala@chromium.org, Aug 30

Labels: Needs-Milestone

Comment 3 by swarnasree.mukkala@chromium.org, Aug 31

Cc: swarnasree.mukkala@chromium.org
Labels: Triaged-ET TE-NeedsTriageHelp
Thanks for filing the issue..

As the issuue seems to related to build, which is out of TE-Scope. Hence adding TE-NeedsTriageHelp label for further investigation of issue.

Thanks.!

Comment 4 by pfeldman@chromium.org, Sep 3

Components: -Platform>DevTools
Project Member

Comment 6 by bugdroid1@chromium.org, Sep 7 

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

commit 297928530fc400a897f62fa98ccf4a5485990f22
Author: Martin Robinson <mrobinson@igalia.com>
Date: Fri Sep 07 18:11:18 2018

Add an implementation of ax_dump_tree for Aura Linux

Since it is quite complicated to go from an X11 window id to an
accessibility tree with AT-SPI, add a pattern option that accomplishes
essentially the same thing as the PowerShell scripts for Windows.
Eventually we can add support for Windows and the PowerShell scripts
will no longer be required.

Bug:  879147 
Change-Id: Ib9fae5af9849df500f8cbdb6766f22092ac3ac90
Reviewed-on: https://chromium-review.googlesource.com/1196448
Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Martin Robinson <mrobinson@igalia.com>
Cr-Commit-Position: refs/heads/master@{#589583}
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/BUILD.gn
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/build/config/linux/BUILD.gn
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/chrome/installer/linux/debian/dist_package_versions.json
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/chrome/installer/linux/debian/update_dist_package_versions.py
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/chrome/installer/linux/rpm/dist_package_provides.json
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/chrome/installer/linux/rpm/update_package_provides.py
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/content/browser/BUILD.gn
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/content/browser/accessibility/accessibility_tree_formatter.h
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/content/browser/accessibility/accessibility_tree_formatter_auralinux.cc
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/content/browser/accessibility/accessibility_tree_formatter_browser.cc
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/content/browser/accessibility/accessibility_tree_formatter_browser.h
[add] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/content/browser/accessibility/accessibility_tree_formatter_utils_auralinux.cc
[add] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/content/browser/accessibility/accessibility_tree_formatter_utils_auralinux.h
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/content/browser/accessibility/accessibility_tree_formatter_win.cc
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/tools/accessibility/inspect/BUILD.gn
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/tools/accessibility/inspect/ax_dump_tree.cc
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/tools/accessibility/inspect/ax_tree_server.cc
[modify] https://crrev.com/297928530fc400a897f62fa98ccf4a5485990f22/tools/accessibility/inspect/ax_tree_server.h

Comment 8 by mrobin...@igalia.com, Sep 10

Components: Internals>Accessibility
Labels: -Via-Wizard-DeveloperTools
Owner: mrobin...@igalia.com
Status: Started (was: Unconfirmed)
Summary: ax_dump_tree and ax_dump_events do not work on Aura Linux (was: ax_dump_tree does not work on Aura Linux)
Project Member

Comment 9 by bugdroid1@chromium.org, Sep 10 

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/puppet/+/9cfb7027273c830e5927a64c9cb56f02aae0146d

commit 9cfb7027273c830e5927a64c9cb56f02aae0146d
Author: Tom Anderson <thomasanderson@google.com>
Date: Mon Sep 10 19:43:51 2018
Project Member

Comment 10 by bugdroid1@chromium.org, Sep 10 

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/puppet/+/e9b3ce93d3a9804457b77838f772426a84587ff5

commit e9b3ce93d3a9804457b77838f772426a84587ff5
Author: Elliott Friedman <friedman@google.com>
Date: Mon Sep 10 20:25:55 2018
Project Member

Comment 11 by bugdroid1@chromium.org, Sep 10 

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

commit 1320001a996e8d90162d86cf88ca1f8bc8a23c82
Author: Tom Anderson <thomasanderson@chromium.org>
Date: Mon Sep 10 22:29:02 2018

Add libatspi2 to install-build-deps.sh

This became necessary to build Chrome after [1] when building without the
sysroot.

[1] https://chromium.googlesource.com/chromium/src/+/297928530fc400a897f62fa98ccf4a5485990f22

BUG= 879147 
TBR=dpranke

Change-Id: Iff2d59b247cd764f404f0beeaf232cc8dc8188aa
Reviewed-on: https://chromium-review.googlesource.com/1214699
Reviewed-by: Michael Moss <mmoss@chromium.org>
Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#590093}
[modify] https://crrev.com/1320001a996e8d90162d86cf88ca1f8bc8a23c82/build/install-build-deps.sh
Project Member

Comment 12 by bugdroid1@chromium.org, Sep 10 

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/puppet/+/293b3b51ad9e72051493fc889f46ff44af98fb09

commit 293b3b51ad9e72051493fc889f46ff44af98fb09
Author: Elliott Friedman <friedman@google.com>
Date: Mon Sep 10 22:51:16 2018
Project Member

Comment 13 by bugdroid1@chromium.org, Sep 18 

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

commit e4fddeded48a3e8649569f486855012be45a7a46
Author: Martin Robinson <mrobinson@igalia.com>
Date: Tue Sep 18 10:30:21 2018

Add Aura/Linux support to ax_dump_events

Bug:  879147 
Change-Id: I167a41fc75206e631d09608acf748a11ef143c22
Reviewed-on: https://chromium-review.googlesource.com/1210182
Commit-Queue: Martin Robinson <mrobinson@igalia.com>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#591989}
[modify] https://crrev.com/e4fddeded48a3e8649569f486855012be45a7a46/BUILD.gn
[modify] https://crrev.com/e4fddeded48a3e8649569f486855012be45a7a46/content/browser/accessibility/accessibility_event_recorder.cc
[modify] https://crrev.com/e4fddeded48a3e8649569f486855012be45a7a46/content/browser/accessibility/accessibility_event_recorder.h
[modify] https://crrev.com/e4fddeded48a3e8649569f486855012be45a7a46/content/browser/accessibility/accessibility_event_recorder_auralinux.cc
[modify] https://crrev.com/e4fddeded48a3e8649569f486855012be45a7a46/content/browser/accessibility/accessibility_event_recorder_mac.mm
[modify] https://crrev.com/e4fddeded48a3e8649569f486855012be45a7a46/content/browser/accessibility/accessibility_event_recorder_win.cc
[modify] https://crrev.com/e4fddeded48a3e8649569f486855012be45a7a46/content/browser/accessibility/accessibility_tree_formatter_utils_auralinux.cc
[modify] https://crrev.com/e4fddeded48a3e8649569f486855012be45a7a46/content/browser/accessibility/accessibility_tree_formatter_utils_auralinux.h
[modify] https://crrev.com/e4fddeded48a3e8649569f486855012be45a7a46/content/test/BUILD.gn
[modify] https://crrev.com/e4fddeded48a3e8649569f486855012be45a7a46/tools/accessibility/inspect/BUILD.gn
[modify] https://crrev.com/e4fddeded48a3e8649569f486855012be45a7a46/tools/accessibility/inspect/ax_dump_events.cc
[modify] https://crrev.com/e4fddeded48a3e8649569f486855012be45a7a46/tools/accessibility/inspect/ax_event_server.cc
[modify] https://crrev.com/e4fddeded48a3e8649569f486855012be45a7a46/tools/accessibility/inspect/ax_event_server.h

Comment 14 by mrobin...@igalia.com, Sep 18

Status: Fixed (was: Started)

Comment 15 by mrobin...@igalia.com, Sep 18

Labels: -TE-NeedsTriageHelp -Needs-Milestone
Project Member

Comment 16 by bugdroid1@chromium.org, Nov 30 

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

commit 32c051ee1d76c33d389492b1cee23f0d1d752c55
Author: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>
Date: Fri Nov 30 23:10:19 2018

Move the "atspi2" target to a separate BUILD.gn file

//build/config/linux/BUILD.gn is read by //build/config/compiler/BUILD.gn,
which means it is processed by both host and target toolchains.

Consequently, having the "atspi2" target there means "gn gen" can fail
because the host lacks atspi-2's development files even though they are only
needed by the target toolchain during the build.

Bug:  879147 
Change-Id: I8780e6e4d3a2b3a44fdb2cbbb0862a1a669acdaa
Reviewed-on: https://chromium-review.googlesource.com/c/1356979
Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>
Cr-Commit-Position: refs/heads/master@{#612831}
[modify] https://crrev.com/32c051ee1d76c33d389492b1cee23f0d1d752c55/build/config/linux/BUILD.gn
[add] https://crrev.com/32c051ee1d76c33d389492b1cee23f0d1d752c55/build/config/linux/atspi2/BUILD.gn
[modify] https://crrev.com/32c051ee1d76c33d389492b1cee23f0d1d752c55/build/dotfile_settings.gni
[modify] https://crrev.com/32c051ee1d76c33d389492b1cee23f0d1d752c55/content/browser/BUILD.gn
[modify] https://crrev.com/32c051ee1d76c33d389492b1cee23f0d1d752c55/content/test/BUILD.gn
►

Sign in to add a comment