Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(81)

Issue 2346473002: Revert of Add fuzzer for AXTree and fix a couple of bugs it found. (Closed)

Created:
4 years, 3 months ago by aberent
Modified:
4 years, 3 months ago
CC:
aboxhall+watch_chromium.org, chromium-reviews, dmazzoni+watch_chromium.org, dtseng+watch_chromium.org, je_julie, nektar+watch_chromium.org, yuzo+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Add fuzzer for AXTree and fix a couple of bugs it found. (patchset #3 id:40001 of https://codereview.chromium.org/2323103002/ ) Reason for revert: Caused Address Sanitizer problem with BrowserAccessibilityManagerTest.TestFatalError BUG=646777 Original issue's description: > Add fuzzer for AXTree and fix a couple of bugs it found. > > BUG=none > > Committed: https://crrev.com/cd760f121c0701c0f7609a3437735751cf4ac416 > Cr-Commit-Position: refs/heads/master@{#418308} TBR=aarya@google.com,aboxhall@chromium.org,mmoroz@chromium.org,ochang@chromium.org,dmazzoni@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=none Committed: https://crrev.com/ce1a186a46de6ade71753efd0cfd578d3a1d124f Cr-Commit-Position: refs/heads/master@{#418541}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -103 lines) Patch
M ui/accessibility/BUILD.gn View 2 chunks +0 lines, -11 lines 0 comments Download
M ui/accessibility/ax_tree.cc View 2 chunks +3 lines, -11 lines 0 comments Download
D ui/accessibility/ax_tree_fuzzer.cc View 1 chunk +0 lines, -52 lines 0 comments Download
M ui/accessibility/ax_tree_unittest.cc View 1 chunk +0 lines, -29 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
aberent
Created Revert of Add fuzzer for AXTree and fix a couple of bugs it found.
4 years, 3 months ago (2016-09-14 13:04:54 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2346473002/1
4 years, 3 months ago (2016-09-14 13:05:11 UTC) #3
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 3 months ago (2016-09-14 13:06:07 UTC) #5
commit-bot: I haz the power
4 years, 3 months ago (2016-09-14 13:07:41 UTC) #7
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/ce1a186a46de6ade71753efd0cfd578d3a1d124f
Cr-Commit-Position: refs/heads/master@{#418541}

Powered by Google App Engine
This is Rietveld 408576698