Node.replaceChild with an empty DocumentFragment should not dispatch DOMSubtreeModified twice.

Node.replaceChild with an empty DocumentFragment should not dispatch DOMSubtreeModified twice.

We should check the emptiness.  The new behavior matches to Firefox and Edge.

* LayoutTests/fast/loader/delete-inside-cancelTimer.html
  The replaceChild behavior change affects the output of the test. This CL
  doesn't update the result and removes the test entirely because
  - It's a crash test for CachedResource. [1]
  - Now the class is on Oilpan heap, and the test isn't helpful at all.

[1] https://chromium.googlesource.com/chromium/src/+/5e8f72d1c3a42b5ab0c727f52c24009b36721255

BUG=645441
Committed: https://crrev.com/8f99074d5ad2b2ef95d3192b37884dc434e71ff3
Cr-Commit-Position: refs/heads/master@{#417871}

[tkent, 2016-09-09 12:51:28]

The CQ bit was checked by tkent@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-09 12:51:36]

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

[tkent, 2016-09-09 13:47:36]

tkent@chromium.org changed reviewers:
+ hayato@chromium.org

[tkent, 2016-09-09 13:47:37]

hayato@, would you review this please?

[commit-bot: I haz the power, 2016-09-09 14:29:02]

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

[commit-bot: I haz the power, 2016-09-09 14:29:03]

Dry run: This issue passed the CQ dry run.

[hayato, 2016-09-12 03:53:38]

lgtm

[tkent, 2016-09-12 04:08:36]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-09-12 04:08:40]

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-09-12 05:41:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-12 05:43:31]

Description was changed from

==========
Node.replaceChild with an empty DocumentFragment should not dispatch
DOMSubtreeModified twice.

We should check the emptiness.  The new behavior matches to Firefox and Edge.

* LayoutTests/fast/loader/delete-inside-cancelTimer.html
  The replaceChild behavior change affects the output of the test. This CL
  doesn't update the result and removes the test entirely because
  - It's a crash test for CachedResource. [1]
  - Now the class is on Oilpan heap, and the test isn't helpful at all.

[1]
https://chromium.googlesource.com/chromium/src/+/5e8f72d1c3a42b5ab0c727f52c24...

BUG=645441
==========

to

==========
Node.replaceChild with an empty DocumentFragment should not dispatch
DOMSubtreeModified twice.

We should check the emptiness.  The new behavior matches to Firefox and Edge.

* LayoutTests/fast/loader/delete-inside-cancelTimer.html
  The replaceChild behavior change affects the output of the test. This CL
  doesn't update the result and removes the test entirely because
  - It's a crash test for CachedResource. [1]
  - Now the class is on Oilpan heap, and the test isn't helpful at all.

[1]
https://chromium.googlesource.com/chromium/src/+/5e8f72d1c3a42b5ab0c727f52c24...

BUG=645441

Committed: https://crrev.com/8f99074d5ad2b2ef95d3192b37884dc434e71ff3
Cr-Commit-Position: refs/heads/master@{#417871}
==========

[commit-bot: I haz the power, 2016-09-12 05:43:32]

Patchset 1 (id:??) landed as
https://crrev.com/8f99074d5ad2b2ef95d3192b37884dc434e71ff3
Cr-Commit-Position: refs/heads/master@{#417871}