Unify setUpEffectiveNodes

This passes unlink_unittest and node_unittest.

As I said to you, create_link contains a suspicious loop (https://github.com/yosin-chromium/editing/blob/master/commands/create_link_command.js#L248) which may cause an infinite loop and this CL takes forever executing create_link_unittest.

PTAL