Completely detach removed nodes.

Completely detach removed nodes.

Scenario:
- a client (ChromeVox), saves a reference to an AutomationNode instance
- a navigation occurs
- a load complete event occurs (reusing the root above)
- ChromeVox checks the saved node to see if it is attached to the same root for the load complete event (it is).
It therefore decides to stay on that node (which has been "removed").

expected:
The node's root property is null.

BUG=605375
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/98f218f9c77c4c1255e522b411994a5e813b8d64
Cr-Commit-Position: refs/heads/master@{#393864}

[David Tseng, 2016-05-12 22:48:56]

Description was changed from

==========
Completely detach removed nodes.
==========

to

==========
Completely detach removed nodes.

Scenario:
- a client (ChromeVox), saves a reference to an AutomationNode instance
- a navigation occurs
- a load complete event occurs (reusing the root above)
- ChromeVox checks the saved node to see if it is attached to the same root for
the load complete event (it is).
It therefore decides to stay on that node (which has been "removed").

expected:
The node's root property is null.

BUG=605375
==========

[David Tseng, 2016-05-12 22:48:56]

dtseng@chromium.org changed reviewers:
+ dmazzoni@chromium.org

[David Tseng, 2016-05-12 23:33:25]

Patchset #1 (id:1) has been deleted

[David Tseng, 2016-05-12 23:34:03]

Patchset #1 (id:20001) has been deleted

[dmazzoni, 2016-05-13 20:06:44]

lgtm

[David Tseng, 2016-05-13 20:46:53]

The CQ bit was checked by dtseng@chromium.org

[commit-bot: I haz the power, 2016-05-13 20:47:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1970153003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1970153003/40001

[commit-bot: I haz the power, 2016-05-13 21:50:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-13 21:50:14]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[David Tseng, 2016-05-16 16:50:29]

Description was changed from

==========
Completely detach removed nodes.

Scenario:
- a client (ChromeVox), saves a reference to an AutomationNode instance
- a navigation occurs
- a load complete event occurs (reusing the root above)
- ChromeVox checks the saved node to see if it is attached to the same root for
the load complete event (it is).
It therefore decides to stay on that node (which has been "removed").

expected:
The node's root property is null.

BUG=605375
==========

to

==========
Completely detach removed nodes.

Scenario:
- a client (ChromeVox), saves a reference to an AutomationNode instance
- a navigation occurs
- a load complete event occurs (reusing the root above)
- ChromeVox checks the saved node to see if it is attached to the same root for
the load complete event (it is).
It therefore decides to stay on that node (which has been "removed").

expected:
The node's root property is null.

BUG=605375
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[David Tseng, 2016-05-16 16:50:36]

The CQ bit was checked by dtseng@chromium.org

[David Tseng, 2016-05-16 16:50:36]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/1970153003/#ps60001
(title: "Fix test.")

[commit-bot: I haz the power, 2016-05-16 16:51:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1970153003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1970153003/60001

[commit-bot: I haz the power, 2016-05-16 18:06:10]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-05-16 18:08:03]

Description was changed from

==========
Completely detach removed nodes.

Scenario:
- a client (ChromeVox), saves a reference to an AutomationNode instance
- a navigation occurs
- a load complete event occurs (reusing the root above)
- ChromeVox checks the saved node to see if it is attached to the same root for
the load complete event (it is).
It therefore decides to stay on that node (which has been "removed").

expected:
The node's root property is null.

BUG=605375
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Completely detach removed nodes.

Scenario:
- a client (ChromeVox), saves a reference to an AutomationNode instance
- a navigation occurs
- a load complete event occurs (reusing the root above)
- ChromeVox checks the saved node to see if it is attached to the same root for
the load complete event (it is).
It therefore decides to stay on that node (which has been "removed").

expected:
The node's root property is null.

BUG=605375
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/98f218f9c77c4c1255e522b411994a5e813b8d64
Cr-Commit-Position: refs/heads/master@{#393864}
==========

[commit-bot: I haz the power, 2016-05-16 18:08:04]

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