[Squad] Don't rely on ancestry for foreignObject LayoutObjectIsNeeded.

The overloaded LayoutObjectIsNeeded was introduced for a crasher which
no longer happens with the overloaded method removed.

This change makes us correctly recalc style for re-attachment of
foreignObject when <g> parent changes display from none to block.

TEST=svg/foreignObject/foreign-object-defs-crash.svg

Bug: 843520
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I78e073ff41090b4b9d53f2f2be921a70ddc993e2
Reviewed-on: https://chromium-review.googlesource.com/1070137
Commit-Queue: Rune Lillesveen <futhark@chromium.org>
Reviewed-by: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#561486}
25 files changed