Don't call Document::nodeWillRemoved() with a node in different document

Don't call Document::nodeWillRemoved() with a node in different document

This patch changes |ContainerNode::willRemoveChild()| not to call
|Document::nodeWillRemoved()| for a node in different document since we
assume |Document::nodeWillRemoved()| will be called for owner document.

This situation can be happened when DOM mutation event handler moves
a removing node to another document via |dispatchChildRemovalEvents()| call in
|ContainerNode::willRemoveChild()| at L439.

BUG=477584
TEST=LayoutTests/fast/dom/Range/range-dom-node-removed-assert.html

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=200791

[yosin_UTC9, 2015-08-18 07:21:01]

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

[yosin_UTC9, 2015-08-18 07:42:24]

PTAL

[tkent, 2015-08-18 07:49:58]

lgtm

https://codereview.chromium.org/1297593004/diff/1/Source/core/dom/ContainerNo...
File Source/core/dom/ContainerNode.cpp (right):

https://codereview.chromium.org/1297593004/diff/1/Source/core/dom/ContainerNo...
Source/core/dom/ContainerNode.cpp:442: // |child| is moved another document by
DOM mutation event handler.
nit: |is moved| -> |was moved to|

https://codereview.chromium.org/1297593004/diff/1/Source/core/dom/ContainerNo...
Source/core/dom/ContainerNode.cpp:1250: // TODO(yosin) We should dispatch
"DOMNodeRemoved" and
DOM mutation event is an obsoleted feature, and we don't want to dispatch
additional events.

[yosin_UTC9, 2015-08-18 08:11:08]

Thanks!

Committing...

https://codereview.chromium.org/1297593004/diff/1/Source/core/dom/ContainerNo...
File Source/core/dom/ContainerNode.cpp (right):

https://codereview.chromium.org/1297593004/diff/1/Source/core/dom/ContainerNo...
Source/core/dom/ContainerNode.cpp:442: // |child| is moved another document by
DOM mutation event handler.
On 2015/08/18 07:49:58, tkent wrote:
> nit: |is moved| -> |was moved to|

Done.

[yosin_UTC9, 2015-08-19 01:04:33]

The CQ bit was checked by yosin@chromium.org

[yosin_UTC9, 2015-08-19 01:04:34]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/1297593004/#ps20001
(title: "2015-08-18T17:03:20 Changes for review comments")

[commit-bot: I haz the power, 2015-08-19 01:04:58]

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

[commit-bot: I haz the power, 2015-08-19 03:51:01]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200791