Remove <video controls> hack.

Remove <video controls> hack.

insertedInto and removedFrom should always be paired, there's no
reason to track this state separately when updating the childShadowRoot
bits.

I also removed the inDocument() checks from ShadowRoot::insertedInto
and removed from since distribution happens in detached subtrees too.
It appears that while they were wrong, it didn't matter because the
state was only used together with containsContentElements() in a check
of the form:

if (!containsShadowRoots() && !containsContentElements())
  return;

which meant that while the containsShadowRoots bit might be wrong false
incorrectly, the containsContentElements is always correct so we'd
never take the early out incorrectly.

R=eseidel@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/ac37e80d199a945bf043146bdea8cab21b6216ec

[esprehn, 2014-12-18 00:39:14]

esprehn@chromium.org changed reviewers:
+ eseidel@chromium.org, ojan@chromium.org

[eseidel, 2014-12-18 00:43:56]

lgtm

[esprehn, 2014-12-18 01:20:18]

Committed patchset #1 (id:1) manually as
ac37e80d199a945bf043146bdea8cab21b6216ec (presubmit successful).