Remove callers of deprecatedShadowAncestorNode

Remove callers of deprecatedShadowAncestorNode

This patch removes callers of deprecatedShadowAncestorNode
from DragController and ReplaceSelectionCommand.

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

[deepak.sa, 2014-06-30 15:13:48]

Please have a look. Thanks!

[tkent, 2014-06-30 23:35:50]

https://codereview.chromium.org/360733002/diff/1/Source/core/editing/ReplaceS...
File Source/core/editing/ReplaceSelectionCommand.cpp (right):

https://codereview.chromium.org/360733002/diff/1/Source/core/editing/ReplaceS...
Source/core/editing/ReplaceSelectionCommand.cpp:160: Node* shadowAncestorNode =
editableRoot->shadowHost();
Is it ok to ignore a case where editableRoot->deprecatedShadowAncestorNode()
returns editableRoot?

[deepak.sa, 2014-07-01 05:45:25]

https://codereview.chromium.org/360733002/diff/1/Source/core/editing/ReplaceS...
File Source/core/editing/ReplaceSelectionCommand.cpp (right):

https://codereview.chromium.org/360733002/diff/1/Source/core/editing/ReplaceS...
Source/core/editing/ReplaceSelectionCommand.cpp:160: Node* shadowAncestorNode =
editableRoot->shadowHost();
On 2014/06/30 23:35:50, tkent wrote:
> Is it ok to ignore a case where editableRoot->deprecatedShadowAncestorNode()
> returns editableRoot?

While making the patch i thought it was safe, but to be on the safe side i'll
add the check, so that it will also handle the above case.

[deepak.sa, 2014-07-07 04:12:11]

I have uploaded a new patch.
Please have a look. Thanks!

[tkent, 2014-07-08 00:05:47]

lgtm

[commit-bot: I haz the power, 2014-07-08 00:05:54]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/deepak.sa@samsung.com/360733002/20001

[commit-bot: I haz the power, 2014-07-08 01:14:49]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/15780)

[commit-bot: I haz the power, 2014-07-08 01:55:43]

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

[commit-bot: I haz the power, 2014-07-08 01:55:44]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/15792)

[tkent, 2014-07-08 01:57:07]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2014-07-08 01:58:49]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/deepak.sa@samsung.com/360733002/20001

[commit-bot: I haz the power, 2014-07-08 03:19:54]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/15798)

[commit-bot: I haz the power, 2014-07-08 03:49:49]

Change committed as 177633