Reland of Accessibility: Ignore all anonymous blocks

Reland of Accessibility: Ignore all anonymous blocks (patchset #5 id:360001 of https://codereview.chromium.org/2440833002/ )

Reason for revert:
Turns out this wasn't the root cause of the bug; even though it was the result of a bisect; sorry for the noise

Original issue's description:
> Revert of Accessibility: Ignore all anonymous blocks (patchset #15 id:280001 of https://codereview.chromium.org/2387533003/ )
>
> Reason for revert:
> crbug.com/658045
>
> Original issue's description:
> > Accessibility: Ignore all anonymous blocks
> >
> > Anonymous blocks are an implementation detail of the
> > Blink layout tree. They're surprising and not useful in
> > the accessibility tree. Get rid of them by marking them
> > all as ignored.
> >
> > While it may seem counterintuitive, this is actually
> > part of a longer-term plan to get rid of the idea of
> > ignoring nodes for accessibility, this will bring us
> > closer to just exposing the DOM tree plus generated
> > content, like FIrefox and IE do.
> >
> > The code change here is tiny, most of the change is
> > rebasing or updating dozens and dozens of affected
> > tests. I deleted a couple of tests that specifically
> > tested corner cases involving anonymous blocks;
> > the whole point of this change is to not make this a
> > special case anymore.
> >
> > Also updates this script, designed to automatically
> > pull test output from try servers to a local change:
> >
> > tools/accessibility/rebase_dump_accessibility_tree_test.py
> >
> > BUG=651614
> > CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
>
> TBR=aboxhall@chromium.org,dmazzoni@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=651614
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
>
> Committed: https://crrev.com/e4db7ae216888672805a38d8f8fee54c836357c9
> Cr-Commit-Position: refs/heads/master@{#426736}

TBR=aboxhall@chromium.org,dmazzoni@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=651614
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/81dd4d6be788b5cb8871fcecf0b978f9e93d2e72
Cr-Commit-Position: refs/heads/master@{#426888}

[David Tseng, 2016-10-21 20:56:28]

Created Reland of Accessibility: Ignore all anonymous blocks

[David Tseng, 2016-10-21 20:59:47]

Description was changed from

==========
Reland of Accessibility: Ignore all anonymous blocks (patchset #5 id:360001 of
https://codereview.chromium.org/2440833002/ )

Reason for revert:
Turns out this wasn't the root cause of the bug; even though it was the result
of a bisect; sorry for the noise

Original issue's description:
> Revert of Accessibility: Ignore all anonymous blocks (patchset #15 id:280001
of https://codereview.chromium.org/2387533003/ )
> 
> Reason for revert:
> crbug.com/658045
> 
> Original issue's description:
> > Accessibility: Ignore all anonymous blocks
> >
> > Anonymous blocks are an implementation detail of the
> > Blink layout tree. They're surprising and not useful in
> > the accessibility tree. Get rid of them by marking them
> > all as ignored.
> >
> > While it may seem counterintuitive, this is actually
> > part of a longer-term plan to get rid of the idea of
> > ignoring nodes for accessibility, this will bring us
> > closer to just exposing the DOM tree plus generated
> > content, like FIrefox and IE do.
> >
> > The code change here is tiny, most of the change is
> > rebasing or updating dozens and dozens of affected
> > tests. I deleted a couple of tests that specifically
> > tested corner cases involving anonymous blocks;
> > the whole point of this change is to not make this a
> > special case anymore.
> >
> > Also updates this script, designed to automatically
> > pull test output from try servers to a local change:
> >
> > tools/accessibility/rebase_dump_accessibility_tree_test.py
> >
> > BUG=651614
> > CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
> 
> TBR=aboxhall@chromium.org,dmazzoni@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=651614
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
> 
> Committed: https://crrev.com/e4db7ae216888672805a38d8f8fee54c836357c9
> Cr-Commit-Position: refs/heads/master@{#426736}

TBR=aboxhall@chromium.org,dmazzoni@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=651614
==========

to

==========
Reland of Accessibility: Ignore all anonymous blocks (patchset #5 id:360001 of
https://codereview.chromium.org/2440833002/ )

Reason for revert:
Turns out this wasn't the root cause of the bug; even though it was the result
of a bisect; sorry for the noise

Original issue's description:
> Revert of Accessibility: Ignore all anonymous blocks (patchset #15 id:280001
of https://codereview.chromium.org/2387533003/ )
> 
> Reason for revert:
> crbug.com/658045
> 
> Original issue's description:
> > Accessibility: Ignore all anonymous blocks
> >
> > Anonymous blocks are an implementation detail of the
> > Blink layout tree. They're surprising and not useful in
> > the accessibility tree. Get rid of them by marking them
> > all as ignored.
> >
> > While it may seem counterintuitive, this is actually
> > part of a longer-term plan to get rid of the idea of
> > ignoring nodes for accessibility, this will bring us
> > closer to just exposing the DOM tree plus generated
> > content, like FIrefox and IE do.
> >
> > The code change here is tiny, most of the change is
> > rebasing or updating dozens and dozens of affected
> > tests. I deleted a couple of tests that specifically
> > tested corner cases involving anonymous blocks;
> > the whole point of this change is to not make this a
> > special case anymore.
> >
> > Also updates this script, designed to automatically
> > pull test output from try servers to a local change:
> >
> > tools/accessibility/rebase_dump_accessibility_tree_test.py
> >
> > BUG=651614
> > CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
> 
> TBR=aboxhall@chromium.org,dmazzoni@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=651614
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
> 
> Committed: https://crrev.com/e4db7ae216888672805a38d8f8fee54c836357c9
> Cr-Commit-Position: refs/heads/master@{#426736}

TBR=aboxhall@chromium.org,dmazzoni@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=651614
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[David Tseng, 2016-10-21 21:00:20]

The CQ bit was checked by dtseng@chromium.org

[commit-bot: I haz the power, 2016-10-21 21:01:11]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-21 21:06:22]

Committed patchset #2 (id:300001)

[commit-bot: I haz the power, 2016-10-21 21:08:39]

Description was changed from

==========
Reland of Accessibility: Ignore all anonymous blocks (patchset #5 id:360001 of
https://codereview.chromium.org/2440833002/ )

Reason for revert:
Turns out this wasn't the root cause of the bug; even though it was the result
of a bisect; sorry for the noise

Original issue's description:
> Revert of Accessibility: Ignore all anonymous blocks (patchset #15 id:280001
of https://codereview.chromium.org/2387533003/ )
> 
> Reason for revert:
> crbug.com/658045
> 
> Original issue's description:
> > Accessibility: Ignore all anonymous blocks
> >
> > Anonymous blocks are an implementation detail of the
> > Blink layout tree. They're surprising and not useful in
> > the accessibility tree. Get rid of them by marking them
> > all as ignored.
> >
> > While it may seem counterintuitive, this is actually
> > part of a longer-term plan to get rid of the idea of
> > ignoring nodes for accessibility, this will bring us
> > closer to just exposing the DOM tree plus generated
> > content, like FIrefox and IE do.
> >
> > The code change here is tiny, most of the change is
> > rebasing or updating dozens and dozens of affected
> > tests. I deleted a couple of tests that specifically
> > tested corner cases involving anonymous blocks;
> > the whole point of this change is to not make this a
> > special case anymore.
> >
> > Also updates this script, designed to automatically
> > pull test output from try servers to a local change:
> >
> > tools/accessibility/rebase_dump_accessibility_tree_test.py
> >
> > BUG=651614
> > CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
> 
> TBR=aboxhall@chromium.org,dmazzoni@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=651614
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
> 
> Committed: https://crrev.com/e4db7ae216888672805a38d8f8fee54c836357c9
> Cr-Commit-Position: refs/heads/master@{#426736}

TBR=aboxhall@chromium.org,dmazzoni@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=651614
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Reland of Accessibility: Ignore all anonymous blocks (patchset #5 id:360001 of
https://codereview.chromium.org/2440833002/ )

Reason for revert:
Turns out this wasn't the root cause of the bug; even though it was the result
of a bisect; sorry for the noise

Original issue's description:
> Revert of Accessibility: Ignore all anonymous blocks (patchset #15 id:280001
of https://codereview.chromium.org/2387533003/ )
>
> Reason for revert:
> crbug.com/658045
>
> Original issue's description:
> > Accessibility: Ignore all anonymous blocks
> >
> > Anonymous blocks are an implementation detail of the
> > Blink layout tree. They're surprising and not useful in
> > the accessibility tree. Get rid of them by marking them
> > all as ignored.
> >
> > While it may seem counterintuitive, this is actually
> > part of a longer-term plan to get rid of the idea of
> > ignoring nodes for accessibility, this will bring us
> > closer to just exposing the DOM tree plus generated
> > content, like FIrefox and IE do.
> >
> > The code change here is tiny, most of the change is
> > rebasing or updating dozens and dozens of affected
> > tests. I deleted a couple of tests that specifically
> > tested corner cases involving anonymous blocks;
> > the whole point of this change is to not make this a
> > special case anymore.
> >
> > Also updates this script, designed to automatically
> > pull test output from try servers to a local change:
> >
> > tools/accessibility/rebase_dump_accessibility_tree_test.py
> >
> > BUG=651614
> > CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
>
> TBR=aboxhall@chromium.org,dmazzoni@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=651614
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
>
> Committed: https://crrev.com/e4db7ae216888672805a38d8f8fee54c836357c9
> Cr-Commit-Position: refs/heads/master@{#426736}

TBR=aboxhall@chromium.org,dmazzoni@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=651614
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/81dd4d6be788b5cb8871fcecf0b978f9e93d2e72
Cr-Commit-Position: refs/heads/master@{#426888}
==========

[commit-bot: I haz the power, 2016-10-21 21:08:40]

Patchset 2 (id:??) landed as
https://crrev.com/81dd4d6be788b5cb8871fcecf0b978f9e93d2e72
Cr-Commit-Position: refs/heads/master@{#426888}