Commit c3740b5c authored by Dominic Mazzoni's avatar Dominic Mazzoni Committed by Commit Bot

Fix crash in AXEventGenerator::PostprocessEvents

This is a quick fix to merge. I'm going to follow up with a patch to
explore the underlying cause.

Bug: 1135103
Change-Id: I7f5f7a63b34e7391873755f0917ac6b971e8b0aa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2459008Reviewed-by: default avatarAaron Leventhal <aleventhal@chromium.org>
Reviewed-by: default avatarAbigail Klein <abigailbklein@google.com>
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#815160}
parent c57b6547
...@@ -877,6 +877,12 @@ void AXEventGenerator::PostprocessEvents() { ...@@ -877,6 +877,12 @@ void AXEventGenerator::PostprocessEvents() {
// First pass through |tree_events_|, remove events that we do not need. // First pass through |tree_events_|, remove events that we do not need.
for (auto& iter : tree_events_) { for (auto& iter : tree_events_) {
AXNode* node = iter.first; AXNode* node = iter.first;
// TODO(http://crbug.com/2279799): remove all of the cases that could
// add a null node to |tree_events|.
if (!node)
continue;
std::set<EventParams>& node_events = iter.second; std::set<EventParams>& node_events = iter.second;
// A newly created live region or alert should not *also* fire a // A newly created live region or alert should not *also* fire a
......
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